Feat/circuitos

[lpirola]

Resumo

• Nova rota “Circuitos”: criação do CircuitosController com action ALL_index, renderizando views/circuitos/index.php e aplicando filtro inicial por selos (operador OR) para listar projetos do circuito. ([GitHub][1])

  • Pseudo-query inicial com @seals (106,107,108,109,110). ([GitHub][1])

• Registro de controller no tema: Theme.php passa a registrar circuitos e mantém funarte_search. Também há ajustes de ícones via hook component(mc-icon).iconset. ([GitHub][1])

• Navegação / UI: inclusão de item de menu para “Circuitos” (header) e criação de layouts/parts/mc-header-menu-circuitos.php. (listado na árvore de arquivos do diff). ([GitHub][1])

• Componentes novos/ajustados:

  • entity-card-circuitos (template, script, texts, README).
  • search-list-circuitos (template, script, texts).
  • Reaproveitamento de agenda-section para programação, project-occurrences, search-list-agenda e versão “lazy” para carregamento sob demanda; search-filter-project agora aceita selo como opção. (todos aparecem no Files Changed). ([GitHub][1])

• Estilos: ajustes de CSS/SCSS para padronizar cores e espaçamentos (theme-BaseV2.scss) e atualização do estilo da lista de eventos. (citados no histórico de commits do PR). ([GitHub][2])

• Outros: correção de link para o manual; atualização de views/project/single.php e config.php. (listado no diff/commits). ([GitHub][2])

[lpirola]

Revisão

1. IDs de selos como “magic numbers”

   • Hoje: ['106,107,108,109,110'] (um único item com IDs separados por vírgula). Confere se o mecanismo do Mapas aceita array com um item string ou se o esperado é um array de IDs (['106','107',...]) ou até ints. Sugiro normalizar e mover para config.php/ENV para evitar “magic numbers” e facilitar manutenção/variação por instância. ([GitHub]1)

2. Extensibilidade da pseudo-query

   • A pseudo-query inicial recebe hook search-projects-initial-pseudo-query. 👍
   • Sugiro permitir overlay via querystring (ex.: ?seals=...&uf=...) preservando defaults, e documentar isso no README do componente/rota. ([GitHub]1)

3. Paginação & lazy loading

   • Como a listagem tende a crescer, garanta paginação/“infinite scroll” em search-list-circuitos. A existência de search-list-agenda-lazy indica a direção certa—reusar padrão “lazy” para circuitos manterá performance. (Arquivos listados mostram ambos). ([GitHub]1)

4. Ordenação estável

   • Há um commit de “Refinamento de componentes e ordenação”. Verifica se a ordenação é determinística (ex.: por período ativo do circuito, depois updatedAt), e se está declarada no front e no backend (para evitar diferenças entre lotes/carregamento incremental). ([GitHub]2)

5. Acessibilidade e i18n

   • Nos novos cards e listas, garantir:

     • textos em texts.php com i() onde for exibido;
     • alt-text e hierarquia semântica (H1/H2/…);
     • foco/aria-labels em botões de filtros e paginação. (Arquitetura de componentes evidencia texts.php). ([GitHub]1)

6. Resiliência de dados

   • Estados “vazio/erro”: quando nenhum projeto possuir selos definidos, renderizar mensagem amigável e CTA (ex.: “veja o manual” — já há commit corrigindo link). ([GitHub]2)

7. Configuração via ENV

   • Em Theme.php já há METABASE_FUNARTE_ENABLED. Seguir a mesma linha para lista de selos, limites de paginação e datas de período melhora portabilidade entre ambientes. ([GitHub]1)

8. Documentação

   • Incluir seção no README do tema explicando: rota /circuitos, parâmetros suportados, filtros (incl. selos), componentes envolvidos e como trocar a lista de selos. Facilita on-boarding.

9. Ícones / Design tokens

   • Os ícones foram ajustados via hook; validar consistência com a iconografia geral do tema (tamanho, stroke/fill). Se já existe token de cor/tamanho, use-o no SCSS. ([GitHub]1)

10. Testabilidade

• Se houver testes de aceitação (Cypress/Playwright), vale criar 2 cenários:

  • “lista com resultados” (mock com pelo menos 2 projetos, selos distintos);
  • “lista vazia” (nenhum selo).

• Garante estabilidade em futuras trocas de selos.

Checklist de QA (pronto para merge?)

• Rota acessível: /circuitos abre sem autenticação e com status 200.
• Menu: item “Circuitos” aparece no header e navega corretamente. ([GitHub]1)
• Filtro inicial: a pseudo-query com selos realmente filtra (validar no banco/fixtures). ([GitHub]1)
• Paginação / lazy: rolagem ou botão “carregar mais” funciona sem regressões (sem flashes de layout).
• Ordenação: ordem dos projetos consistente entre recargas.
• Acessibilidade: navegação por teclado, aria-labels, contraste.
• Traduções: textos em texts.php aparecem localizados.
• Fallbacks: estados vazio/erro exibem mensagem e link para manual (commit “Corrige link para manual”). ([GitHub]2)
• Config: IDs de selos fora do código (ENV/config) — recomendação para o próximo commit.
• Docs: README do tema/rota atualizado.

Se quiser, já te mando um patch colocando os selos em config.php/ENV, ajustando o array para ['106','107','108','109','110'] (ou ints) e adicionando um README curto da rota.