Closed (fixed)
Project:
Panopoly
Version:
7.x-1.x-dev
Component:
Admin
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
19 Dec 2014 at 17:47 UTC
Updated:
30 Mar 2015 at 13:22 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
brandy.brown commentedare you sure that screen readers will still read it if it's set to invisible?
Comment #2
dsnopekI haven't actually tested it, but that's exactly what
'#title_display' => 'invisible'is for!From the docs (emphasis added):
https://api.drupal.org/api/drupal/developer%21topics%21forms_api_referen...
Comment #3
brandy.brown commentedperfect, thanks.
Comment #4
brandy.brown commentedI think this covers it, but would love for others to check it.
Comment #5
brandy.brown commentedComment #6
brandy.brown commentedSorry, missed one of the display nones ... fixed here.
Comment #7
dsnopekThanks for the patch! I tested and everything worked great except for the 'field_featured_categories' bit you added in #6:
This should actually be:
Without the
['0']bit.So, I fixed that and also made a couple other changes to the patch:
'#title_display'rather than CSS.hook_form_alter()is being run for all node edit forms, not just ones that definitely have these fields, so we always need to check if the field exists before changing it. We're already testing for the existance of all these fields, so I just moved the new bits inside the rightif()statements.Anyway, new patch and an interdiff showing my changes is attached! I'd appreciate some testing and review to make sure my changes are actually doing what they should. :-)
Comment #8
dsnopekComment #9
mglamanNo regressions, works as expected to proper utilize element-invisible.
Comment #10
dsnopekThanks, @mglaman, for testing! And thanks again to @brandy.brown for pushing this forward! Committed. :-)
Comment #13
dsnopekHrm, I just realized that this didn't remove the Javascript in panopoly-admin.js which clears out the Body label! So, the Body label is still not accessible. We can fix this over in #2461077: Hiding the Body field label breaks the summary toggle because that's already a follow-up to this issue.
EDIT: Copy & pasted wrong issue number. :-)