Skip to content

Commit 565683b

Browse files
committed
refactor(TimePicker): enhance component styles and add new variables to manage cell width and height
1 parent 38d495b commit 565683b

File tree

2 files changed

+12
-9
lines changed

2 files changed

+12
-9
lines changed

scss/_time-picker.scss

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,8 @@
5151
--#{$prefix}time-picker-dropdown-box-shadow: #{$time-picker-dropdown-box-shadow};
5252
--#{$prefix}time-picker-roll-col-border-width: #{$time-picker-roll-col-border-width};
5353
--#{$prefix}time-picker-roll-col-border-color: #{$time-picker-roll-col-border-color};
54+
--#{$prefix}time-picker-roll-cell-width: #{$time-picker-roll-cell-width};
55+
--#{$prefix}time-picker-roll-cell-height: #{$time-picker-roll-cell-height};
5456
--#{$prefix}time-picker-roll-cell-hover-color: #{$time-picker-roll-cell-hover-color};
5557
--#{$prefix}time-picker-roll-cell-hover-bg: #{$time-picker-roll-cell-hover-bg};
5658
--#{$prefix}time-picker-roll-cell-selected-color: #{$time-picker-roll-cell-selected-color};
@@ -292,7 +294,7 @@
292294
}
293295

294296
.time-picker-roll-col {
295-
height: calc(8 * 32px); // stylelint-disable-line
297+
height: calc(8 * var(--#{$prefix}time-picker-roll-cell-height)); // stylelint-disable-line
296298
overflow: scroll;
297299

298300
-ms-overflow-style: none; /* Internet Explorer 10+ */
@@ -309,10 +311,11 @@
309311
}
310312

311313
.time-picker-roll-cell {
312-
width: 50px;
313-
height: 32px;
314-
padding-inline: $spacer * .5 $spacer;
315-
line-height: 32px;
314+
display: flex;
315+
align-items: center;
316+
justify-content: center;
317+
width: var(--#{$prefix}time-picker-roll-cell-width);
318+
height: var(--#{$prefix}time-picker-roll-cell-height);
316319
cursor: pointer;
317320

318321
&:hover,
@@ -336,10 +339,8 @@
336339
}
337340
}
338341

339-
&:last-child::after {
340-
display: block;
341-
height: calc(7 * 32px); // stylelint-disable-line
342-
content: "";
342+
&:last-child {
343+
margin-bottom: calc(7 * var(--#{$prefix}time-picker-roll-cell-height)); // stylelint-disable-line
343344
}
344345
}
345346

scss/_variables.scss

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2686,6 +2686,8 @@ $time-picker-footer-border-color: var(--#{$prefix}border-color) !defau
26862686

26872687
$time-picker-roll-col-border-width: var(--#{$prefix}border-width) !default;
26882688
$time-picker-roll-col-border-color: var(--#{$prefix}border-color) !default;
2689+
$time-picker-roll-cell-width: 3rem !default;
2690+
$time-picker-roll-cell-height: 2rem !default;
26892691
$time-picker-roll-cell-hover-color: var(--#{$prefix}body-color) !default;
26902692
$time-picker-roll-cell-hover-bg: var(--#{$prefix}tertiary-bg) !default;
26912693
$time-picker-roll-cell-selected-color: var(--#{$prefix}white) !default;

0 commit comments

Comments
 (0)