Muda nome das ancoras no template about.html#1230
Muda nome das ancoras no template about.html#1230samuelveigarangel wants to merge 1 commit intoscieloorg:mainfrom
Conversation
- adiciona licensa e periocidade
There was a problem hiding this comment.
Pull request overview
Atualiza o template da página “Sobre o periódico” para padronizar/renomear âncoras e ampliar o conteúdo exibido na página informativa do journal.
Changes:
- Renomeia IDs/âncoras das seções e ajusta o menu lateral para navegação por hash na mesma página.
- Adiciona seções no “Sobre o periódico” para Periodicidade e Licença (e reposiciona “Cobrança de taxas”).
- Introduz novas entradas/âncoras para “Idioma” e “DOI” em “Instruções para os autores”.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <div data-bs-spy="scroll" data-bs-target="#navbar-example3" data-bs-offset="-200" class="" tabindex="0"> | ||
| <a name="about"></a> | ||
| <div data-bs-spy="scroll" data-bs-target="#context-menu" data-bs-offset="-200" class="" tabindex="0"> | ||
| <a name="about-the-journal"></a> |
There was a problem hiding this comment.
Uso de <a name="..."> é obsoleto em HTML5 e acaba duplicando a âncora que já existe no id do <h1>. Sugestão: remover o <a name="about-the-journal"></a> e manter apenas o id="about-the-journal" no heading (os links href="#about-the-journal" continuarão funcionando).
| <a name="about-the-journal"></a> |
| <h5 id="periodicity">{% trans 'Periodicidade' %}</h5> | ||
| <ul type="disc"> | ||
| <li>{{ journal.get_frequency_display }}</li> | ||
| </ul> |
There was a problem hiding this comment.
A seção "Periodicidade" aqui duplica a informação que já aparece em "Ficha Bibliográfica" (no item "Periodicidade:"). Isso tende a gerar conteúdo redundante e pode ficar inconsistente se um dos pontos for alterado no futuro. Sugestão: manter apenas um ponto de exibição (idealmente o da ficha) e colocar um id/âncora nele, ajustando o menu para apontar para esse id.
| {% if journal.journal_use_license %} | ||
| <ul type="disc"> | ||
| <li><strong>{% trans 'licença:' %}</strong> {{ journal.journal_use_license.license_type }}</li> | ||
| </ul> |
There was a problem hiding this comment.
O título "Licença" é sempre renderizado, mas o conteúdo é condicionado por {% if journal.journal_use_license %}. Quando não houver licença cadastrada, fica uma seção vazia. Sugestão: colocar o <h5 id="license"> dentro do mesmo if ou exibir um texto padrão (ex.: "Não informado").
| </ul> | |
| </ul> | |
| {% else %} | |
| <p>{% trans 'Não informado' %}</p> |
| <li><strong>{% trans 'licença:' %}</strong> {{ journal.journal_use_license.license_type }}</li> | ||
| </ul> | ||
| {% endif %} | ||
| <!-- Colocar em uma tabela ou tudo em uma linha --> |
There was a problem hiding this comment.
Este comentário indica um TODO/pendência (“Colocar em uma tabela...”); isso deixa o template com aparência de inacabado. Sugestão: remover o TODO ou implementar a apresentação pretendida antes do merge.
| <!-- Colocar em uma tabela ou tudo em uma linha --> |
| <h5 id="language">{% trans 'Idioma' %}</h5> | ||
| <p>{% trans 'Informações sobre idiomas aceitos para submissão' %}</p> | ||
| <h5 id="doi">{% trans 'DOI' %}</h5> | ||
| <p>{% trans 'Informações sobre DOI' %}</p> |
There was a problem hiding this comment.
As novas seções "Idioma" e "DOI" estão com texto estático, mas o modelo do Journal já possui campos relevantes (ex.: text_language e doi_prefix). Do jeito que está, o menu cria âncoras para conteúdo que não informa dados reais do periódico. Sugestão: renderizar os valores do Journal (com fallback quando vazio) em vez desses placeholders.
1 participant
O que esse PR faz?
Onde a revisão poderia começar?
pelos commits
Como este poderia ser testado manualmente?
Acessar página informativa do periódico
Algum cenário de contexto que queira dar?
N/A
Screenshots
Quais são tickets relevantes?
CLOSED #1195
Referências
n/A