Alterados htmls 3.0 - diretorio htmlgenerator - Design system artigo #331
Conversation
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
…evem ser removidos ao concluir os envios dos ajustes. Trocadas as classes css do elemento article e do elemento articleMenu.
…a-bs-toggle. Sai data-target e entra data-bs-target. Foram comentados os css antigos para não impactar na visualização dos ajustes. Esses css antigos devem voltar para o arquivo ao final dos ajustes.
…i alterada e alguns itens trocados. Esse mesmo procedimento será adotado para todos os modais existentes na página em commimts futuros
…com espaço após a imagem
…o idioma da interface. Este se mantém como h1. Os títulos abaixo do h3 devem seguir a hierarquia: h4, h5, etc.
…reference` (dar preferência a tex-math ou mml:math) Aceita os valores de '2.0' e '3.0' para o parâmetro `xslt`
…733c53c) Troca ``` <ul class="col-md-2 hidden-sm articleMenu"></ul> <article id="articleText" class="col-md-10 col-md-offset-2 col-sm-12 col-sm-offset-0"><div class="articleSection" data-anchor="RESUMO"> ``` por ``` <ul class="d-none d-lg-block col-lg-2 articleMenu"></ul> <article id="articleText" class="col-sm-12 col-lg-10 offset-lg-2"><div class="articleSection" data-anchor="RESUMO"> ```
Sai data-toggle e entra data-bs-toggle. Sai data-target e entra data-bs-target. 7d29894
Linkados os javascripts do design system versão 1.1.3. A sequencia correta dos javascripts é jquery, bootstrap e scielo-ds. Atualizado o caminho para o logo-open-acces.svg
2 - inclusão dos css bootstrap.css e article.css no topo do arquivo 3 - mantive comentada a chamada do css de impressão pois esta está pendente de ajustes e testes. 4 - inclusão da url da imagem do icone open acces. Muito importante manter um espaço logo após a imagem do open access. 5 - adicionadas classes btn btn-sm btn-secondary ao botao sobre o autor 6 - inclusão dos javascripts jquery, bootstrap.bundle.min e scielo-ds-min.js
2 - inclusão dos css bootstrap.css e article.css no topo do arquivo 3 - mantive comentada a chamada do css de impressão pois esta está pendente de ajustes e testes. 4 - inclusão da url da imagem do icone open acces. Muito importante manter um espaço logo após a imagem do open access. 5 - adicionadas classes btn btn-sm btn-secondary ao botao sobre o autor 6 - inclusão dos javascripts jquery, bootstrap.bundle.min e scielo-ds-min.js 7 - dentro do elemento modal-header o link para abrir em nova janela deve vir antes do botão de fechar.
2 - inclusão dos css bootstrap.css e article.css no topo do arquivo 3 - mantive comentada a chamada do css de impressão pois esta está pendente de ajustes e testes. 4 - inclusão da url da imagem do icone open acces. Muito importante manter um espaço logo após a imagem do open access. 5 - adicionadas classes btn btn-sm btn-secondary ao botao sobre o autor 6 - inclusão dos javascripts jquery, bootstrap.bundle.min e scielo-ds-min.js 7 - dentro do elemento modal-header o link para abrir em nova janela deve vir antes do botão de fechar.
2 - inclusão dos css bootstrap.css e article.css no topo do arquivo 3 - mantive comentada a chamada do css de impressão pois esta está pendente de ajustes e testes. 4 - inclusão da url da imagem do icone open acces. Muito importante manter um espaço logo após a imagem do open access. 5 - adicionadas classes btn btn-sm btn-secondary ao botao sobre o autor 6 - inclusão dos javascripts jquery, bootstrap.bundle.min e scielo-ds-min.js 7 - dentro do elemento modal-header o link para abrir em nova janela deve vir antes do botão de fechar.
2 - inclusão dos css bootstrap.css e article.css no topo do arquivo 3 - mantive comentada a chamada do css de impressão pois esta está pendente de ajustes e testes. 4 - inclusão da url da imagem do icone open acces. Muito importante manter um espaço logo após a imagem do open access. 5 - adicionadas classes btn btn-sm btn-secondary ao botao sobre o autor 6 - inclusão dos javascripts jquery, bootstrap.bundle.min e scielo-ds-min.js 7 - removidas as tags <br> antes de links com classe link-newWindow showTooltip nos modais
2 - inclusão dos css bootstrap.css e article.css no topo do arquivo 3 - mantive comentada a chamada do css de impressão pois esta está pendente de ajustes e testes. 4 - inclusão da url da imagem do icone open acces. Muito importante manter um espaço logo após a imagem do open access. 5 - adicionadas classes btn btn-sm btn-secondary ao botao sobre o autor 6 - inclusão dos javascripts jquery, bootstrap.bundle.min e scielo-ds-min.js 7 - removido o atributo href dos links que abrem modais
2 - inclusão dos css bootstrap.css e article.css no topo do arquivo 3 - mantive comentada a chamada do css de impressão pois esta está pendente de ajustes e testes. 4 - inclusão da url da imagem do icone open acces. Muito importante manter um espaço logo após a imagem do open access. 5 - adicionadas classes btn btn-sm btn-secondary ao botao sobre o autor 6 - inclusão dos javascripts jquery, bootstrap.bundle.min e scielo-ds-min.js 7 - removidas as tags <br> antes de links com classe link-newWindow showTooltip nos modais 8 - links com classe link-newWindow showTooltip nos modais devem estar junto com o título do modal, sem <br>
2 - inclusão dos css bootstrap.css e article.css no topo do arquivo 3 - mantive comentada a chamada do css de impressão pois esta está pendente de ajustes e testes. 4 - inclusão da url da imagem do icone open acces. Muito importante manter um espaço logo após a imagem do open access. 5 - adicionadas classes btn btn-sm btn-secondary ao botao sobre o autor 6 - inclusão dos javascripts jquery, bootstrap.bundle.min e scielo-ds-min.js 7 - links com classe link-newWindow showTooltip nos modais devem estar junto com o título do modal, sem <br>
2 - inclusão dos css bootstrap.css e article.css no topo do arquivo 3 - mantive comentada a chamada do css de impressão pois esta está pendente de ajustes e testes. 4 - inclusão da url da imagem do icone open acces. Muito importante manter um espaço logo após a imagem do open access. 5 - adicionadas classes btn btn-sm btn-secondary ao botao sobre o autor 6 - inclusão dos javascripts jquery, bootstrap.bundle.min e scielo-ds-min.js 7 - links com classe link-newWindow showTooltip nos modais devem estar junto com o título do modal, sem <br> apresenta problema de renderizacao de formula na linha 198
2 - inclusão dos css bootstrap.css e article.css no topo do arquivo 3 - mantive comentada a chamada do css de impressão pois esta está pendente de ajustes e testes. 4 - inclusão da url da imagem do icone open acces. Muito importante manter um espaço logo após a imagem do open access. 5 - adicionadas classes btn btn-sm btn-secondary ao botao sobre o autor 6 - inclusão dos javascripts jquery, bootstrap.bundle.min e scielo-ds-min.js
2 - inclusão dos css bootstrap.css e article.css no topo do arquivo 3 - mantive comentada a chamada do css de impressão pois esta está pendente de ajustes e testes. 4 - inclusão da url da imagem do icone open acces. Muito importante manter um espaço logo após a imagem do open access. 5 - adicionadas classes btn btn-sm btn-secondary ao botao sobre o autor 6 - inclusão dos javascripts jquery, bootstrap.bundle.min e scielo-ds-min.js
2 - inclusão dos css bootstrap.css e article.css no topo do arquivo 3 - mantive comentada a chamada do css de impressão pois esta está pendente de ajustes e testes. 4 - inclusão da url da imagem do icone open acces. Muito importante manter um espaço logo após a imagem do open access. 5 - adicionadas classes btn btn-sm btn-secondary ao botao sobre o autor 6 - inclusão dos javascripts jquery, bootstrap.bundle.min e scielo-ds-min.js
2 - inclusão dos css bootstrap.css e article.css no topo do arquivo 3 - mantive comentada a chamada do css de impressão pois esta está pendente de ajustes e testes. 4 - inclusão da url da imagem do icone open acces. Muito importante manter um espaço logo após a imagem do open access. 5 - adicionadas classes btn btn-sm btn-secondary ao botao sobre o autor 6 - inclusão dos javascripts jquery, bootstrap.bundle.min e scielo-ds-min.js
2 - inclusão dos css bootstrap.css e article.css no topo do arquivo 3 - mantive comentada a chamada do css de impressão pois esta está pendente de ajustes e testes. 4 - inclusão da url da imagem do icone open acces. Muito importante manter um espaço logo após a imagem do open access. 5 - adicionadas classes btn btn-sm btn-secondary ao botao sobre o autor 6 - inclusão dos javascripts jquery, bootstrap.bundle.min e scielo-ds-min.js 7 - removidas as tags <br> antes de links com classe link-newWindow showTooltip nos modais 8 - removido o atributo href dos links que abrem modais 9 - textos nos modais devem estar dentro da div com classe .modal-body 10 - existe um elemento ref footnote no título. Este deve ser tratado no css do design system.
2 - inclusão dos css bootstrap.css e article.css no topo do arquivo 3 - mantive comentada a chamada do css de impressão pois esta está pendente de ajustes e testes. 4 - inclusão da url da imagem do icone open acces. Muito importante manter um espaço logo após a imagem do open access. 5 - adicionadas classes btn btn-sm btn-secondary ao botao sobre o autor 6 - inclusão dos javascripts jquery, bootstrap.bundle.min e scielo-ds-min.js 7 - existe um elemento ref footnote no título. Este deve ser tratado no css do design system.
2 - inclusão dos css bootstrap.css e article.css no topo do arquivo 3 - mantive comentada a chamada do css de impressão pois esta está pendente de ajustes e testes. 4 - inclusão da url da imagem do icone open acces. Muito importante manter um espaço logo após a imagem do open access. 5 - adicionadas classes btn btn-sm btn-secondary ao botao sobre o autor 6 - inclusão dos javascripts jquery, bootstrap.bundle.min e scielo-ds-min.js 7 - textos nos modais devem estar dentro da div com classe .modal-body
2 - inclusão dos css bootstrap.css e article.css no topo do arquivo 3 - mantive comentada a chamada do css de impressão pois esta está pendente de ajustes e testes. 4 - inclusão da url da imagem do icone open acces. Muito importante manter um espaço logo após a imagem do open access. 5 - adicionadas classes btn btn-sm btn-secondary ao botao sobre o autor 6 - inclusão dos javascripts jquery, bootstrap.bundle.min e scielo-ds-min.js 7 - removidas as tags <br> antes de links com classe link-newWindow showTooltip nos modais 8 - removido o atributo href dos links que abrem modais
| <button class="btn-close" data-bs-dismiss="modal" aria-label="Close"></button> | ||
| </div>Tabela 1Classificação Sucessional adotada por alguns autores ao longo dos anos.<div class="modal-body"><div class="table table-hover"><table> | ||
| </div><div class="modal-body"> | ||
| Tabela 1Classificação Sucessional adotada por alguns autores ao longo dos anos. |
Member
There was a problem hiding this comment.
@paratiuid este texto está a mais. Ele já tinha sido inserido na linha 747 | 750
Member
|
@paratiuid @alexxxmendonca
PROBLEMA 1: Nota do título muito grande e a caixa de texto correspondente está ilegível
Observação:
PROBLEMA 2: Aparência dos botões/links no final da página está conforme esperado?
Observação: PROBLEMA 3: Ao clicar nas abas, não altera o conteúdo.Modal dos elementos gráficos (Figuras, Tabelas, Equações) que são apresentados somente se existem ao clicar em um dos botões no final da página
O problema é que o link da aba 2 não funciona PROBLEMA 4: Não é possível ver o conteúdo completoModal dos elementos gráficos (Figuras, Tabelas, Equações) que são apresentados somente se existem ao clicar em um dos botões no final da página PROBLEMA 5: Modal como citar - link de copiar está com aparência estranha e não funciona
|
Member
|
@alexxxmendonca @RPostalli |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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?
Contexto do ambiente de desenvolvimento:
1 - abri o PR: Aplica design system no htmlgenerator #317
2 - fiz merge com os arquivos da master
3 - verifiquei os arquivos html com final 3.0
Para todos os arquivos com final 3.0 foram linkados css atualizados do design system. Estes foram utilizados via url absoluta:
https://scielo.parati.design/aberto-ds-scielo/dist/version/1.1.3/css/bootstrap.css
https://scielo.parati.design/aberto-ds-scielo/dist/version/1.1.3/css/article.css
o mesmo foi feito para os arquivos .js
https://scielo.parati.design/aberto-ds-scielo/dist/version/1.1.3/js/bootstrap.bundle.min.js
https://scielo.parati.design/aberto-ds-scielo/dist/version/1.1.3/js/scielo/scielo-ds-min.js
Para os javascripts, foram adicionados ainda, antes dos dois mencionados, o script jquery. Para o correto funcionamento das funcionalidades do design system, esta deve ser a ordem de chamada dos scripts:
1 - jquery
2 - bootstrap.bundle.js
3 - scielo-ds-min.js
Com esses arquivos css e js devidamente alterados, ao abrir esses html com final 3.0 a aparencia final do artigo deve estar correta. Modais abrindo, css funcionando.
Os arquivos foram comitados individualmente ou no máximo em grupos de 2 ou 3 arquivos com descrição que explica o que foi alterado e por qual motivo.
Onde a revisão poderia começar?
Analisar somente os arquivos html dentro do diretório:
htmlgenerator/*/*3.0.html
Cada html com contendo 3_0.html foi alterado, corrigido e testado.
A maioria das correções feitas foram:
a) adicionar um espaço após a imagem do ícone open access
b) adicionar classes corretas no botão Sobre os autores
c) remoção de tag
antes de link para abrir em nova janela dentro dos modais
d) adicionada a tag meta viewport para permitir comportamento responsivo
Como este poderia ser testado manualmente?
Abrindo cada um dos html que contém 3.0.html no título.
Algum cenário de contexto que queira dar?
Não foi mexido no componente floatingMenu.
É necessário entender se este componente é gerado nessa etapa do processo ou em outra. Senti falta de algumas funções nesse componente quando este aparecia nos htmls gerados.
Screenshots
Quais são tickets relevantes?
Referências
Indique as referências utilizadas para a elaboração do pull request.