Skip to content
This repository was archived by the owner on Oct 8, 2021. It is now read-only.

Accessibly hiding labels isn't working consistently #3587

Closed
toddparker opened this issue Feb 15, 2012 · 4 comments
Closed

Accessibly hiding labels isn't working consistently #3587

toddparker opened this issue Feb 15, 2012 · 4 comments
Assignees

Comments

@toddparker
Copy link
Contributor

Adding the class="ui-hidden-accessible" class to a class="ui-hidden-accessible" should hide it. Oddly, this works for text, search inputs, and sliders:
http://jquerymobile.com/test/docs/toolbars/bars-fixed-forms-b.html
http://jquerymobile.com/test/docs/toolbars/bars-fixed-forms-c.html
http://jquerymobile.com/test/docs/toolbars/bars-fixed-forms-f.html

...but not for horizontally grouped radios and checks which sort of makes sense because these use a fieldset and legend structure:
http://jquerymobile.com/test/docs/toolbars/bars-fixed-forms-d.html
http://jquerymobile.com/test/docs/toolbars/bars-fixed-forms-e.html

Select labels are also not hiding either, not sure why:
http://jquerymobile.com/test/docs/toolbars/bars-fixed-forms-g.html

I just checked and this doesn't seem to be limited to labels in toolbars. Hiding is also broken with these elements int he content area too.

@ghost ghost assigned Wilto Feb 15, 2012
@Wilto
Copy link
Contributor

Wilto commented Feb 20, 2012

This issue will be addressed by the addition of the functionality described here:
#3577

We’ll need to add a suffixed version of the ui-hidden-accessible class once the above fix is in place, so I’ll leave this issue open for the time being.

@toddparker
Copy link
Contributor Author

Moving this to the 1.2 milestone since we're not going to get the class wrapper feature in for 1.1.

@arschmitz
Copy link
Contributor

Since we moved #3577 to 1.4 im going to move this one as well

@ghost ghost assigned jaspermdegroot and Wilto Dec 6, 2012
@jaspermdegroot
Copy link
Contributor

Actually I fixed this already 6 months ago with commit 15c355d.
The reason you still saw the label of the select on that demo page was wrong markup: the label had two class attributes. I will fix that.

Closing as fixed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants