See #1493324: Inline form errors for accessibility and UX

Below is an example of the properly themed inline error messages.
Only local images are allowed.

CommentFileSizeAuthor
#9 support_inline_error-2548187-9.patch5.01 KBs_leu
#9 support_inline_error-2548187-9.interdiff.txt1.53 KBs_leu
#4 Screen Shot 2015-08-08 at 8.00.04 PM.png12.99 KBneardark
#3 support_inline_error-2548187-3.patch3.49 KBneardark

Comments

neardark created an issue. See original summary.

neardark’s picture

neardark commented
Issue summary: View changes
neardark’s picture

neardark commented
StatusFileSize
new support_inline_error-2548187-3.patch3.49 KB

I've used the bootstrap class text-danger to denote inline error messages. This patch also works with the Drupal Bootstrap setting for setting required fields to immediately show an error message and clear upon input. However, this doesn't work with ckEditor fields. This will be addressed in #2548189: Support inline error messages for ckEditor fields

neardark’s picture

neardark commented
Issue summary: View changes
StatusFileSize
new Screen Shot 2015-08-08 at 8.00.04 PM.png12.99 KB
neardark’s picture

neardark commented
Issue summary: View changes
Status: Active » Needs review
neardark’s picture

neardark commented
Issue summary: View changes
neardark’s picture

neardark commented
Related issues: +#2548189: Support inline error messages for ckEditor fields
neardark’s picture

neardark commented
Status: Needs review » Needs work

Setting this to needs work as some select items are failing, such as Radios/Checkboxes.

s_leu’s picture

s_leu commented
Status: Needs work » Needs review
StatusFileSize
new support_inline_error-2548187-9.interdiff.txt1.53 KB
new support_inline_error-2548187-9.patch5.01 KB

Here's a patch that also considers fields with checkboxes/radios widgets.

markhalliwell’s picture

markhalliwell
Primary language English
commented
Assigned: neardark » Unassigned
Status: Needs review » Closed (duplicate)
Related issues: +#2266079: Fix form elements

Pretty sure this related issue/commits fixed this. If not, please re-open.