Make status column icons colorblind friendly#3401
Conversation
Ensures these icons are differentiated not only by colour, but also by icon.
drewnoakes
requested review from
davidfowl,
leslierichardson95,
timheuer and
tlmii
ghost
added
the
|
do any of the 3 remaining uses of ""Icons.Filled.Size16.Circle"" in that file need to be visually distinguishable? |
|
I think the featureless circle makes sense for "unknown" state, which is two of the three. The other is "starting" state. Perhaps that could be an outlined circle. |
|
Did we ever consider using a spinner for "starting" status items? |
|
Looks appropriate to me, my 2c |
|
/backport to release/8.0 |
|
Started backporting to release/8.0: https://github.com/dotnet/aspire/actions/runs/8654453546 |
@drewnoakes do we still need to distinguish this, to be accessible? |
This makes the difference between 'Starting' and 'Unknown' states visually differentiable by glyph, rather than colour alone. Follows on from dotnet#3401.
Yes, #3689 I decided against using a spinner, mostly because it didn't seem possible to make the Blazor Fluent UI one that small. |
This makes the difference between 'Starting' and 'Unknown' states visually differentiable by glyph, rather than colour alone. Follows on from #3401.
This makes the difference between 'Starting' and 'Unknown' states visually differentiable by glyph, rather than colour alone. Follows on from #3401.
7 participants
Ensures these icons are differentiated not only by colour, but also by icon.
Also fixes this issue, pointed out offline by @timheuer:
Microsoft Reviewers: Open in CodeFlow