Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Responsividade e Acessibilidade] Truncar username longo na lista de conteúdos e mais #1573

Merged
merged 2 commits into from
Dec 18, 2023

Conversation

aprendendofelipe
Copy link
Collaborator

Aplica algumas melhorias de responsividade e acessibilidade.

Lista de conteúdos

Agora será truncado o username sempre que faltar espaço.

Usando display: grid, consegui eliminar a necessidade dos diversos overflow: hidden e do position: absolute no Tooltip, então acredito que chegamos em um resultado melhor, pois a solução anterior (#1562) parecia mais um bug do que o bug que ela corrigiu: 😅

Antes Depois
image image

Por estarmos truncando o username, adicionei um Tooltip:

image

Em resoluções com pouca largura (menores de 360px), achei melhor manter o scroll horizontal como ocorria anteriormente, pois ele já vai ser necessário por causa do Header. Exemplo com 160px:

Antes Depois
image image

Obs. O título continua quebrando a linha para facilitar a leitura sem precisar do scroll horizontal.

Header

Em telas de 360px de largura, para usuários com grandes saldos de TabCoins e/ou TabCash, ainda podia ser necessário um pequeno scroll horizontal para visualizar todo o Header, então reduzi alguns espaçamentos para telas menores do que o target 544px. O mais perceptível é o espaçamento entre os saldos:

Antes Depois
image image

Telas ainda menores

Dois problemas que só ocorriam em telas muito menores do que 360px, mas que achei que valia a pena melhorar:

Antes Depois
image image
image image

HTML semântico

Como ajustei o Tooltip do PastTime, já aproveitei para trocar a tag span pela time.

Copy link

vercel bot commented Dec 17, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
tabnews ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 18, 2023 11:00am

Copy link
Collaborator

@Rafatcb Rafatcb left a comment

Choose a reason for hiding this comment

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

Acho que o resultado ficou melhor! Bom, pelo menos a data não está mais sozinha em outra linha 😅

Aproveitando, acho que poderia ajustar o alinhamento do ícone de busca quando o usuário está deslogado:

Header em tela pequena

A primeira impressão que tive é que o ícone do tema estava errado, mas na verdade ele está alinhada com os textos. O ícone de busca está centralizado, mas me aprece que alinhado embaixo ficaria melhor.

E também poderia adicionar um ml nele, já que nas telas "super pequenas" ele está desproporcionalmente mais próximo de "Recentes" do que do ícone do tema.

pages/interface/components/Header/index.js Outdated Show resolved Hide resolved
@Rafatcb Rafatcb added front Envolve modificações no frontend acessibilidade Melhoria de acessibilidade responsividade Melhoria necessária para um determinado tamanho de tela labels Dec 17, 2023
@aprendendofelipe
Copy link
Collaborator Author

Acho que o resultado ficou melhor! Bom, pelo menos a data não está mais sozinha em outra linha 😅

Aproveitando, acho que poderia ajustar o alinhamento do ícone de busca quando o usuário está deslogado:

Header em tela pequena

A primeira impressão que tive é que o ícone do tema estava errado, mas na verdade ele está alinhada com os textos. O ícone de busca está centralizado, mas me aprece que alinhado embaixo ficaria melhor.

E também poderia adicionar um ml nele, já que nas telas "super pequenas" ele está desproporcionalmente mais próximo de "Recentes" do que do ícone do tema.

Boa! Como tem coisa pra arrumar no PR, já aproveito para ver se dá para melhorar isso 👍

@aprendendofelipe
Copy link
Collaborator Author

Ficou melhor?

  1. Produção
  2. PR v1
  3. PR v2

image

image

image

@Rafatcb
Copy link
Collaborator

Rafatcb commented Dec 17, 2023

Ficou melhor?

Sim, a opção 3 ("PR v2") me parece ótima!

@aprendendofelipe
Copy link
Collaborator Author

Como mexemos nos alinhamentos do Header, fui conferir o restante: logado, deslogado e os demais tamanhos de tela.

Na versão logado temos o ícone de novo conteúdo (PlusIcon) que também estava centralizado. Só que ele não ficou bom em qualquer outro alinhamento, então voltei ícone de busca para o alinhamento original e subi o ícone de alterar o tema.

Concordo que esses dois ficaram melhores um pouco pra baixo (como na PR v2), mas temos que alinhar também com os outros elementos, inclusive as coins.

O que acham da v3?

image

image

@Rafatcb
Copy link
Collaborator

Rafatcb commented Dec 17, 2023

Acho que tudo bem, agora está melhor do que antes do PR 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
acessibilidade Melhoria de acessibilidade front Envolve modificações no frontend responsividade Melhoria necessária para um determinado tamanho de tela
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants