-
-
Notifications
You must be signed in to change notification settings - Fork 86
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pega tema do sistema do usuário, remove piscada quando tema escuro #219
Conversation
Signed-off-by: George Araújo <george.gcac@gmail.com>
Signed-off-by: George Araújo <george.gcac@gmail.com>
✅ Deploy Preview for diciotech ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Signed-off-by: George Araújo <george.gcac@gmail.com>
Signed-off-by: George Araújo <george.gcac@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Obrigada pela contribuição 🚀
@george-gca se tu conseguir dar um check nos conflitos 🫰🏻 |
Então, o #218 mudou os arquivos css, mas na verdade é pra gente mexer nos sass e regerar os css. Vou ver se consigo pegar a mudança dele e botar no sass. |
Signed-off-by: George Araújo <george.gcac@gmail.com>
@levxyca foi o/ |
@george-gca boaaaaaaaa |
Descrição de PR
Removi arquivos inutilizados, resolvi piscada quando o tema escuro é o que está selecionado (ao atualizar a página) e adicionei suporte a pegar tema do sistema.
Issue relacionado
Motivações
Já que eu estava com a mão na massa (implementando uma versão da tradução no #217), vi que ia ser tranquilo resolver a #173 e acabei junto resolvendo a outra.
Informações adicionais
Eu apaguei vários arquivos css, já que o sass compila tudo no style.css. Também criei um arquivo pro tema claro, pra ficar mais separado onde as cores dos temas são tratadas.