Replies: 7 comments
-
Yes, agreed. Let’s add a more contrasting color indeed. |
Beta Was this translation helpful? Give feedback.
-
It seems |
Beta Was this translation helpful? Give feedback.
-
I don't know if you consider the current state to be final, but personally I think the contrast difference in dark in the latest version is a bit too much now, whereas in light it is just perfect. |
Beta Was this translation helpful? Give feedback.
-
We could tone it down a little. The current contrast ratio is ~7.5, based on |
Beta Was this translation helpful? Give feedback.
-
I don't know to be honest. I can experiment with it this weekend and let you know what I prefer. That doesn't mean you have to agree with it obviously :) |
Beta Was this translation helpful? Give feedback.
-
I checked it, and changing the color to |
Beta Was this translation helpful? Give feedback.
-
Thanks, I agree. I fixed the text color of colored/accented tables too. The following code has been pushed to the main repo: @if $enable-dark-mode {
@include color-mode(dark) {
.table-striped, .table-striped-columns {
--bs-table-striped-bg: var(--bs-tertiary-bg);
--bs-table-striped-color: var(--bs-body-color);
}
.table-hover {
--bs-table-hover-bg: var(--bs-tertiary-bg);
--bs-table-striped-color: var(--bs-body-color);
}
}
} |
Beta Was this translation helpful? Give feedback.
-
Personally, I think the contrast difference when using a table with
table-striped
is quite low in dark mode. It is fine in light, but in dark, I need to look really close to actually notice there is a difference.Perhaps using the same background colour as a code block could already be an improvement.
Beta Was this translation helpful? Give feedback.
All reactions