-
Notifications
You must be signed in to change notification settings - Fork 24
Correções no corpo do artigo solicitados no pr 331 #361
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
…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
<!-- exibe em tudo menos mobile--> | ||
<ul class="fm-list d-none d-sm-block"> | ||
<li> | ||
<a class="fm-button-child" --data-bs-toggle="tooltip" title="" data-mobile-tooltip="Figuras e tabelas" data-bs-toggle="modal" data-bs-target="#ModalTablesFigures" data-bs-original-title="Figuras e tabelas"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@paratiuid --data-bs-toggle=
está correto?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@robertatakenaka não. O correto é data-bs-toggle=
Sem os "--"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@paratiuid notei que data-bs-toggle
está duplicado. Qual prevalece? data-bs-toggle="modal" ou ="tooltip"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
modal
@paratiuid por favor, verificar que há imagens que não são figuras, então elas devem ser apresentadas sem thumbnail Veja que o site atual já apresenta problema: https://www.scielo.br/j/rlae/a/L34w8qg8ccfQxW79FZH3Bnh/?lang=en Os HTML gerados pelo packtools v2.0 (atual) fica correto apesar de no site não ficar. |
modifiquei o html para exibir a imagem fora do thumbnail. |
O que esse PR faz?
Uma série de correções de interface alterando classes para que o comportamento seja compatível com a nova versão do boostrap e design system.
Onde a revisão poderia começar?
Abra o arquivo:
tests/fixtures/htmlgenerator/table_wrap_group_and_fig_group/2236-8906-hoehnea-49-e1082020/2236-8906-hoehnea-49-e1082020.pt.3_0.html
Teste ele no navegador e verifique que os ajustes solicitados no pr 331 foram corrigidas.
Como este poderia ser testado manualmente?
Siga os passos descritos acima.
Algum cenário de contexto que queira dar?
Foram mantidos apenas para fins de teste os estilos css necessários para a correção solicitada. Estes estilos que estão contidos das linhas 14 até 32 não devem ser incorporados a produção. Estes estão somente a fim de testes. Estes novos estilos entrarão na mais recente versão do SciELO Design System.
Screenshots
Quais são tickets relevantes?
Indique uma issue ao qual o pull request faz relacionamento.
Referências
--