#4 Fix text highlight color in dark mode

Chiuso
aperto 6 anni fa da matt · 4 commenti
matt 6 anni fa ha commentato

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 anni fa ha commentato
Collaboratori

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 anni fa
adrian 5 anni fa ha commentato
Collaboratori

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 anni fa ha commentato
Proprietario

Nope, that works for me.

Nope, that works for me.
adrian 5 anni fa ha commentato
Collaboratori

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+.
adrian closed this issue 5 anni fa
Effettua l'accesso per partecipare alla conversazione.
Nessuna milestone
Nessuna assegnatario
2 Partecipanti
Notifiche
Data di scadenza

Nessuna data di scadenza impostata.

Dipendenze

Questo problema attualmente non ha alcuna dipendenza.

Caricamento…
Non ci sono ancora contenuti.