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

fix low contrast in homepage #7595

Conversation

Rakielle
Copy link
Contributor

What does this PR change?

This PR fixes the low contrast between the menu items and the background color, as identified by WAVE

GUI diff

Before:

1

After:

2

  • DONE

Documentation

  • No documentation needed: This changes improves the contrast color of the texts and the background to enhance accessibility, no additional documentation is needed.

  • DONE

Test coverage

  • No tests: already covered

  • DONE

Links

Fixes #7101

  • DONE

Changelogs

Make sure the changelogs entries you are adding are compliant with https://github.com/uyuni-project/uyuni/wiki/Contributing#changelogs and https://github.com/uyuni-project/uyuni/wiki/Contributing#uyuni-projectuyuni-repository

If you don't need a changelog check, please mark this checkbox:

  • No changelog needed

If you uncheck the checkbox after the PR is created, you will need to re-run changelog_test (see below)

Re-run a test

If you need to re-run a test, please mark the related checkbox, it will be unchecked automatically once it has re-run:

  • Re-run test "changelog_test"
  • Re-run test "backend_unittests_pgsql"
  • Re-run test "java_pgsql_tests"
  • Re-run test "schema_migration_test_pgsql"
  • Re-run test "susemanager_unittests"
  • Re-run test "javascript_lint"
  • Re-run test "spacecmd_unittests"

@Rakielle Rakielle requested a review from a team as a code owner September 27, 2023 14:19
@Rakielle Rakielle requested review from wweellddeerr and removed request for a team September 27, 2023 14:19
@cbbayburt cbbayburt requested review from cbbayburt and Etheryte and removed request for wweellddeerr September 28, 2023 12:54
@Etheryte
Copy link
Member

Etheryte commented Oct 2, 2023

While this fixes the contrast issue, I think it creates another new problem we don't want to have, which is that we now have a new color we only use in one place. Given that it's right next to the old color used in the home icon, I think we might have some better options here.

What about using the same menu highlighting logic as the Suma theme uses? Over time we're aiming to bring the two themes closer together anyway and this way we can solve both the contrast issue as well as keep the color. What do you think @Rakielle @cbbayburt?

For context, this is how the highlighting is handled in the Suma theme:

Screenshot 2023-10-02 at 16 21 31

@cbbayburt
Copy link
Contributor

What about using the same menu highlighting logic as the Suma theme uses? Over time we're aiming to bring the two themes closer together anyway and this way we can solve both the contrast issue as well as keep the color. What do you think @Rakielle @cbbayburt?

I think it's a good idea. @Etheryte could you point out where this SUMA style menu highlighting is implemented?

@Etheryte
Copy link
Member

Etheryte commented Oct 4, 2023

If I'm not mistaken that part should come from the old theme base in web/html/src/branding/css/base/theme.less, grep for @aside-menu-active. You will probably need other changes too aside from setting just that bit though, since I'd wager the underlying styles are different.

Copy link
Contributor

github-actions bot commented Dec 4, 2023

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the Stale label Dec 4, 2023
@cbbayburt cbbayburt removed the Stale label Dec 4, 2023
Copy link
Contributor

github-actions bot commented Feb 3, 2024

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the Stale label Feb 3, 2024
Copy link
Contributor

This PR was closed because it has been stalled for 10 days with no activity.

@github-actions github-actions bot closed this Feb 13, 2024
@cbbayburt cbbayburt reopened this Feb 13, 2024
@github-actions github-actions bot removed the Stale label Feb 14, 2024
@Etheryte
Copy link
Member

I've manually cherry-picked the changes from this PR into #8420 and #8169 so this should be safe to close. Since the underlying structure changed a fair bit the change set doesn't apply cleanly directly, but the contrast should now be fine.

@Etheryte Etheryte closed this Mar 27, 2024
@Rakielle Rakielle deleted the fix-low-contrast-homepage branch April 1, 2024 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants