New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
7771 accessibility update form labels #8042
Conversation
src/main/webapp/dataverseuser.xhtml
Outdated
@@ -571,7 +571,7 @@ | |||
<div class="form-horizontal"> | |||
<div class="form-group" jsf:rendered="#{DataverseUserPage.editMode == 'CREATE' or DataverseUserPage.editMode == 'EDIT'}"> | |||
<!-- Username --> | |||
<label for="userNameEmail" class="col-sm-3 control-label"> | |||
<label for="dataverseUserForm:accountInfoView:userName" class="col-sm-3 control-label"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sekmiller rather than add the whole name (though dropping the mail was definitely needed), did you try using <h:outputLabel> or <p:outputLabel>? If that works, that would be better, in case the structure changes some day.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't try it originally because I wasn't sure if I could replicate the formatting. I'll see if I can get it to look right with outputLabel.
What this PR does / why we need it: Fixes the correlation between labels and inputs on the dataverse user page when in create mode.
Which issue(s) this PR closes:
Closes #7771 Accessibility form labels not associated with inputs on Sign Up Page
Special notes for your reviewer:
Suggestions on how to test this:
I installed the checking tool mentioned in the issue https://wave.webaim.org/
Does this PR introduce a user interface change? If mockups are available, please link/include them here:
No
Is there a release notes update needed for this change?:
no
Additional documentation:
none