+
Skip to content

Conversation

paratiuid
Copy link
Contributor

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

Screen Shot 2023-01-12 at 08 37 24
Screen Shot 2023-01-12 at 08 37 38
Screen Shot 2023-01-12 at 08 38 03

Quais são tickets relevantes?

Indique uma issue ao qual o pull request faz relacionamento.

Referências

--

Ramon Cordini and others added 30 commits March 18, 2022 15:06
…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
…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
Ramon Cordini added 25 commits September 22, 2022 17:29
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">
Copy link
Member

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?

Copy link
Contributor Author

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 "--"

Copy link
Member

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"?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

modal

@robertatakenaka
Copy link
Member

@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
Veja o PDF https://www.scielo.br/j/rlae/a/L34w8qg8ccfQxW79FZH3Bnh/?format=pdf&lang=en

Os HTML gerados pelo packtools v2.0 (atual) fica correto apesar de no site não ficar.
A versão 3.0 fica cortada.
Imagens_tamanho_padrao_nao_colocar_thumbnail.zip

@paratiuid
Copy link
Contributor Author

@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 Veja o PDF https://www.scielo.br/j/rlae/a/L34w8qg8ccfQxW79FZH3Bnh/?format=pdf&lang=en

Os HTML gerados pelo packtools v2.0 (atual) fica correto apesar de no site não ficar. A versão 3.0 fica cortada. Imagens_tamanho_padrao_nao_colocar_thumbnail.zip

@robertatakenaka

modifiquei o html para exibir a imagem fora do thumbnail.
Veja o arquivo em anexo:
corrigido - imagens_fora_de_fig.en.3_0.html.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载