Skip to content

hotfix: remove borda dupla no fim das listas de membros e representantes da turma - #545

Merged
BlMedeiros merged 4 commits into
mainfrom
fix/543-borda-dupla-listas-turma
Jul 24, 2026
Merged

hotfix: remove borda dupla no fim das listas de membros e representantes da turma#545
BlMedeiros merged 4 commits into
mainfrom
fix/543-borda-dupla-listas-turma

Conversation

@BlMedeiros

Copy link
Copy Markdown
Contributor

Contexto

Reportado ao gerenciar uma turma: as listas de Alunos/Professores e o painel de Representantes (/turmas/[classId]) mostravam uma linha extra ("borda dupla") coladinha na borda inferior do card, mais visível ao adicionar um representante com nome longo (o break-words quebra a linha e destaca o problema).

O que muda

O átomo ListItem (packages/ui) desenha border-b em todo item, inclusive o último — sem exceção pra quando ele está dentro de um card que já tem sua própria borda. Em vez de mexer no design system (mudança gated pelo squad de Front-End, AGENTS.md), o fix ficou local nos dois pontos que consomem ListItem dentro de um card com borda própria: ClassMembersTabs.tsx e PageTurmaDetalhe.tsx ganharam className="[&>li:last-child>*]:border-b-0" no <ul>, removendo a borda inferior só do último item.

Issue relacionada

Closes #543

Como testar

  1. Ir em /turmas/[classId] de uma turma com pelo menos 1 aluno e 1 representante.
  2. Reparar a borda inferior da lista de Alunos/Professores e do painel de Representantes — antes do fix havia uma linha fina extra logo acima da borda arredondada do card; depois do fix, só a borda do card.
  3. Opcional: em /turmas/[classId]/representantes, promover um aluno com nome longo a representante e conferir que a quebra de linha não deixa a borda duplicada mais evidente.

Tipo de mudança

  • Nova feature
  • Correção de bug
  • Refatoração (sem mudança de comportamento)
  • Documentação
  • Infraestrutura / config / build
  • Outro: ___

Checklist do autor

  • Código segue convenções definidas em CONTRIBUTING.md
  • Validei localmente que a aplicação compila/gera build sem erros (quando aplicável) — tsc --noEmit limpo em packages/core.
  • Verifiquei que não há erros de análise estática ou alertas relevantes no código (quando aplicável) — pnpm lint limpo.
  • Confirmei que não há erros de tipagem/TypeScript no escopo da mudança (quando aplicável)
  • Testei manualmente os cenários principais — reproduzido o bug e validada a correção ao vivo em localhost:3006, inclusive com nome longo forçando quebra de linha.
  • Componentes novos/alterados documentados no Storybook (se aplicável) — não é componente do design system (packages/core, não packages/ui).
  • Documentação atualizada (se aplicável)
  • Não introduzi dependências novas sem alinhamento prévio

Notas pro revisor

A causa raiz é do ListItem (packages/ui), que não tem uma variante "sem borda no último item". O fix aqui é local/escopado de propósito, pra não mexer no design system sem aprovação do squad de Front-End (AGENTS.md). Se esse padrão se repetir em outras listas pelo app, vale considerar promover isso pro ListItem futuramente.

@BlMedeiros BlMedeiros added the bug Algo não está funcionando label Jul 24, 2026
@BlMedeiros BlMedeiros changed the title fix: remove borda dupla no fim das listas de membros e representantes da turma hotfix: remove borda dupla no fim das listas de membros e representantes da turma Jul 24, 2026
@BlMedeiros
BlMedeiros changed the base branch from develop to main July 24, 2026 18:22
@BlMedeiros BlMedeiros self-assigned this Jul 24, 2026
@PabloTzeliks
PabloTzeliks self-requested a review July 24, 2026 19:10

@PabloTzeliks PabloTzeliks left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ok!

@BlMedeiros
BlMedeiros merged commit 447d046 into main Jul 24, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Algo não está funcionando

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: borda dupla no fim das listas de membros/representantes da turma

2 participants