Skip to content

Commit 24a9fd1

Browse files
authored
refactor(ui5-input): move styles to base theme (#170)
1 parent 2e0f8a0 commit 24a9fd1

File tree

3 files changed

+60
-76
lines changed

3 files changed

+60
-76
lines changed

packages/main/src/themes/base/Input.less

Lines changed: 49 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,18 @@
1010
/* Base theme */
1111
/* ================================ */
1212

13+
@_ui5_input_height: 2.5rem;
14+
@_ui5_input_compact_height: 1.625rem;
15+
@_ui5_input_wrapper_border_radius: 0;
16+
@_ui5_input_state_border_width: 0.125rem;
17+
@_ui5_input_error_font_weight: normal;
18+
@_ui5_input_focus_border_width: 1px;
19+
@_ui5_input_error_warning_border_style: solid;
20+
@_ui5_input_error_warning_font_style: normal;
21+
@_ui5_input_disabled_color: @sapUiContentDisabledTextColor;
22+
@_ui5_input_disabled_font_weight: normal;
23+
@_ui5_input_disabled_border_color: @sapUiFieldBorderColor;
24+
@_ui5_input_disabled_background: @sapUiFieldBackground;
1325
@sap_wc_input_disabled_opacity: 1;
1426
@sap_wc_input_icon_min_width: 2.375rem;
1527

@@ -24,7 +36,7 @@ ui5-input {
2436
}
2537

2638
.sapWCInputBase {
27-
height: 2.5rem;
39+
height: @_ui5_input_height;
2840
background: transparent;
2941
position: relative;
3042
display: inline-block;
@@ -36,12 +48,12 @@ ui5-input {
3648
&.sapWCFocus .sapWCInputBaseContentWrapper::after {
3749
content: "";
3850
position: absolute;
39-
width: calc(~"100% - 0.25rem");
40-
height: calc(~"100% - 0.25rem");
41-
border: 1px dotted @sapUiContentFocusColor;
51+
border: @_ui5_input_focus_border_width dotted @sapUiContentFocusColor;
4252
pointer-events: none;
4353
top: 1px;
4454
left: 1px;
55+
right: 1px;
56+
bottom: 1px;
4557
}
4658

4759
&.sapWCInputBaseDisabled {
@@ -94,7 +106,7 @@ ui5-input {
94106
outline: none;
95107
background-color: @sapUiFieldBackground;
96108
border: 1px solid @sapUiFieldBorderColor;
97-
border-radius: 0;
109+
border-radius: @_ui5_input_wrapper_border_radius;
98110

99111
&.sapWCInputBaseDisabledWrapper {
100112
pointer-events: none;
@@ -111,19 +123,47 @@ ui5-input {
111123
}
112124
}
113125

126+
.sapWCInputBaseDisabledWrapper {
127+
background: @sapUiFieldReadOnlyBackground;
128+
border-color: @sapUiFieldReadOnlyBorderColor;
129+
-webkit-text-fill-color: @sapUiContentDisabledTextColor;
130+
131+
& .sapWCInputBaseInner {
132+
color: @sapUiContentDisabledTextColor;
133+
}
134+
}
135+
136+
.sapWCInputBaseContentWrapperState {
137+
border-width: @_ui5_input_state_border_width;
138+
}
139+
140+
.sapWCInputBaseContentWrapperError .sapWCInputBaseInner,
141+
.sapWCInputBaseContentWrapperWarning .sapWCInputBaseInner {
142+
font-style: @_ui5_input_error_warning_font_style;
143+
}
144+
145+
.sapWCInputBaseContentWrapperError .sapWCInputBaseInner {
146+
font-weight: @_ui5_input_error_font_weight;
147+
}
148+
114149
.sapWCInputBaseContentWrapperError:not(.sapWCInputBaseReadonlyWrapper) {
115150
background-color: @sapUiFieldInvalidBackground;
116-
border: 0.125rem solid @sapUiFieldInvalidColor;
151+
border-color: @sapUiFieldInvalidColor;
152+
}
153+
154+
.sapWCInputBaseContentWrapperError:not(.sapWCInputBaseReadonlyWrapper):not(.sapWCInputBaseDisabledWrapper),
155+
.sapWCInputBaseContentWrapperWarning:not(.sapWCInputBaseReadonlyWrapper):not(.sapWCInputBaseDisabledWrapper) {
156+
border-style: @_ui5_input_error_warning_border_style;
117157
}
118158

119159
.sapWCInputBaseContentWrapperWarning:not(.sapWCInputBaseReadonlyWrapper) {
120160
background-color: @sapUiFieldWarningBackground;
121-
border: 0.125rem solid @sapUiFieldWarningColor;
161+
border-color: @sapUiFieldWarningColor;
122162
}
123163

124164
.sapWCInputBaseContentWrapperSuccess:not(.sapWCInputBaseReadonlyWrapper) {
125165
background-color: @sapUiFieldSuccessBackground;
126-
border: 0.125rem solid @sapUiFieldSuccessColor;
166+
border-color: @sapUiFieldSuccessColor;
127167
}
128168

129169
/* Remove IE's defaut clear button */
@@ -134,7 +174,7 @@ ui5-input {
134174

135175
.sapUiSizeCompact {
136176
& .sapWCInputBase {
137-
height: 1.625rem;
177+
height: @_ui5_input_compact_height;
138178
}
139179

140180
& .sapWCInputBaseInner {

packages/main/src/themes/sap_belize_hcb/Input.less

Lines changed: 9 additions & 48 deletions
Original file line numberDiff line numberDiff line change
@@ -10,51 +10,12 @@
1010
@import "./base.less";
1111
@import "./global.less";
1212

13-
.sapWCInputBase {
14-
&.sapWCFocus .sapWCInputBaseContentWrapper::after {
15-
width: calc(~"100% - 0.375rem");
16-
height: calc(~"100% - 0.375rem");
17-
border: 0.125rem dotted @sapUiContentFocusColor;
18-
}
19-
}
20-
21-
.sapWCInputBaseContentWrapper {
22-
23-
& .sapWCInputBaseInner {
24-
color: @sapUiFieldTextColor;
25-
}
26-
27-
&.sapWCInputBaseContentWrapperError:not(.sapWCInputBaseReadonlyWrapper):not(.sapWCInputBaseDisabledWrapper),
28-
&.sapWCInputBaseContentWrapperWarning:not(.sapWCInputBaseReadonlyWrapper):not(.sapWCInputBaseDisabledWrapper) {
29-
border-style: dashed;
30-
border-width: 1px;
31-
}
32-
33-
&.sapWCInputBaseContentWrapperSuccess,
34-
&.sapWCInputBaseContentWrapperError,
35-
&.sapWCInputBaseContentWrapperWarning {
36-
border-width: 1px;
37-
}
38-
39-
&.sapWCInputBaseContentWrapperError .sapWCInputBaseInner {
40-
font-weight: bold;
41-
}
42-
43-
&.sapWCInputBaseContentWrapperError,
44-
&.sapWCInputBaseContentWrapperWarning {
45-
& .sapWCInputBaseInner {
46-
font-style: italic;
47-
}
48-
}
49-
50-
&.sapWCInputBaseDisabledWrapper {
51-
background: @sapUiFieldReadOnlyBackground;
52-
border-color: @sapUiFieldReadOnlyBorderColor;
53-
font-weight: normal;
54-
-webkit-text-fill-color: @sapUiContentDisabledTextColor;
55-
56-
& .sapWCInputBaseInner {
57-
color: @sapUiContentDisabledTextColor;
58-
}
59-
}
60-
}
13+
@_ui5_input_focus_border_width: 0.125rem;
14+
@_ui5_input_state_border_width: 1px;
15+
@_ui5_input_error_warning_border_style: dashed;
16+
@_ui5_input_error_warning_font_style: italic;
17+
@_ui5_input_error_font_weight: bold;
18+
@_ui5_input_disabled_color: @sapUiContentDisabledTextColor;
19+
@_ui5_input_disabled_font_weight: normal;
20+
@_ui5_input_disabled_border_color: @sapUiFieldReadOnlyBorderColor;
21+
@_ui5_input_disabled_background: @sapUiFieldReadOnlyBackground;

packages/main/src/themes/sap_fiori_3/Input.less

Lines changed: 2 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -14,23 +14,6 @@
1414
/* Belize theme */
1515
/* ================================ */
1616

17+
@_ui5_input_height: 2.25rem;
1718
@sap_wc_input_disabled_opacity: 0.4;
18-
19-
.sapWCInputBase {
20-
height: 2.25rem;
21-
}
22-
23-
.sapWCInputBaseContentWrapper {
24-
border-radius: 0.125rem;
25-
26-
&.sapWCInputBaseContentWrapperState {
27-
border-width: 0.125rem;
28-
}
29-
30-
}
31-
32-
.sapUiSizeCompact {
33-
& .sapWCInputBase {
34-
height: 1.625rem;
35-
}
36-
}
19+
@_ui5_input_wrapper_border_radius: 0.125rem;

0 commit comments

Comments
 (0)