scieloorg / search-journals

iAHx Search Interface for SciELO

http://search.scielo.org

BSD 2-Clause "Simplified" License

3 stars 13 forks source link

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

Open gitnnolabs opened 3 years ago

gitnnolabs commented 3 years ago

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 à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 commented 3 years ago

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

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

gitnnolabs commented 3 years ago

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 commented 3 years ago

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

alexxxmendonca commented 3 years ago

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 commented 2 years ago

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

alexxxmendonca commented 2 years ago

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

podemos pular esse ticket?

gitnnolabs commented 2 years ago

OK, vamos deixa-lo para depois.