Skip to content

Commit 197fef1

Browse files
fix(ui5-step-input): align the min width to visual specification (#4884)
Aligning the minimum width of the step input control to specification. There is no specific min width by the design team, but by their suggestions we have to combine min width of input field, two buttons and also the borders of the buttons inside. Related to: #4786
1 parent 3e715c4 commit 197fef1

File tree

3 files changed

+6
-1
lines changed

3 files changed

+6
-1
lines changed

packages/main/src/themes/StepInput.css

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
box-sizing: border-box;
1515
height: var(--_ui5_input_height);
1616
position: relative;
17+
min-width: var(--_ui5_step_input_min_width);
1718
}
1819

1920
:host .ui5-step-input-input {
@@ -144,7 +145,7 @@
144145
box-sizing: border-box;
145146
vertical-align: top;
146147
margin-top: var(--_ui5_step_input_input_margin_top);
147-
min-width: 9.125rem;
148+
min-width: var(--_ui5_step_input_min_width);
148149
position: relative;
149150
padding: 0px 2.5rem 0px 2.4375rem;
150151
outline: none;

packages/main/src/themes/base/StepInput-parameters.css

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,4 +20,5 @@
2020
--_ui5_step_input_border_color_hover: var(--sapField_Hover_Background);
2121
--_ui5_step_input_border_hover: 1px solid var(--sapField_Hover_BorderColor);
2222
--_ui5_input_input_background_color: var(--sapField_InvalidBackground);
23+
--_ui5_step_input_min_width: 7.25rem;
2324
}

packages/main/src/themes/base/sizes-parameters.css

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -282,6 +282,9 @@
282282
--_ui5_slider_progress_outline_offset: -0.625rem;
283283
--_ui5_slider_outer_height: 1.3125rem;
284284

285+
/* StepInput */
286+
--_ui5_step_input_min_width: 6rem;
287+
285288
/* Table */
286289
--_ui5_load_more_text_height: 2.625rem;
287290
--_ui5_load_more_text_font_size: var(--sapFontSize);

0 commit comments

Comments
 (0)