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

T 282/dashboardmodelfix #54

Merged
merged 2 commits into from
Apr 12, 2022
Merged

T 282/dashboardmodelfix #54

merged 2 commits into from
Apr 12, 2022

Conversation

Aszurar
Copy link
Contributor

@Aszurar Aszurar commented Apr 12, 2022

  1. Corrigido o problema do modal do Header do Dashboard ficar piscando.

    • O problema não foi "nosso", é um problema relacionado à animações e modais direto com o React-Native, para resolve-lo poderia ser usado 2 propriedades de Modal:
    • backdropTransitionOutTiming={0} -> Altera o tempo de esconder o modal e acaba contornando o problema acima(mas não seria a solução ideal)
    • useNativeDriver -> Faz com que as animações do Modal ocorram totalmente dentro da Thread de Interface do usuário, seria o mais perto da solução, mas o modal ainda era para funcionar sem essa configuração.
    • Post sobre o problema na biblioteca de modais
  2. A estilização do Modal foi refatorada para evitar estilizações direto na tag de Modal em si pela recomendação dos próprios desenvolvedores da biblioteca de modal, isso evitaria outros problemas também relacionado a animações.

  • O Modal agora ta -4px para esquerda, antes estava 0, essa mudança foi feita pois em um celular "largo" do emulador(os celulares que testo fisicamente são 20:9 e aí tava de boa, mas agr quando vi no emulador em proporção mais comum o botão de abrir o modal estava aparcendo à esquerda, aí coloquei -4px para garantir que não aparece em nenhum celular.
  1. Os botões do Modal foram refatorados para botões da biblioteca react-native-gesture-handler, pois eles mantém o comportamento de cada sistema operacional, para isso, em qualquer modal de react-native há imcopatibilidade com essa bilioteca de botões.
    • Existem uma série de configurações para tentar contornar essa incompatibilidade, dentre elas, usei a propriedade coverScreen={false}, o natural dela é true. Ela é responsável por cobrir toda tela com a tag Modal na hierarquia dos componentes, como essa tag Modal não possui mais estilização própria, não houve impacto visual, e acredito que esteja tudo bem pois tem uma galera que usa tbm ela para contornar esse problema.
      Post sobre esse problema

Detalhes:
4. Foi alterado o ícone do botão de "Sair" do modal, o ícone anterior era o de Login transladado de forma reversa, só que, o ícone em si, ´continua indicando que vc ta fazendo um login, pois a seta dele indica entrar no app. Agora, o ícone é o próprio de Logout da biblioteca de ícones, que é justamente a seta saindo do app
- antigo:
image
- novo:
image

  1. A opacidade dos botões que ativam o modal do dashboard e do botão que é o nome do usuário junto a sua foto de perfil da dashboard em si foram alteradas para melhor indicação visual que foram pressionados, Antes o botão do usuário em si da dashboard apagava por inteiro ao ser pressionado, e o botão de ativação do Modal não tinha nenhuma reação.

  2. Adicionado a fonte aos textos dos botões do Modal

Screenrecorder-2022-04-12-10-12-38-191.mp4

@Aszurar Aszurar requested a review from Thomasr-02 April 12, 2022 13:50
@height
Copy link

height bot commented Apr 12, 2022

This pull request has been linked to 1 task:

💡Tip: Add "Close T-282" to the pull request title or description, to a commit message, or in a comment to mark this task as "Done" when the pull request is merged.

@Thomasr-02 Thomasr-02 merged commit e8a55ed into dev Apr 12, 2022
@ICodsTech ICodsTech deleted the T-282/dashboardmodelfix branch May 18, 2022 04:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants