Skip to content

frontend: Label: Fix warning label color contrast#4284

Merged
illume merged 1 commit intokubernetes-sigs:mainfrom
vyncent-t:fix-contrast-cluster-labels
Jan 6, 2026
Merged

frontend: Label: Fix warning label color contrast#4284
illume merged 1 commit intokubernetes-sigs:mainfrom
vyncent-t:fix-contrast-cluster-labels

Conversation

@vyncent-t
Copy link
Contributor

Summary

image

Ran FastPass and it seems there is an issue when using white text and orange background. I have tried adjusting the background color to be darker to match allowed contrast, but this results in the "allowed" orange basically being just red. (Please refer to the image above for details)

This clashes with the red color used for failures so I am opting for a use of black text when displaying orange labels in light mode.

Changes

  • Adjusts label text color to black when on light mode

Steps to Test

  1. Navigate to a resource that is displaying warning label
  2. Note that the white text is now black
  3. Run FastPass if available and observe no issues

Screenshots

Before

image

After

image

@vyncent-t vyncent-t self-assigned this Dec 17, 2025
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Dec 17, 2025
@vyncent-t vyncent-t added frontend Issues related to the frontend a11y Accessibility related issues and removed cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Dec 17, 2025
@k8s-ci-robot k8s-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Dec 17, 2025
@k8s-triage-robot
Copy link

Unknown CLA label state. Rechecking for CLA labels.

Send feedback to sig-contributor-experience at kubernetes/community.

/check-cla
/easycla

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Dec 17, 2025
Copy link
Contributor

@skoeva skoeva left a comment

Choose a reason for hiding this comment

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

would be good to test this with a color contrast checker like this one

Copy link
Contributor

@skoeva skoeva left a comment

Choose a reason for hiding this comment

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

CI failure should also go away with a rebase

@vyncent-t vyncent-t force-pushed the fix-contrast-cluster-labels branch from 56fc4e4 to 456aa2d Compare January 6, 2026 18:59
Copy link
Contributor

@illume illume left a comment

Choose a reason for hiding this comment

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

🎉 thanks!

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: illume, vyncent-t

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 6, 2026
@illume illume merged commit 493808f into kubernetes-sigs:main Jan 6, 2026
11 of 12 checks passed
@illume illume added this to the v0.40.0 milestone Feb 2, 2026
@illume illume added the kind/bug Categorizes issue or PR as related to a bug. label Feb 2, 2026
@illume illume changed the title frontend: Fix warning label color contrast frontend: Label: Fix warning label color contrast Feb 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a11y Accessibility related issues approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. frontend Issues related to the frontend kind/bug Categorizes issue or PR as related to a bug. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants