Skip to content

Commit 0a8a078

Browse files
committed
copy the same error style as numeric input on focu
1 parent d00ceec commit 0a8a078

4 files changed

Lines changed: 15 additions & 20 deletions

File tree

src/components/widgets/DateTimePicker.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,10 @@ export default class DateTimePicker extends Component {
2323
calendarOpen: false,
2424
dateInputClassName: isValidDate
2525
? 'datetimepicker-container-date-input'
26-
: 'datetimepicker-container-date-input--error',
26+
: 'datetimepicker-container-date-input +error',
2727
timeInputClassName: isValidTime
2828
? 'datetimepicker-container-time-input'
29-
: 'datetimepicker-container-time-input--error',
29+
: 'datetimepicker-container-time-input +error',
3030
timeValue: time,
3131
dateValue: date,
3232
};
@@ -144,7 +144,7 @@ export default class DateTimePicker extends Component {
144144

145145
if (!isValidTime) {
146146
this.setState({
147-
timeInputClassName: 'datetimepicker-container-time-input--error',
147+
timeInputClassName: 'datetimepicker-container-time-input +error',
148148
timeValue: value,
149149
});
150150
}
@@ -179,7 +179,7 @@ export default class DateTimePicker extends Component {
179179
if (!isValidDate) {
180180
this.setState({
181181
dateValue: value,
182-
dateInputClassName: 'datetimepicker-container-date-input--error',
182+
dateInputClassName: 'datetimepicker-container-date-input +error',
183183
});
184184
}
185185
}

src/styles/_helpers.scss

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -8,25 +8,30 @@
88
color: var(--color-text-active);
99
}
1010

11-
@function spacing($name, $true-val:false) {
11+
.\+error {
12+
border-color: var(--color-sienna) !important;
13+
outline-color: var(--color-sienna) !important;
14+
}
15+
16+
@function spacing($name, $true-val: false) {
1217
@if $true-val == true {
13-
@return map-get($spacings,$name);
18+
@return map-get($spacings, $name);
1419
} @else {
1520
@return var(--spacing-#{$name});
1621
}
1722
}
1823

19-
@function font($name, $true-val:false) {
24+
@function font($name, $true-val: false) {
2025
@if $true-val == true {
21-
@return map-get($fonts,$name);
26+
@return map-get($fonts, $name);
2227
} @else {
2328
@return var(--font-#{$name});
2429
}
2530
}
2631

27-
@function color($name, $true-val:false) {
32+
@function color($name, $true-val: false) {
2833
@if $true-val == true {
29-
@return map-get($colors,$name);
34+
@return map-get($colors, $name);
3035
} @else {
3136
@return var(--color-#{$name});
3237
}

src/styles/components/widgets/_datetimepicker.scss

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -24,15 +24,6 @@
2424
margin-top: -20px;
2525
}
2626

27-
.datetimepicker-container-date-input--error {
28-
margin-top: -20px;
29-
border-color: var(--color-sienna);
30-
}
31-
32-
.datetimepicker-container-time-input--error {
33-
border-color: var(--color-sienna);
34-
}
35-
3627
.datetimepicker-container-icons {
3728
display: inline-block;
3829
margin-top: 2px;

src/styles/components/widgets/_text-input.scss

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,5 +32,4 @@
3232
font-size: inherit;
3333
color: inherit;
3434
font-family: inherit;
35-
outline: none;
3635
}

0 commit comments

Comments
 (0)