- Issue created by @andre.bonon
- 🇮🇳India dineshkumarbollu
provided patch, added
'form-type-' ~ type|clean_class
in stable9 theme.please review. - last update
over 1 year ago 29,465 pass - Status changed to RTBC
over 1 year ago 3:35pm 16 August 2023 - 🇧🇷Brazil andre.bonon
@dineshkumarbollu it works. Thanks for providing a patch for that.
- Status changed to Needs review
over 1 year ago 5:46pm 16 August 2023 - 🇫🇮Finland lauriii Finland
The contributed modules should be using
js-form-type-{type}
class. Looks like there are quite a few uses of this class in contrib so maybe the pragmatic solution would be to just add the class to Stable. Tagging with FEFM review tag to get their feedback. - 🇺🇸United States bnjmnm Ann Arbor, MI
I was trying to put myself in the shoes of a stark-theme-diehard who wants no excess markup, but this is a convention intended for functionality, not presentation, and I'd hope that theoretical purist would be OK with the markup being a little busier in order for contrib. modules to work as expected. This is a nice addition for the next minor.
- Status changed to RTBC
over 1 year ago 7:27pm 5 September 2023 - 🇫🇮Finland lauriii Finland
Looks like this would be ready to go then! Should we open a CR for this?
- last update
about 1 year ago 29,482 pass - last update
about 1 year ago 29,482 pass - last update
about 1 year ago Build Successful - last update
about 1 year ago 29,642 pass - last update
about 1 year ago 29,643 pass - last update
about 1 year ago 29,643 pass - Status changed to Fixed
about 1 year ago 9:25am 2 October 2023 Automatically closed - issue fixed for 2 weeks with no activity.