Skip to content

[FIX] Fullscreen do mapa dessincroniza com Esc; paginação da tabela nunca renderiza #174

Description

@iacapuca

Dois defeitos menores encontrados durante triagem de lint (nenhum dos dois foi apontado pelo linter).

1. Fullscreen preso ao sair com Esc

app/components/Commom/Maps/AmecicloMap.tsx:15-26 altera o estado isFullscreen apenas no clique do botão. Não existe listener de fullscreenchange no repositório.

Sair do fullscreen nativo com Esc deixa o React achando que ainda está em tela cheia, e o container permanece em fixed inset-0 z-50 w-screen h-screen (linha 591) — o mapa cobre a página inteira sem ter como voltar pelo botão.

Relacionado: a prop isFullscreen é passada mas ignorada; toggleFullscreendocument.fullscreenElement em vez dela. Por isso o title do botão é fixo em "Expandir mapa em tela cheia" e o ícone nunca inverte.

2. "Pular para página" construído e nunca renderizado

Em app/components/Commom/Table/Table.tsx, o TableFooter monta pageNumberInput, handlePageNumberChange, handleKeyPress, handleGoToPage, shouldUseInput (linha ~458) e pagesButtons (~459) — e o JSX retornado (490-532) tem apenas Anterior/Próxima.

É por isso que a prop data do footer aparece como não usada: a feature inteira saiu do render e ficou no corpo.

Adjacente (não corrigir junto)

Table.tsx:345,378 usa índice do array como key das linhas (React.Fragment key={i}) em vez de row.id. Com reordenação ou filtro, o React pode reaproveitar estado da linha errada.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingfrontend

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions