Criada ação de copiar DOI na interface responsiva

[paratiuid]

O que esse PR faz?

Adiciona a funcionalidade de copiar DOI na lista de resultados da busca, quando esta está com largura compatível com dispositivos mobile. Além de copiar o doi, é exibido um feedback deixando o botão verde e exibindo um ícone de check na cor branca, indicando que a ação foi efetuada com sucesso.

Onde a revisão poderia começar?

1 - Acesse a lista de resultados 2 - clique sobre um item da lista 3 - no box de detalhes que aparecerá, localize o botão COPIAR DOI e clique. Deve ocorrer a animação de feedback e o DOI deve estar na área de transferencia. Tente colar em algum campo de texto para ver se foi copiado corretamente.

Como este poderia ser testado manualmente?

Siga os passo já citados.

Algum cenário de contexto que queira dar?

• Neste PR estão sendo enviados vários commits além do meu. Trata-se de commits que foram enviados para a master. Ao criar essa branch de trabalho, atualizei com a versão mais recente da master. Ao alinhar sobre esses vários commits com o @jamilatta , entendemos que o processo está correto, pois estamos atualizando a branch interface-responsiva com os ajustes efetuados na master.

Screenshots

Quais são tickets relevantes?

476

Referências

Um botão semelhante já existe na tela do artigo, no opac. Foi mantido o mesmo padrão.

[paratiuid]

@paratiuid, fiz o teste do PR na branch interface-responsiva e o botão de cópia do DOI funciona como esperado. Entretanto, o botão de compartilhamento, por exemplo, parou de funcionar. Entendo que não faz parte deste PR mas, ao incorporar este PR à branch quebrará as implementações anteriores. Poderia verificar?

@patymori este problema ocorre devido a um ajuste que está sendo realizado pelo @jamilatta com relação aos documentos preprint.

Peço que se puder, converse com ele para verificar se é isso mesmo. Caso não seja, posso verificar com certeza.

[jamilatta]

@paratiuid @joffilyfe @robertatakenaka @gustavofonseca

Assim como comentado pelo slack, peço que seja dado prioridade para a aprovação do PR: https://github.com/scieloorg/search-journals-proc/pull/4.

Esse PR trata de inconsistências na interface, não asseguro que seja esse o motivo, porém a prioridade para esse PR irá evitar comentário e obvervações desnecessárias.

[jamilatta]

Video que mostra a validação com artigos em preprint:

Gravação de Tela 2020-04-28 às 14.25.06.mov.zip

[jamilatta]

@paratiuid conversando com a equipe ficamos, um pouco preocupados com a futura incorporação do branch interface-responsiva.

Vou explicar esse preocupação já mencionado pela @patymori no comentário: https://github.com/scieloorg/search-journals/pull/505#pullrequestreview-400235047

Por enquanto que o branch está seguindo isoladamente da master talvez não enxergamos esse problema, porém é necessário se atentar no momento de incorporação, pois, é possível que as alterações realizadas em javascript quebre outras funcionalidades, isso ocorre quando, por exemplo, é removido um id do HTML e temos um seletor(jquery) esperando por esse id, em fim, além de termos outras alterações sendo realizada pelo @rafaelpezzuto que também irá fundir com a master

Para minimizar isso é importante realizarmos antes do envio do PR uma fusão do master com o ramo interface-responsiva e realizar o envio do .min.js e .min.css desse fusão, além de teste de usabilidade.

Dessa forma garantirmos que não teremos quebra de código, ausência de código e ou inconsistências como mencionado no exemplo.