Skip to content

Ada 232#56

Merged
yzhoubk merged 5 commits intomainfrom
ADA-232
Jun 5, 2025
Merged

Ada 232#56
yzhoubk merged 5 commits intomainfrom
ADA-232

Conversation

@yzhoubk
Copy link
Copy Markdown
Contributor

@yzhoubk yzhoubk commented Jun 3, 2025

In addition to the link elements, the Axe tool also flagged other elements for insufficient color contrast. These issues have also been resolved.

Search
Element has insufficient color contrast of 3.97 (foreground color: #ffffff, background color: #007bff, font size: 12.0pt (16px), font weight: normal). Expected contrast ratio of 4.5:1

Element has insufficient color contrast of 3.13 (foreground color: #28a745, background color: #ffffff, font size: 12.0pt (16px), font weight: normal). Expected contrast ratio of 4.5:1

  1. Elements from the metadata modal:
    ISO 19139
    Element has insufficient color contrast of 3.97 (foreground color: #ffffff, background color: #007bff, font size: 12.0pt (16px), font weight: normal). Expected contrast ratio of 4.5:1

4.Elements from the metadata modal:
Spatial Reference Information
Element has insufficient color contrast of 4.14 (foreground color: #0073eb, background color: #f5f5f5, font size: 12.0pt (16px), font weight: normal). Expected contrast ratio of 4.5:1

@yzhoubk yzhoubk requested a review from awilfox June 3, 2025 16:10
Copy link
Copy Markdown
Member

@awilfox awilfox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall, very good. I was unable to find one element to test it. We need to broaden one of the CSS rules to apply to all parts of the selected facet. After that is fixed, this should be ready to merge.

Comment thread app/assets/stylesheets/_customizations.scss Outdated
Comment thread app/assets/stylesheets/_customizations.scss
Copy link
Copy Markdown
Member

@awilfox awilfox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! r+

@yzhoubk yzhoubk merged commit 219f83e into main Jun 5, 2025
3 of 4 checks passed
@yzhoubk yzhoubk deleted the ADA-232 branch June 5, 2025 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants