diego3g / rsxp-2023

Repositório contendo o código do app do RS/XP 2023
MIT License
440 stars 84 forks source link

feat(mobile): layout cabeçalho e abas #77

Closed RennanD closed 1 year ago

RennanD commented 1 year ago

📋 Descrição

Essa PR finaliza o que o @rodrigorgtic criou, com o Drawer e cria o header tanto das telas do Tab Navigation, quanto das telas de Drawer. Ela também remove as tela de navegação em pilha.

Fixes #67

🛠️ Tipo da mudança

Exclua as opções que não são relevantes.

📷 Screenshots

Captura de Tela 2023-04-06 às 10 39 07

✅ Checklist:

diego3g commented 1 year ago

Fala @RennanD, pela print parece que as abas não estão seguindo o espaçamento interno vertical de 16px, confere? Vale dar um double check no layout do Figma pra ver se tudo está 100% <3

CleanShot 2023-04-06 at 10 47 52

Fora isso, a cor também não está batendo totalmente, acredito que seja pela falta do merge da PR #71 certo?!

RennanD commented 1 year ago

Fala @RennanD, pela print parece que as abas não estão seguindo o espaçamento interno vertical de 16px, confere? Vale dar um double check no layout do Figma pra ver se tudo está 100% <3

CleanShot 2023-04-06 at 10 47 52

Fora isso, a cor também não está batendo totalmente, acredito que seja pela falta do merge da PR #71 certo?!

Em relação as cores é isso mesmo, pelo menos cor de ícone ativo e inativo está assim por conta dos tokens de cor não estarem configurados.

Posso abrir uma issue para alterar essas cores após a #71 ser aprovada, o que achas?

diego3g commented 1 year ago

@RennanD #71 entrou :)

RennanD commented 1 year ago

@RennanD #71 entrou :)

Shoow!! Já vou usar as cores certas. 💜

RennanD commented 1 year ago

@RennanD #71 entrou :)

amigo , percebi que utilizando as cores no dentro do extends no tailwind.config.js só funciona para as classes, então quando utilizo dentro das opções do TabBar, para as labels por exemplo, acaba ficando com o token padrão do Tailwind.

Vou mover essas cores para a raiz do tema, já que só elas vão ser utilizadas no app, beleza?

diego3g commented 1 year ago

@RennanD Acho que precisa usar o resolveConfig: https://tailwindcss.com/docs/configuration#referencing-in-java-script

RennanD commented 1 year ago

@diego3g confere se tá tudo certinho agora. 💜

diego3g commented 1 year ago

@RennanD dois últimos ajustes e se puder colocar uma print de como ficou seria massa! :)

RennanD commented 1 year ago

@diego3g tranquilo amigo, o print acha melhor colocar como um comentário aqui mesmo?

diego3g commented 1 year ago

@RennanD sim

melquisedeque-magalhaes commented 1 year ago

@RennanD #71 entrou :)

amigo , percebi que utilizando as cores no dentro do extends no tailwind.config.js só funciona para as classes, então quando utilizo dentro das opções do TabBar, para as labels por exemplo, acaba ficando com o token padrão do Tailwind.

Vou mover essas cores para a raiz do tema, já que só elas vão ser utilizadas no app, beleza?

mano na Pr #81 eu fiz uma forma de utilizar as cores fora das classe se quiser dar uma olhada mano, acho que e uma abordagem interessante.

diego3g commented 1 year ago

@melquisedeque-magalhaes acho que como o @RennanD fez é melhor, com o resolveConfig, inclusive é o recomendado na doc do tailwind.

melquisedeque-magalhaes commented 1 year ago

@melquisedeque-magalhaes acho que como o @RennanD fez é melhor, com o resolveConfig, inclusive é o recomendado na doc do tailwind.

shooow @diego3g e @RennanD gostei da implementação do resolveConfig também muito massa, quando der o merge nessa Pr vou atualizar a minha.

RennanD commented 1 year ago

@diego3g acho que agora foi huheuhe!! Além do que tinha pedido para ajustar, refatorei para utilizar os path e ficou muito melhor.

Captura de Tela 2023-04-07 às 09 41 12

henriqgoncalvs commented 1 year ago

Pessoal, o CI foi atualizado então vcs vão ver que já tem alguns jobs falhando.

Pra o o job do Semantic PR Titles passar vcs tem que escrever o título no formato do conventional commits (só add um feat(mobile):) na frente já resolve.

Sobre o de types checking, podem ignorar por enquanto pois é um erro de node_modules que está em resolução

vlww 👊

diego3g commented 1 year ago

@RennanD No layout eu coloquei o padding das tabs abaixo maior porque é iOS e tem o notch, mas no Android deveria ficar igual. Acho que ali também tem que usar o safeAreaInsets

RennanD commented 1 year ago

Fala @diego3g , fiz uma condição para verificar o Inset Bottom e aplicar uma altura e um Padding bottom diferentes par o ios no caso do notch.

Captura de Tela 2023-04-10 às 09 25 23