Skip to content

Adiciona no badges no modal e adiciona a capacidade de criar os estáticos para o template do scieloorg#541

Open
gitnnolabs wants to merge 1 commit intoscieloorg:masterfrom
gitnnolabs:tks419_540
Open

Adiciona no badges no modal e adiciona a capacidade de criar os estáticos para o template do scieloorg#541
gitnnolabs wants to merge 1 commit intoscieloorg:masterfrom
gitnnolabs:tks419_540

Conversation

@gitnnolabs
Copy link
Collaborator

O que esse PR faz?

Esse PR adiciona as métricas em modal assim como no scielo.br e adiciona a capacidade de criar os arquivos .css e js a partir do gulp para o template do scieloorg.

Onde a revisão poderia começar?

Sugiro que a revisão seja feita em uma instalação local.

Para a validação do @alexxxmendonca nessa atividade irei atualiza o ambiente de homologação, link para o ambiente: https://homolog-search.scielo.org/

Como este poderia ser testado manualmente?

Acessando a instância de homologação: https://homolog-search.scielo.org/

Algum cenário de contexto que queira dar?

Não foi inserido o scienceopen, pois, deve ser solicitado um novo ID para a instituição. Tentei utiliza o mesmo ID que é utilizado pelo site scielo.br porém sem sucesso.

Screenshots

Captura de Tela 2021-07-27 às 23 23 12

IMPORTANTE: o Plumx não aparece pois depende de um domínio público.

Quais são tickets relevantes?

#419 e #540

Referências

N/A

@gitnnolabs gitnnolabs added the enhancement New feature or request label Jul 28, 2021
@gitnnolabs
Copy link
Collaborator Author

gitnnolabs commented Jul 28, 2021

@alexxxmendonca o ambiente de homolog esta com essa alteração.

Link para aceso: https://homolog-search.scielo.org/

@gitnnolabs
Copy link
Collaborator Author

gitnnolabs commented Jul 28, 2021

IMPORTANTE: Os badges do altmetrics, dimensions, scite_ estão com uma flag que não exibi o badge em caso de não existir métrica.

Ai ficamos com os textos no bottom... o que parece um erro! Não é simples exibir ou ocultar o esse texto e para cada artigo teríamos que fazer vários requests para terceiros, o que me leva a sugerir a remoção desse texto!

No caso do search isso seria mais complicado ainda pela quantidade de requisição…

@robertatakenaka
Copy link
Member

@gitnnolabs @alexxxmendonca Que tal se o texto ficar como tooltip?

@alexxxmendonca
Copy link

Eu não sei se a solução satisfaz. Esconder o texto pode comprometer a acessibilidade. Podemos usar tooltips, desde que sejam acessíveis (acessibilidade).

@gitnnolabs
Copy link
Collaborator Author

@alexxxmendonca podemos incluir esse PR, para disponibilizar os badges?

@alexxxmendonca
Copy link

@gitnnolabs eu não sei se tooltip satisfaz a questão da acessibilidade.

podemos pular esse ticket?

@gitnnolabs
Copy link
Collaborator Author

OK, vamos deixa-lo para depois.

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants