fix: correct articleMenu class name in v3.0 XSLT and regenerate fixtures#1123
Draft
fix: correct articleMenu class name in v3.0 XSLT and regenerate fixtures#1123
Conversation
…tures - Fix `<ul>` class from `-articleMenu` to `articleMenu` in v3.0 article.xsl - Structure already uses correct column layout (col-12 col-md-4 col-lg-3 for menu, col-sm-12 col-md-8 col-lg-9 for article content) - Article element has no grid classes (grid applied to parent div) - Menu uses list-group Bootstrap component classes - Regenerate all v3.0 HTML fixture files to reflect the updated XSLT output Co-authored-by: robertatakenaka <505143+robertatakenaka@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Update HTML of article section menu
fix: correct articleMenu class name in v3.0 XSLT and regenerate fixtures
Mar 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
O que esse PR faz?
Corrige o nome da classe CSS no
<ul>do menu de seções do artigo no XSLT v3.0:-articleMenu→articleMenu(remover prefixo-incorreto).A estrutura de colunas separadas (menu + conteúdo) já estava implementada no v3.0 — apenas o nome da classe estava errado. Os fixtures HTML foram regenerados para refletir a saída correta.
Antes:
Depois:
A estrutura v3.0 já atende todos os requisitos do issue:
div.col-12.col-md-4.col-lg-3com<nav>acessíveldiv.col-sm-12.col-md-8.col-lg-9<article id="articleText">sem classes de grid<ul>com classeslist-group mt-4do BootstrapOnde a revisão poderia começar?
packtools/catalogs/htmlgenerator/v3.0/article.xsl— linha 201, única alteração de código.Como este poderia ser testado manualmente?
Verificar no HTML gerado que a
<ul>possuiclass="articleMenu list-group mt-4"e que a estrutura de colunas está correta.Algum cenário de contexto que queira dar?
As classes
list-group-itemelist-group-item-actionnos<li>e<a>são aplicadas dinamicamente pelo front-end, conforme nota no issue original. Os 72 fixtures.3_0.htmlforam regenerados a partir dos XMLs fonte — esses arquivos não são referenciados por testes automatizados.Screenshots
N/A — alteração estrutural no HTML gerado.
Quais são tickets relevantes?
Atualizar HTML do menu de seções do artigo.
Referências
Original prompt
This section details on the original issue you should resolve
<issue_title>Atualizar html do menu de seções do artigo.</issue_title>
<issue_description>### Descrição da tarefa
Atualizar html do menu de seções do artigo.
O HTML atual renderiza o menu de seções diretamente dentro da .row e aplica classes de grid diretamente na
<ul>e no<article>.No novo layout, o menu e o conteúdo do artigo precisam estar separados em colunas distintas, seguindo o novo padrão de grid.
Além disso, os itens do menu passam a utilizar classes do componente list-group do Bootstrap.
Estrutura atual (HTML gerado atualmente)
Estrutura esperada (novo layout)
Subtarefas
O packtools deve renderizar a estrutura dentro da .row da seguinte forma:
row
├── coluna menu
└── coluna conteúdo do artigo
Exemplo esperado:
A lista deve utilizar o componente Bootstrap List Group.
Atual
<ul class="d-none d-lg-block col-lg-2 articleMenu">Novo
<ul class="articleMenu list-group mt-4">Cada
<li>deve receber a classe:list-group-itemO item ativo continua utilizando:
selectedAtual
Novo
Cada link
<a>dentro do menu deve possuir:list-group-item-action d-block<article>Atual
<article id="articleText" class="col-sm-12 col-lg-10 offset-lg-2">Novo
<article id="articleText">O grid passa a ser aplicado apenas no container pai:
<div class="col-sm-12 col-md-8 col-lg-9">Checklist de implementação
div.col-12 col-md-4 col-lg-3div.col-sm-12 col-md-8 col-lg-9<ul>paraarticleMenu list-group mt-4list-group-itemem cada<li>list-group-item-action d-blockem cada<a><article>Considerações e notas
Os itens relacionados à inclusão das classes:
podem estar associados à geração ou manipulação do menu no lado do front-end.
Caso essas classes estejam sendo aplicadas dinamicamente pelo front-end, os pontos referentes a essas classes podem ser desconsiderados nesta implementação, mantendo apenas os ajustes estruturais descritos neste issue.
</issue_description>
<agent_instructions>@copilot ajuste as xsl da versão v3.0 do htmlgenerator para realizar a correção no layout da página do artigo. Faça commits atômicos.</agent_instructions>
Comments on the Issue (you are @copilot in this section)
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.