Skip to content
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

Dark Mode #1199

Closed
filipedeschamps opened this issue Dec 29, 2022 · 9 comments
Closed

Dark Mode #1199

filipedeschamps opened this issue Dec 29, 2022 · 9 comments
Labels
front Envolve modificações no frontend

Comments

@filipedeschamps
Copy link
Owner

Contexto

Inicialmente disponibilizar um controle no Frontend apenas para a pessoa alternar entre os modos de visualização Claro e Escuro e persistir isto em Local Storage. Em outro momento (quando necessário), fazer esta informação persistir no backend (dentro do perfil do usuário), para ser sincronizado entre os devices.

Execução

Dado a alguns detalhes técnicos da stack que usamos (Next.js e BytMD), esta implementação não é tão simples, mas procurando no histórico, encontrei estas duas publicações:

Lembro que teve uma terceira tentativa, mas não estou conseguindo encontrar... mas posso estar enganado sobre a existência dela.

@ozzono
Copy link

ozzono commented Jan 10, 2023

Oi @filipedeschamps , desconheço o Next.JS então vou opinar com o que conheço do Vue.JS.

Que tal acionar o tema da página utilizando info do navegador ou do sistema oepracional do usuário? Dessa forma o tema escuro já seguiria a preferência do usuário, concorda?

Ao menos no VueJS a implementação é relativamente fácil e sou capaz de apostar que o NextJS tb dá uma forcinha.

@alvesa
Copy link

alvesa commented Jan 31, 2023

com ideia de remover duplicidades, a mesma issue foi aberta #1127

@saraiva1989
Copy link

@filipedeschamps ou admins, Eu iniciei a implementação do dark mode;

image

image

Ainda tem alguns ajustes a ser feito, colocar no checkbox um css bonitinho e ajustar algumas coisas mais.

basicamente eu criei um CSS e altero as variaveis dependendo do tema

image

image

Gostaria de saber se posso dar continuidade na implementação.

@ozzono
Copy link

ozzono commented Feb 1, 2023

Eu não aprovo nada, mas acho que ficou ótimo, mesmo com o checkbox padrão.

@saraiva1989
Copy link

Eu vou deixar melhor não é difícil.
Só não fiz tudo porque não sei se está nos moldes do projeto, aí mandei como estou fazendo para ver se faz sentido. Caso faça eu dou continuidade e devo fazer um PR esse fim de semana ainda .

@aprendendofelipe
Copy link
Collaborator

Gostaria de saber se posso dar continuidade na implementação.

@saraiva1989, não precisa pedir permissão para fazer contribuições. 👍

Especificamente sobre o dark mode, vale a pena levar em consideração tudo que foi discutido no #377 e #916 para não ficar travado nos mesmos pontos. 🤝

@saraiva1989
Copy link

Eu iniciei então, estou implementando e fazendo os testes.

Agora vou ter que trabalhar um pouco no dark dos "pre" e no markdown na escrita de artigos

TabNews_.Conteudos.para.quem.trabalha.com.Programacao.e.Tecnologia.-.Google.Chrome.2023-02-01.21-46-52.mp4

@birobirobiro
Copy link

Eu iniciei então, estou implementando e fazendo os testes.

Agora vou ter que trabalhar um pouco no dark dos "pre" e no markdown na escrita de artigos

TabNews_.Conteudos.para.quem.trabalha.com.Programacao.e.Tecnologia.-.Google.Chrome.2023-02-01.21-46-52.mp4

Parabéns pela iniciativa! Enquanto a sua PR não chega eu desenvolvi uma maneira alternativa de ter o dark mode direto no navegador:

https://www.tabnews.com.br/birobirobiro/tema-catppuccin-para-o-tabnews

@aprendendofelipe
Copy link
Collaborator

aprendendofelipe commented Apr 6, 2023

Foi implementado no PR #1348, e mais ajustes foram realizados nos PRs #1353 e #1394 🎉

Obrigado a todos que contribuíram de alguma forma para o dark mode! 💪

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
front Envolve modificações no frontend
Projects
None yet
Development

No branches or pull requests

6 participants