Skip to content
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

Accessibility fixes: Manage IDs #545

Open
2 tasks done
Tracked by #546
mariagould opened this issue Jan 9, 2024 · 4 comments · May be fixed by #814
Open
2 tasks done
Tracked by #546

Accessibility fixes: Manage IDs #545

mariagould opened this issue Jan 9, 2024 · 4 comments · May be fixed by #814
Assignees

Comments

@mariagould
Copy link

mariagould commented Jan 9, 2024

The following accessibility issues were identified on the Manage IDs page:

  • Remove the “Read more/Read less" option to expand and collapse search tips. Show all tips by default.
Screenshot 2024-01-09 at 08 34 31
  • The identifiers table has redundant links. All metadata fields are links, but they all go to the same identifiers page. Only the content in the "Identifier" column should have a link.
Screenshot 2024-01-09 at 08 33 35
@sfisher sfisher self-assigned this Dec 16, 2024
@sfisher sfisher linked a pull request Dec 17, 2024 that will close this issue
@sfisher sfisher linked a pull request Dec 17, 2024 that will close this issue
@adambuttrick
Copy link

@sfisher Reviewed these on dev, but neither fix appeared to be present.

@sfisher
Copy link
Contributor

sfisher commented Jan 21, 2025

I'll must not have merged everything. I'll look into it. :-)

@sfisher
Copy link
Contributor

sfisher commented Jan 21, 2025

@adambuttrick Can you double check, maybe refresh browser with SHIFT-refresh or CTRL-refresh? I tried to see if I had missed a merge or something and the code changes appear present and when I go to dev I see that these things are changed with the only link to the dataset being the ARK/DOI and the explanatory text at top always present without the hide/show: https://ezid-dev.cdlib.org/manage?ps=10&p=1&order_by=c_update_time&sort=asc&owner_selected=all&c_title=t&c_creator=t&c_identifier=t&c_owner=t&c_create_time=t&c_update_time=t&c_id_status=t&owner_selected=group_cdl

Image

Though maybe I missed something or there is some other similar page that I missed.

Thank you for going through all these things and being so thorough. It's really helpful.

@adambuttrick
Copy link

@sfisher I cleared everything in the browser and this now displays fine! Apologies for the cached confusion.

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

Successfully merging a pull request may close this issue.

3 participants