Closed (fixed)
Project:
Drupal core
Version:
8.4.x-dev
Component:
inline_form_errors.module
Priority:
Normal
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
26 Jul 2017 at 18:33 UTC
Updated:
16 Aug 2017 at 05:20 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
dmsmidtLet's at least get the documentation update issue in #2888189: Update hook_help and DO docs for Inline Form Errors and other RTBC's in.
Comment #3
dmsmidtComment #5
rootworkIs this still true/how we want to phrase this for a core module?
Comment #6
dmsmidt@rootwork, it has just been changed, that's why the patch doesn't apply I guess. See #2888189: Update hook_help and DO docs for Inline Form Errors.
Comment #7
dmsmidtDouble post.
Comment #8
rootworkOh gotcha, sorry for the noise then :)
Comment #9
naiduharish commentedModified patch.
Comment #11
naiduharish commentedSorry for wrong file, uploading correct file.
Comment #12
dries commented+1 from me to move forward with marking this stable, assuming the other committers agree this is ready.
Comment #13
skaught#11 manually tested on simplytest.me. looks good.
Comment #14
xjmAll the must-haves are complete. We have a couple should-have issues for the upcoming alpha/beta deadline, two of which are RTBC:
So that part seems ready to go.
We have most of the needed signoffs, from other committers, from the module maintainers, and from Dries. The outstanding must-have from #2504847: [meta] Roadmap for stabilizing Inline Form Errors module (IFE) is full accessibility review. Based on ongoing review from @andrewmacpherson, I think this has probably already happened. However, can we get signoffs from a usability and accessibility maintainer explicitly on the issue so we can mark it stable for this release! The issue can be set RTBC when those signoffs are here.
Edit: Royoy is on vacation so I'd suggest joining a UX meeting or pinging Bojhan for an overall review of the module or simply signoff if it was recently reviewed.
Thanks everyone! This is amazing work.
Comment #15
andrewmacpherson commentedAccessibility maintainer review, short version: Yes indeed I've had my head in this all year, testing various issues most weeks. I've been reporting various bugs/quirks along the way. Most of these have been fixed already, and a handful have been classed as non-blocking follow-ups at #2504847: [meta] Roadmap for stabilizing Inline Form Errors module (IFE).
This is so exciting! IFE module goes the extra mile towards addressing WCAG input assistance guidelines in an elegant way which benefits lots of different users and scenarios. For instance, the error jump-links in the help region are categorized merely as an "additional, advisory" technique in WCAG, which just goes to show how far Drupal's accessibility approach has come.
Comment #17
dmsmidtComment #18
xjmI talked to @dmsmidt about the usability review, and he said that each time an issue introduced new designs, it was reviewed at that time in UX meetings. So hopefully that covers the usability gate.
I've marked the module beta at least for now. Looking over the last few should-haves and their status now.
Comment #21
xjmAlright, committed and pushed to 8.5.x and 8.4.x! Hopefully we'll see the three should-have issues land soon as well.
Comment #22
xjmComment #23
dmsmidtThanks everybody who helped realizing this!