Skip to content

Style: Substituicao do window.confirm por um AlertDialog (shadcn/ui)#592

Merged
jefter-dev merged 4 commits intodevfrom
526-padronizar-confirmação-de-exclusão-vacinas-e-transtornos-com-modal-shadcnui
Mar 30, 2026

Hidden character warning

The head ref may contain hidden characters: "526-padronizar-confirma\u00e7\u00e3o-de-exclus\u00e3o-vacinas-e-transtornos-com-modal-shadcnui"
Merged

Style: Substituicao do window.confirm por um AlertDialog (shadcn/ui)#592
jefter-dev merged 4 commits intodevfrom
526-padronizar-confirmação-de-exclusão-vacinas-e-transtornos-com-modal-shadcnui

Conversation

@euAllanDev
Copy link
Copy Markdown

Descrição da Issue:

  • Substituição dos diálogos de confirmação nativos (window.confirm) pelo componente AlertDialog do Shadcn/UI.

Essa mudança padroniza a interface de modais de confirmação no sistema, proporcionando:

  • Experiência do usuário aprimorada, com feedback visual consistente;
  • Interface moderna e alinhada ao design do sistema;
526-padronizar-confirmacao-de-exclusao-vacinas-e-t.mp4

@euAllanDev euAllanDev self-assigned this Mar 26, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 26, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 8866422e-9c0d-475d-b63a-cd77d3e412b4

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 526-padronizar-confirmação-de-exclusão-vacinas-e-transtornos-com-modal-shadcnui

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@jefter-dev
Copy link
Copy Markdown
Collaborator

Sugestão de melhoria

Seria interessante abstrair o AlertDialog em um componente reutilizável, por exemplo ConfirmModal, para padronizar o comportamento de confirmação em toda a aplicação e evitar duplicação de código.

Proposta

  • Criar um componente ConfirmModal que receba props como:

    • title
    • description
    • onConfirm
    • trigger (botão/elemento que abre o modal)

Mensagem dinâmica

Atualmente a mensagem está genérica ("excluir este item"). Podemos melhorar a experiência tornando-a dinâmica, permitindo passar o nome da entidade:

Exemplo:

  • "Essa ação não pode ser desfeita. Isso irá excluir permanentemente o transtorno Transtorno 1."
  • "Essa ação não pode ser desfeita. Isso irá excluir permanentemente a vacina Vacina 1."

Ajuste de botão

  • Como o componente será reutilizável, o ideal é:

    • Alterar o texto do botão de "Excluir" para "Confirmar"
    • Utilizar a cor padrão do sistema (primary) ao invés de uma cor customizada (ex: azul fixo)

Isso deixa o componente mais genérico e reutilizável para outros cenários além de exclusão.


Benefícios:

  • Reuso de código
  • Padronização visual e de comportamento
  • Melhor experiência do usuário com mensagens mais claras e específicas

@jefter-dev jefter-dev self-requested a review March 27, 2026 12:27
@jefter-dev jefter-dev added the pacientes Refatorar e evoluir o módulo de paciente para garantir maior aderência às necessidades da APAE. label Mar 27, 2026
@euAllanDev euAllanDev force-pushed the 526-padronizar-confirmação-de-exclusão-vacinas-e-transtornos-com-modal-shadcnui branch from 8783cf2 to 538638c Compare March 28, 2026 22:01
@euAllanDev
Copy link
Copy Markdown
Author

Abstração dos diálogos de confirmação duplicados em um componente reutilizável ConfirmModal, eliminando repetição de código e padronizando o comportamento de confirmação em toda a aplicação.

Essa mudança traz:

  • Reutilização e manutenibilidade, com um único componente centralizado para todos os fluxos de confirmação;
  • Mensagens dinâmicas, permitindo passar o nome da entidade para tornar o feedback mais claro ao usuário (ex: "Isso irá excluir permanentemente a vacina Vacina 1.");
  • Interface padronizada, utilizando a cor primária do sistema ao invés de cores fixas, alinhando o componente ao design system;
Screenshot_2026-03-28_19-08-23 Screenshot_2026-03-28_19-08-58

Copy link
Copy Markdown
Collaborator

@jefter-dev jefter-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jefter-dev jefter-dev merged commit a78b686 into dev Mar 30, 2026
7 checks passed
@jefter-dev jefter-dev deleted the 526-padronizar-confirmação-de-exclusão-vacinas-e-transtornos-com-modal-shadcnui branch March 30, 2026 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pacientes Refatorar e evoluir o módulo de paciente para garantir maior aderência às necessidades da APAE.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants