ARTHURPC03 / New-GoBarber-Web

Aplicação Web Feito com Typescript + React, durante a nova jornada do GoStack!
43 stars 10 forks source link

Sugestão de alteração no componente #1

Open gusflopes opened 4 years ago

gusflopes commented 4 years ago

Gostei bastante da sua implementação! Ficou show!

Mas tenho duas sugestões:

1- Ao invés de criar um Header, e se fosse criado um componente apenas com o Switch? Assim você poderia colocar ele onde bem entendesse, além de não ficar com o Header na tela de login;

2- Se for por essa abordagem, acho que seria necessário alguma alteração mínima para exportar separadamente o tema da função que altera o tema, parecido com o que fizemos no AuthContext.

O que acha?

ARTHURPC03 commented 4 years ago

Obrigado! 🚀

1 - Mas aí no caso desse exemplo aí, iria colocar o Switch onde? tipo em um canto fixo na tela? Eu não me lembro de ter visto algo desse tipo, da aplicações que eu utilizo, ou estão no Header ou estão nas configurações.

2 - Imagino que não ia precisar alterar tanto assim não, pois o Header está como um componente global, acredito que seria apenas deixar o Switch nesse componente, mudar o estilo e utilizar nas páginas.

A ideia me parece interessante, mas eu não imaginei muito bem um outro lugar para colocar o Switch.

gusflopes commented 4 years ago

Eu acho que ele cabe no Header mesmo... mas somente depois do login... Daí o problema que surge quando quer colocar no Header, é que como ele está sendo importado e com o state diretamente no App, você não tem acesso ao componente Header que vai estar aninhado em Route > Page > Header...

Então pelo que eu entendi: o toggle e o Switch tem que ir para o Header.. mas o theme tem que ir para o App... por isso falei da divisão...

Eu tentei mas meio que me enrolei tentando fazer... Daí pra não desistir eu acabei colocando ele com position absolue pra não ter que colocar o header na tela de login: image

ARTHURPC03 commented 4 years ago

Entendi, maneiro!