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 de compose_tweet_page #274

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

RoxyRoses
Copy link
Contributor

Proposta de Pull Request: Refatoração da Página de Composição de Tweets

Descrição

Este Pull Request implementa melhorias na página de composição de tweets, atualizando componentes para utilizar o novo Design System e ajustando a lógica de código para maior consistência e clareza.

Alterações Principais

  1. Substituição do RaisedButton por PenhasButton.roundedFilled:

    • Adotado o botão atualizado do Design System, eliminando atributos redundantes como elevation, shape e color.
    • Melhora a padronização e a estética do botão.
  2. Remoção de operador nulo condicional obsoleto:

    • Substituição de WidgetsBinding.instance?.focusManager.primaryFocus?.unfocus() por WidgetsBinding.instance.focusManager.primaryFocus?.unfocus().
  3. Atualização dos arquivos de snapshot de UI:

    • Alteração dos snapshots para refletir as mudanças no componente de botão.

Alterações nos Arquivos

Código

  • Arquivo: lib/app/features/feed/presentation/compose_tweet/compose_tweet_page.dart
    • Atualização do botão de "Publicar".
    • Ajustes no gerenciamento de foco para garantir compatibilidade com as versões mais recentes do Flutter.

Testes de Interface

  • Arquivos Atualizados:
    • test/app/features/feed/presentation/compose_tweet/goldens/ci/compose_tweet_page.png
    • test/app/features/feed/presentation/compose_tweet/goldens/ci/compose_tweet_page_with_app_bar.png
    • test/app/features/feed/presentation/compose_tweet/goldens/ci/compose_tweet_page_with_error.png
    • test/app/features/feed/presentation/compose_tweet/goldens/ci/compose_tweet_page_with_text.png

Trechos de Código

Antes

return RaisedButton(
  onPressed: controller.isEnableCreateButton
      ? () => controller.createTweetPressed(context)
      : null,
  elevation: 0.0,
  shape: kButtonShapeFilled,
  color: DesignSystemColors.ligthPurple,
  child: const Text(
    'Publicar',
    style: kTextStyleDefaultFilledButtonLabel,
  ),
);

Depois

return PenhasButton.roundedFilled(
  onPressed: controller.isEnableCreateButton
      ? () => controller.createTweetPressed(context)
      : null,
  child: const Text(
    'Publicar',
    style: kTextStyleDefaultFilledButtonLabel,
  ),
);

Como Testar

  1. Verificar o botão "Publicar":
  • Certificar-se de que o botão está funcional e com o estilo atualizado.
  • Confirmar que o botão só habilita quando os requisitos para publicação forem atendidos.
  1. Testes Visuais:
  • Validar que os snapshots atualizados refletem as mudanças esperadas.

Justificativa

Estas alterações são necessárias para garantir consistência com o Design System atual e modernizar o código, removendo elementos obsoletos. Além disso, o uso de componentes reutilizáveis melhora a manutenção do projeto e a legibilidade do código.

@RoxyRoses RoxyRoses added the refactor Envolve melhorias no código, como substituir o componentes obsoletos label Nov 21, 2024
@RoxyRoses RoxyRoses self-assigned this Nov 21, 2024
Copy link

sonarcloud bot commented Nov 21, 2024

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
1 participant