status-im/spiff-arena
2
0
Fork 0
mirror of https://github.com/status-im/spiff-arena.git synced 2025-02-28 09:20:32 +00:00
Code Issues Packages Projects Releases Wiki Activity

Revert "make all browsers act the same way on our site w/ jasquat"

Browse Source 
This reverts commit ebc10041018ddd3152eb122a429237555010d021.
This commit is contained in:
burnettk 2023-05-08 23:04:16 -04:00
parent ebc1004101
commit 13f51f9f82
No known key found for this signature in database
1 changed files with 0 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
spiffworkflow-frontend/src/themes/carbon/SelectWidget/SelectWidget.tsx
View File

@ -56,17 +56,6 @@ function SelectWidget({
errorMessageForField = rawErrors[0];
}
// ok. so in safari, the select widget showed the first option, whereas in chrome it forced you to select an option.
// this change causes chrome to also show the first option. now all browsers are consistent on our site. you no longer have to select anything.
// setting the value prop causes it to show the first option.
// calling onChange actually gets it set in the form data.
if (value === undefined) {
if (enumOptions) {
value = enumOptions[0].value;
onChange(value);
}
}
// maybe use placeholder somehow. it was previously jammed into the helperText field,
// but allowing ui:help to grab that spot seems much more appropriate.