#4 Fix text highlight color in dark mode

Fechado
aberto por matt 6 anos atrás · 4 comentários
matt comentou 6 anos atrás

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 comentou 6 anos atrás
Colaborador

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 anos atrás
adrian comentou 5 anos atrás
Colaborador

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 comentou 5 anos atrás
Proprietário

Nope, that works for me.

Nope, that works for me.
adrian comentou 5 anos atrás
Colaborador

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 anos atrás
Acesse para participar desta conversação.
Sem marco
Sem responsável
2 participante(s)
Notificações
Data limite

Data limite não informada.

Dependências

Esta issue atualmente não tem dependências.

Carregando…
Ainda não há conteúdo.