FdelMazo / FIUBA-Plan

Organizador de horarios de la Facultad de Ingenieria
https://fede.dm/FIUBA-Plan/
MIT License
46 stars 6 forks source link

Dark Mode #23

Closed nicomatex closed 3 years ago

nicomatex commented 3 years ago

Pinta agregar ese Dark Mode al calendario?

komod0 commented 3 years ago

YES, el otro día estaba chusmeando algo de eso, por lo que vi hay varias formas de hacerlo, esta guía esta bastante bonita: https://css-tricks.com/a-complete-guide-to-dark-mode-on-the-web/

nicomatex commented 3 years ago

Justo estaba indagando un poco! Tome el approach de usar dos stylesheets separadas, y dos theme.js diferentes, tipo image

Para que quede algo asi

image

komod0 commented 3 years ago

Esta bueno, yo me habia puesto a jugar modificando el css desde la consola y llegue a algo parecido, pero se me hacia medio raro el contraste que hay entre las celdas grises y las mas oscuras asi que lo termine sacando directamente y me quedo algo asi: darkmodo

komod0 commented 3 years ago

Me di cuenta que el calendario de teams de microsoft usa un estilo super parecido para los eventos y les queda algo asi el modo oscuro: teams

nicomatex commented 3 years ago

Si, la verdad que queda mejor asi. Voy a ver si puedo lograr una primera version y la pusheo a la noche!

FdelMazo commented 3 years ago

Creo que la manera correcta de implementar el dark mode sería utilizando el hook de chakra: https://chakra-ui.com/color-mode

El único cuidado a tener aca es que el estilo es un choclo... Tenemos cosas tiradas en theme.js, cosas hardcodeadas inline como la prop style de los componentes, y cosas en Style.css que es css puro (más que nada para estilar el react-big-calendar.

Si conseguimos una manera de unificar eso y no repetirse, va como piña el hook.

(ot: hermoso el chusmear el de microsoft. Me parece que el estilo de las rayitas en el border left de los eventos pueden ser copadas para diferenciar eventos (maybe con rayitas == teorica, sin rayitas == practica, o algo con las distintas sedes, o solo para mostrar que hay overlap). Idem me parece copado el botoncito de sync abajo a la der, solo que no tenemos cosas copadas para hacer onclick sobre la materia. (un boton de remover materia ahí seria un choclo imo))

FdelMazo commented 3 years ago

(Ojo, un botoncito abajo a la der de cada evento que sea algo como "No la voy a cursar" y te remueva ese evento (solo la clase, no la materia), va como piña. Para esas materias que "a las teoricas no hace falta ir")

Anyway, si hay quorum, hay que llevarlo a un issue nuevo

nicomatex commented 3 years ago

Un amigo me acaba de sugerir exactamente lo mismo hace 15 minutos! Lo agrego como issue porque parece que seria util (tambien hay algunas materias que aparecen en 2 dias, pero que uno elige que dia cursarla). Agrego el issue

FdelMazo commented 3 years ago

offtopic movido a #24

FdelMazo commented 3 years ago

En 89f5376d76c6f5b79757b4faaade4d73005d0aad implemente esto @nicomatex @komod0

La verdad, no me parece que haya quedado muy lindo el general view, siento que pequé de minimalista, pero la agenda me gusto. Acepto PRs para dejarlo mas lindo!!

image

image