Skip to content

feat(ui): seleção de mapas em grade navegável (BUG-67) - #377

Merged
rubenmarcus merged 1 commit into
v2/fauna-todos-mapasfrom
v2/grade-selecao-mapas
Aug 19, 2026
Merged

feat(ui): seleção de mapas em grade navegável (BUG-67)#377
rubenmarcus merged 1 commit into
v2/fauna-todos-mapasfrom
v2/grade-selecao-mapas

Conversation

@rubenmarcus

Copy link
Copy Markdown
Member

Palavras do dono (18/08): "vamos melhorar a tela de seleção de mapas, ela tá muito difícil de navegar e poluída porque já tem muitos mapas". Empilhado no #370 (mergar na ordem: #369#370 → este).

Defeito medido antes

A strip de carrossel desenhada para 5 cartazes de 196px espremia os 12 cards a:

viewport antes depois
1536×1024 (3:2 do dono) 98px, overflow-x transbordando, 2 cliques de paginação até o fim 275px, grade 3 colunas
960×640 50px (selo) 228px

A paginação de 5 quebrava o scan visual — ver um mapa específico exigia saber em que página ele estava.

O que mudou

  • Ficha do mapa continua à esquerda; o catálogo virou grade repeat(auto-fill,minmax(208px,1fr)) com scroll vertical na coluna direita — todos os mapas alcançáveis numa tela, zero paginação (fim dos ms-dashes)
  • ↑↓ pulam a LINHA da grade (não a página): colunas contadas por offsetTop real, porque auto-fill muda com o viewport; ←→ e as setas continuam na ordem da categoria
  • ?tela=mapas, tabs de categoria, filtro por autor e as opções de partida (armas/jogadores/rounds) intactas

Réguas — escritas ANTES, reprovavam o carrossel (conferido)

  • UIR25 no eval:redesign: esqueleto da grade + teclado 4 direções + fim dos dashes
  • eval:screenquery:browser passou a medir o uso: cardMin ≥ 190px, cols ≥ 3 via offsetTop, overflow-y: auto, e dois ArrowDown reais avançam exatamente 2×cols cards — 19/19 verdes
  • Correção de expectativa pré-existente no mesmo arquivo: a régua CIDADES esperava atacadao_treta na categoria, mas o MAP_CATS (a06b2b2) o pôs em ARENA+COMUNIDADE — a régua browser estava quebrada desde então por falta de execução

Verificação

  • check:fast: 53/54 (vermelho único changelog:check, pré-existente na main)
  • eval:redesign, eval:screenquery (node), docs:check, arch:check, feet:check: verdes
  • Placar do quality gate atualizado no KNOWN-BUGS.md (execução real de 18/08, com BOT4/MAT1 como dívidas documentadas)

PR substituta da #371 — a original ficou com registro wedged (head/base congelados no commit de um acidente de force-push, commits=0, sem reopen possível). Branch e conteúdo idênticos; reviews da original seguem lá.

@vercel

vercel Bot commented Aug 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
csbrasil Ready Ready Preview Aug 19, 2026 3:03am

Request Review

@csbrasil-BOT csbrasil-BOT added needs-human-gameplay Mudança exige revisão humana de gameplay/render/HUD needs-staging Mudança precisa validação integrada em staging target:main PR apontando para a branch main labels Aug 19, 2026
@csbrasil-BOT

csbrasil-BOT commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

csbrasil-bot classification

Note

@rubenmarcus abriu 15 arquivo(s) · +143 −155, base v2/fauna-todos-mapas.

O que mudou

área arquivos + amostra
📚 docs 8 +53 −111 ARCH.generated.md, KNOWN-BUGS.md, README.md
🧪 arnês 3 +47 −20 ARCH.md, redesign-check.mjs, screen-query-browser.mjs
🎮 jogo 1 +21 −11 main.js
🎨 assets 1 +1 −1 foot-offsets.json
📦 outros 1 +16 −9 style.css
🌐 site 1 +5 −3 index.astro

Labels

label por quê
needs-human-gameplay toca gameplay/render/HUD/mapa/personagem — decisão de humano
needs-staging mudança de runtime/UI — valida integrado no staging antes do merge
target:main vai direto pra main, sem fila de integração

@estraga-codigo estraga-codigo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Palhaço Mal

Palhaço Mal · revisor da casa · 🤡

mapa da mudança

  • docs (8): ARCH.generated.md, KNOWN-BUGS.md, README.md, STATUS.md
  • arnês (3): ARCH.md, redesign-check.mjs, screen-query-browser.mjs
  • jogo (1): main.js
  • assets (1): foot-offsets.json
  • outros (1): style.css
  • site (1): index.astro

Tip

Risco BAIXO · nota 10/10 ▓▓▓▓▓▓▓▓▓▓

limpo pelos critérios das minhas ações desta rodada

Revisão de bot — fio não resolvido TRAVA o merge (a main exige conversa resolvida). Resolve ou refuta.


todo bug confessa no diff, eu só faço o interrogatório 🤡

@estraga-codigo

Copy link
Copy Markdown
Collaborator

Esbirro

Esbirro · plantão da produção · 🕵️

check vermelho parado neste PR

Caution

Não sei classificar este vermelho. Segue o que o CI cuspiu — palpite de bot sobre bug alheio só atrapalha.

o quê valor
head 0f48ca65c108
checks vermelhos 2
classe outra
hora do plantão 2026-08-19T02:15:11.193Z

Vermelho:

classify (https://github.com/corosolto/client/actions/runs/32206150050/job/95929714234)
portao (https://github.com/corosolto/client/actions/runs/32205742947/job/95928578296)

Rabo do log:

2026-08-19T01:47:02.2163770Z env:
2026-08-19T01:47:02.2164180Z   GH_TOKEN: ***
2026-08-19T01:47:02.2164403Z   PR_NUMBER: 377
2026-08-19T01:47:02.2164616Z   DESIRED_BASE_BRANCH: staging
2026-08-19T01:47:02.2164925Z   pythonLocation: /opt/hostedtoolcache/Python/3.13.15/x64
2026-08-19T01:47:02.2165369Z   PKG_CONFIG_PATH: /opt/hostedtoolcache/Python/3.13.15/x64/lib/pkgconfig
2026-08-19T01:47:02.2165816Z   Python_ROOT_DIR: /opt/hostedtoolcache/Python/3.13.15/x64
2026-08-19T01:47:02.2166221Z   Python2_ROOT_DIR: /opt/hostedtoolcache/Python/3.13.15/x64
2026-08-19T01:47:02.2166605Z   Python3_ROOT_DIR: /opt/hostedtoolcache/Python/3.13.15/x64
2026-08-19T01:47:02.2166991Z   LD_LIBRARY_PATH: /opt/hostedtoolcache/Python/3.13.15/x64/lib
2026-08-19T01:47:02.2167315Z ##[endgroup]
2026-08-19T01:47:02.4529871Z GraphQL: Your token has not been granted the required scopes to execute this query. The 'login' field requires one of the following scopes: ['read:org'], but your token has only been granted the: ['repo', 'workflow'] scopes. Please modify your token's scopes at: https://github.com/settings/tokens., Your token has not been granted the required scopes to execute this query. The 'name' field requires one of the following scopes: ['read:org', 'read:discussion'], but your token has only been granted the: ['repo', 'workflow'] scopes. Please modify your token's scopes at: https://github.com/settings/tokens., Your token has not been granted the required scopes to execute this query. The 'slug' field requires one of the following scopes: ['read:org', 'read:discussion'], but your token has only been granted the: ['repo', 'workflow'] scopes. Please modify your token's scopes at: https://github.com/settings/tokens.
2026-08-19T01:47:02.5144372Z Traceback (most recent call last):
2026-08-19T01:47:02.5240062Z   File "<stdin>", line 7, in <module>
2026-08-19T01:47:02.5241395Z   File "/opt/hostedtoolcache/Python/3.13.15/x64/lib/python3.13/subprocess.py", line 577, in run
2026-08-19T01:47:02.5242220Z     raise CalledProcessError(retcode, process.args,
2026-08-19T01:47:02.5242678Z                              output=stdout, stderr=stderr)
2026-08-19T01:47:02.5243789Z subprocess.CalledProcessError: Command '['gh', 'pr', 'edit', '377', '--add-label', 'needs-human-gameplay,needs-staging,target:main']' returned non-zero exit status 1.
2026-08-19T01:47:02.5308473Z ##[error]Process completed with exit code 1.


nada fora do normal no meu turno 🕵️

@rubenmarcus
rubenmarcus force-pushed the v2/fauna-todos-mapas branch from 30a63c6 to dbdb7b0 Compare August 19, 2026 03:01
Palavras do dono (18/08): 'a tela tá muito difícil de navegar e poluída
porque já tem muitos mapas'. Medido antes: a strip de carrossel desenhada
pra 5 cartazes de 196px espremia os 12 cards a 98px (1536×1024) e 50px
(960×640), transbordando a faixa, com paginação de 5 quebrando o scan.

Agora: ficha do mapa à esquerda + GRADE auto-fill minmax(208px,1fr) com
scroll vertical à direita. Cards a 275px (3:2) e 228px (960), zero
paginação (fim dos dashes), ↑↓ pulam a LINHA da grade — colunas contadas
por offsetTop real, porque auto-fill muda com o viewport — e ←→/setas
continuam na ordem da categoria.

Réguas escritas ANTES (reprovavam o carrossel — conferido):
- UIR25 no eval:redesign: esqueleto da grade + teclado 4 direções
- eval:screenquery:browser mede o USO: cardMin ≥190px, cols ≥3 por
  offsetTop, overflow-y auto, e dois ArrowDown reais avançam 2×cols
- telas-menu7 continua válida (o fluxo de abertura não mudou)

Commit-grande: docs/arch regeridos após main.js/style/index mudarem de
tamanho (blocos gerados acompanham a árvore em cada PR).

Signed-off-by: rubenmarcus <rubenmarcus.dev@gmail.com>
Agent: OpenCode

@estraga-codigo estraga-codigo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Palhaço Mal

Palhaço Mal · revisor da casa · 🤡

mapa da mudança

  • docs (8): ARCH.generated.md, KNOWN-BUGS.md, README.md, STATUS.md
  • arnês (3): ARCH.md, redesign-check.mjs, screen-query-browser.mjs
  • jogo (1): main.js
  • assets (1): foot-offsets.json
  • outros (1): style.css
  • site (1): index.astro

Tip

Risco BAIXO · nota 10/10 ▓▓▓▓▓▓▓▓▓▓

nada aqui que quebre amanhã. amanhã é outro diff

Revisão de bot — fio não resolvido TRAVA o merge (a main exige conversa resolvida). Resolve ou refuta.


todo bug confessa no diff, eu só faço o interrogatório 🤡

@estraga-codigo

Copy link
Copy Markdown
Collaborator

Esbirro

Esbirro · plantão da produção · 🕵️

check vermelho parado neste PR

Caution

Não sei classificar este vermelho. Segue o que o CI cuspiu — palpite de bot sobre bug alheio só atrapalha.

o quê valor
head a4b8cfeaf9d6
checks vermelhos 1
classe outra
hora do plantão 2026-08-19T04:18:11.979Z

Vermelho:

portao (https://github.com/corosolto/client/actions/runs/32210746472/job/95942809723)

Log do run já expirou — abrir pelo link do check.


nada fora do normal no meu turno 🕵️

@rubenmarcus
rubenmarcus merged commit 5459196 into v2/fauna-todos-mapas Aug 19, 2026
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-human-gameplay Mudança exige revisão humana de gameplay/render/HUD needs-staging Mudança precisa validação integrada em staging target:main PR apontando para a branch main

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants