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

Substituir RaisedButton por PenhasButton na página SignUpTwo #271

Conversation

RoxyRoses
Copy link
Contributor

Substituição do RaisedButton por PenhasButton na página SignUpTwo

Descrição

Este pull request implementa as seguintes alterações:

  • Substituição do componente: O RaisedButton foi substituído pelo PenhasButton.roundedFilled no botão "Próximo" da página SignUpTwo.
  • Limpeza de importações: Removidas importações obsoletas, como button_shape.dart e colors.dart.
  • Melhoria no código: Atualização do uso de WidgetsBinding.instance.focusManager para remover o operador null-aware.
  • Atualização de asset visual: O asset sign_up_step_2_page.png foi ajustado para refletir a nova interface do botão.

Motivação

Essas mudanças seguem o padrão atualizado de design do sistema e garantem consistência na experiência visual e de interação do aplicativo. Além disso, a limpeza de importações reduz o peso do código e melhora a manutenção.

Alterações

Arquivos alterados

  1. sign_up_two_page.dart

    • Substituído RaisedButton por PenhasButton.roundedFilled.
    • Removidas importações de button_shape.dart e colors.dart.
    • Ajustada lógica de foco para evitar redundância no uso do operador null-aware.
  2. sign_up_step_2_page.png

    • Atualização do asset visual correspondente.

Imagem atualizada

Antes Depois

|
image
|
image
|

Impactos

  • Testes visuais: Podem ser impactados devido à atualização do botão na interface.
  • Consistência visual: Garantida ao adotar o novo componente de botão.

Testes realizados

  • Verificação da renderização correta do botão.
  • Teste funcional do clique no botão "Próximo".
  • Atualização dos testes de snapshot para refletir a nova interface.

Checklist

  • O código segue os padrões de estilo e boas práticas.
  • Os testes associados foram atualizados e executados.
  • Documentação relevante atualizada.

Issue relacionada

https://github.com/institutoazmina/penhas-app/issues/260

@RoxyRoses RoxyRoses added the refactor Envolve melhorias no código, como substituir o componentes obsoletos label Nov 19, 2024
@RoxyRoses RoxyRoses self-assigned this Nov 19, 2024
@RoxyRoses RoxyRoses requested a review from a team as a code owner November 19, 2024 16:03
Copy link

sonarcloud bot commented Nov 19, 2024

@RoxyRoses RoxyRoses changed the title substituir RaisedButton por PenhasButton na página SignUpTwo Substituir RaisedButton por PenhasButton na página SignUpTwo Nov 19, 2024
Copy link

codeclimate bot commented Nov 19, 2024

Code Climate has analyzed commit 03464ea and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 1

View more on Code Climate.

@shonorio shonorio merged commit 3e8ede7 into main Nov 26, 2024
7 checks passed
@shonorio shonorio deleted the 260-fix-corrigir-uso-de-operador-null-aware-desnecessário-e-raisedbutton-descontinuado-no-arquivo-sign_up_two_pagedart branch November 26, 2024 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Envolve melhorias no código, como substituir o componentes obsoletos
Projects
None yet
2 participants