-
Notifications
You must be signed in to change notification settings - Fork 494
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
6109 dataset summary cleanup #6253
Conversation
Can one of the admins verify this patch? |
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.
Does it matter that there's still a <label... for Host Dataverse and Template when you're in create mode? (around line 506 and 522 in dataset.xhtml.)
@sekmiller Good question. Thank you for looking at the PR with that level of detail. The There are other accessibility issues to address on the dataset pg edit/create form, and we can address those in another issue. |
@mheppler It's such a big change that I wanted to make sure I understood what was going on. It looks good so I'm going to pass it along. |
Related Issues
label
tag outside of form (viewMode) #6109 Metadata Labels - Review use oflabel
tag outside of form (viewMode)Pull Request Checklist