Skip to content

Commit

Permalink
Warning StatusIcon should use black fill for a11y
Browse files Browse the repository at this point in the history
To ensure proper contrast for a11y purposes, the warning status icon
should use a black fill for the '!' in the middle. The default is
transparent which means that in the light theme the contrast is not
sufficient to meet minimum accessibility requirements.
  • Loading branch information
AlanGreene authored and tekton-robot committed Oct 2, 2023
1 parent 2a13946 commit 5f61d4d
Showing 1 changed file with 5 additions and 1 deletion.
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
Copyright 2020-2022 The Tekton Authors
Copyright 2020-2023 The Tekton Authors
Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at
Expand Down Expand Up @@ -43,6 +43,10 @@ limitations under the License.

&.tkn--status-icon--type-inverse {
fill: $support-03;

path:first-child {
fill: black;
}
}
}
}

0 comments on commit 5f61d4d

Please sign in to comment.