Opened 3 days ago
Last modified 28 hours ago
#36229 assigned Bug
Forced colors mode icon color in default_urlconf “congratulations” view
Description ¶
On the Congrats! page, the icons aren’t very legible due to their text color. This is particularly a problem in forced colors mode, where they should be clearly identifiable as link text.
This can be fixed by changing the .options svg fill and border-color to currentColor, so the icon visuals and border always match the text color of the surrounding link element. Although the problem is particularly obvious in forced colors mode, the "regular" mode would also benefit from this change, since it improves the color contrast of the icons.
According to the ticket's flags, the next step(s) to move this issue forward are:
- To provide a patch by sending a pull request. Claim the ticket when you start working so that someone else doesn't duplicate effort. Before sending a pull request, review your work against the patch review checklist. Check the "Has patch" flag on the ticket after sending a pull request and include a link to the pull request in the ticket comment when making that update. The usual format is:
[https://github.com/django/django/pull/#### PR]
.
Change History (6)
by , 2 days ago
Attachment: | image-20250305-094854.png added |
---|
by , 2 days ago
Attachment: | image-20250305-094914.png added |
---|
by , 2 days ago
Attachment: | image-20250305-095730.png added |
---|
comment:1 by , 2 days ago
Component: | Uncategorized → Core (Other) |
---|---|
Triage Stage: | Unreviewed → Accepted |
comment:2 by , 29 hours ago
The contrast is only part of the issue here. My understanding is that when using forced colors mode, all link content should be the link colour, which includes the icons in this case.
comment:3 by , 28 hours ago
I also think these links in light mode should look more like a link (with an underline) or look like a button
Here is the color contrast checker for grey and I think I agree that we either need to make the lines thicker or the color darker for it to pass standards

I wrote a visual regression test to generate a couple of screenshots:
TabularUnified tests/view_tests/tests/test_debug.py
Sample generated screenshots:

