#4 Fix text highlight color in dark mode

Затворени
отворен преди 6 години от matt · 4 коментара
matt коментира преди 6 години

Right now when you select text in dark mode you can’t see what you’ve highlighted.

In my testing, this occurred on *buntu, but not on Elementary OS.

Right now when you select text in dark mode you can't see what you've highlighted. In my testing, this occurred on *buntu, but not on Elementary OS.
adrian коментира преди 6 години
Сътрудник

I attempted to use the appropriate selector (selection) for this in the CSS I’m applying to the textview, and it made no difference.

It’s worth noting in my theme this isn’t an issue as the text selection colour is blue, but if we’re taking over the styling there we should be ensuring the text selection colour contrasts enough to be visible.

I can look into this some more but that would probably take some time.

I attempted to use the appropriate selector (`selection`) for this in the CSS I'm applying to the textview, and it made no difference. It's worth noting in my theme this isn't an issue as the text selection colour is blue, but if we're taking over the styling there we should be ensuring the text selection colour contrasts enough to be visible. I can look into this some more but that would probably take some time.
matt added the
high priority
label преди 5 години
adrian коментира преди 5 години
Сътрудник

I tried several things, and I am not succeeding in getting the selection to show up.

Do you mind if I look into a way to remove the darkmode toggle button from the UI if the current theme doesn’t have one? Because it works when the theme defines a darkmode.

I tried several things, and I am not succeeding in getting the selection to show up. Do you mind if I look into a way to remove the darkmode toggle button from the UI if the current theme doesn't have one? Because it works when the theme defines a darkmode.
matt коментира преди 5 години
Притежател

Nope, that works for me.

Nope, that works for me.
adrian коментира преди 5 години
Сътрудник

Fixed, though what I’ve managed to implement adds a couple seconds onto the startup time.

For performance it’d be better to reimplement a bit of GTK, though that risks getting the test wrong. The good thing about the approach I used is that it can withstand changes to GTK+.

Fixed, though what I've managed to implement adds a couple seconds onto the startup time. For performance it'd be better to reimplement a bit of GTK, though that risks getting the test wrong. The good thing about the approach I used is that it can withstand changes to GTK+.
Впишете се за да се присъедините към разговора.
Няма етап
Няма изпълнители
2 участника
Известия
Due Date

No due date set.

Зависимости

This issue currently doesn't have any dependencies.

Зареждане…
Все още няма съдържание.