paratiuid commented 4 years ago

O que esse PR faz?

Após redimensionar a tela, ocorre uma verificação que identifica a nova largura da tela e classifica se é mobile ou tablet. São exibidos e ocultados elementos da interface de acordo com a largura identificada.

Onde a revisão poderia começar?

Na tela de resultados de busca. Tanto iniciando com a tela na largura de mobile quanto iniciando na largura de desktop.

Se for mobile:

exibe a barra de itens selecionados no topo da tela, se houver item selecionado
exibe os detalhes do item da lista, ao clicar sobre um item.

Se for diferente de mobile:

a barra de itens selecionados no topo da tela deve sumir.
os detalhes do item da busca devem sumir.

Como este poderia ser testado manualmente?

Faça uma busca. Na lista de resultados, interaja com o sistema. Redimensione a tela.

Algum cenário de contexto que queira dar?

O que esse PR corrige: Atualiza a identificação de tipo de dispositivo de acordo com a largura da tela ao redimensionar. Com isso é possível exibir e ocultar elementos da interface de acordo com o dispositivo identificado.

O que esse PR NÃO corrige: Bug ao selecionar itens.

Screenshots

--

Quais são tickets relevantes?

485

Referências

https://api.jquery.com/slideUp/

jamilatta commented 4 years ago

@paratiuid

Validando!

jamilatta commented 4 years ago

@paratiuid

Realizei a validação e funciona como o esperado.

Tenho uma ressalva que acredito que seja relevante para que isso não fique inviável no futuro.

A branch(ramo) master recebeu recentemente algumas alteração por conta do formato de exibição do resultado de pesquisa quando o registro está na coleção "preprint".

As alterações foram nos seguintes arquivos: https://github.com/scieloorg/search-journals/commit/f49391fb68cbc057250895f4a127c1befae28cd2

Sugiro que seja realizado uma atualização na branch interface-responsiva com esses ajustes para que uma branch(ramo) não fique tão distante da outra ao ponto de a fusão(merge) entre as duas seja inviável.

Podem contar com meu apoio nessa atividade, mas acho que para mantermos ramos(branch) distintos com desenvolvimentos em paralelo precisamos aprender a atualizar os ramos(branch).

Somente para ilustrar o que comento acima:

responsive_search

Esse não é o formato de apresentação que ajustamos para artigos em fase de preprint.

alexxxmendonca commented 4 years ago

Funcionou como o esperado.

scieloorg / search-journals

Verificação de largura da tela ao redimensionar. Permite exibir e ocultar elementos de acordo com o tamanho do dispositivo identificado. #496