Problem/Motivation
Noticed a form error, in a special case the for attribute was missing from a datetime label.
Spun from another twig fix #3568044: wet-boew compliance - form-required and js-form-required classes are redundant to required
Steps to reproduce
Theme the node form with wxt_bootstrap
Might occur in other cases.
Proposed resolution
css to ensure datepicker displays correctly
twig to ensure datetime wrapper label has valid for attribute (wcag compliance)
Here's the latest patch.
Remaining tasks
Review patch for wxt_bootstrap
User interface changes
Resolve browser console error(s).
API changes
Twig change
Data model changes
N/A
| Comment | File | Size | Author |
|---|---|---|---|
| #18 | wxt_bootstrap-element-lable-and-datetime-wrapper-3568863-18.patch | 3.86 KB | joseph.olstad |
Comments
Comment #2
joseph.olstadPatch file for wxt_bootstrap
Comment #3
joseph.olstadComment #4
joseph.olstadI have an improvement/update for this, coming shortly.
Comment #5
joseph.olstadComment #6
joseph.olstadforget the before, that was a partial patch #5
With patch #5

Comment #7
joseph.olstadah w00ps, actually both of these patches are needed.
Comment #8
joseph.olstadComment #9
joseph.olstadok, I combined both patches into one.
Comment #10
joseph.olstadComment #11
joseph.olstadstill working on this. Will upload a new patch
Comment #12
joseph.olstadOk this seems to resolve all the date field issues. The other related fields are functioning afaik.
Comment #13
joseph.olstadScreenshots using twig from patch 12
Comment #14
joseph.olstadstill reviewing
Comment #15
joseph.olstadEnsures datepicker date fields month widget shows up to the right of the input box.
Comment #16
joseph.olstadComment #17
joseph.olstadok, going to roll that css into the patch. I also have a helper module for webforms that I will add in a new issue.
Comment #18
joseph.olstadNew patch, includes the css this time.
Comment #19
joseph.olstadComment #20
joseph.olstadComment #21
smulvih2Tested this on 6.3.x, and it fixes the WCAG issue as advertised. Pushed a new wxt_bootstrap version 11.3.x and pulled into the latest WxT 6.3.x.