hyrax icon indicating copy to clipboard operation
hyrax copied to clipboard

Links on New User Collection page to not meet WCAG AA minimum color contrast requirements

Open rjkati opened this issue 1 year ago • 2 comments

Descriptive summary

On dev.nurax.samvera.org (Hyrax 5.0.1), the Add another Title and Cancel links do not meet WCAG AA minimum color contrast requirements

Steps to reproduce the behavior in User Interface (UI)

  1. In Chrome, download the Site Improve browser extension
  2. Go to https://dev.nurax.samvera.org/dashboard/collections/new?locale=en&collection_type_id=3 and run the Site Improve browser extension

color_contrast_links

rjkati avatar May 16 '24 18:05 rjkati

This issue also occurs on the following pages:

Hopefully, a CSS tweak can fix all of these, but please comment here if it is preferable to create tickets for each occurrence and I'll do that.

rjkati avatar May 17 '24 14:05 rjkati

Similar alerts are noted for Create New Work (single work) for the "Add Another..."/"Remove" links when a multi-valued field option is presented.

eporter23 avatar Aug 16 '24 14:08 eporter23

On nurax-pg, this shows as a AAA issue

aahurford avatar Oct 28 '24 17:10 aahurford

I am no longer seeing any SiteImprove issues on the Add New Collection edit form on dev.nurax or pg.nurax. I am also not seeing any SiteImprove issues on the Edit Work page, Add new work by batch, Edit File (at least not the color contrast for the Cancel link), Edit Collection, or Collection Admin page (at least not the color contrast for those links). Some other PR must have also addressed these link color contrast issues? I'm not sure which one but there is a bootstrap override in place to set $blue: #0056B3 and that seems to be the color that is applied now on all of these links and is passing WCAG AA.

jlhardes avatar Jan 21 '25 21:01 jlhardes

Confirmed that this is no longer an issue on the Add New Collection, Edit Work, Add New Works by Batch, Edit File, Edit Collection or Collections (I think this is the Collection Admin page) pages.

christhepianist avatar Jan 22 '25 15:01 christhepianist

Took me a minute to find the PR but here it is: https://github.com/samvera/hyrax/pull/6896 That was for https://github.com/samvera/hyrax/issues/6755 Closing this issue

jlhardes avatar Jan 22 '25 18:01 jlhardes