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

fix: gap between number and item in ContentList #389

Merged
merged 1 commit into from
May 26, 2022

Conversation

omariosouto
Copy link
Contributor

@omariosouto omariosouto commented May 26, 2022

🤔 Por que você está abrindo esse Pull Request?

  • Eu estou resolvendo um problema visual do gap entre o numero e o titulo do conteúdo na ContentList que fica na home.

🧐 Descreva sua solução:

  • Eu aumentei o gap, mas acredito que poderíamos ter uma solução melhor para isso no futuro, em breve vamos ter mais posts e essa solução não vai escalar.
Before After
image image

@vercel
Copy link

vercel bot commented May 26, 2022

@omariosouto is attempting to deploy a commit to the TabNews Team on Vercel.

To accomplish this, @omariosouto needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

@omariosouto omariosouto changed the title feat: fix gap between number and item in ContentList fix: gap between number and item in ContentList May 26, 2022
@rodrigoKulb
Copy link
Contributor

Olha quem está por aqui @omariosouto 🤩️

@filipedeschamps
Copy link
Owner

Show!!! Três pontos:

  1. Tudo que é relacionado a código vamos deixar em inglês e tudo que é comunicação aqui pelo Github vamos deixar em português.
  2. Muito obrigado pelo PR, isso era algo que está me incomodando e obrigado pelos prints de antes e depois, isso mostra um carinho enorme com as contribuições no projeto.
  3. Mais pra frente precisamos encontrar de fato algo melhor para alinhar esses itens, e eu tinha sugerido em outra issue usar uma lista ordenada, mas pensando agora o motivo do porque eu fiz com o Box é para o "scan" das notícias sempre ter o começo no exato mesmo lugar (horizontalmente falando). Se for uma lista ordenada padrão, ao passar do número 9. para o 10. vai empurrar o começo da linha pra direita.

@filipedeschamps filipedeschamps merged commit f1d7373 into main May 26, 2022
@filipedeschamps filipedeschamps deleted the fix-gap-contentlist branch May 26, 2022 19:51
@filipedeschamps
Copy link
Owner

Merged! Let's gooooo!!!

@filipedeschamps
Copy link
Owner

Em paralelo, tem um outro comportamento que não entendo porque acontece (e que quebra o "scan", que é a feature mais importante da home)

image

@omariosouto
Copy link
Contributor Author

@filipedeschamps

  1. Vaaleu pelo toque, talvez da pra puxar um CONTRIBUTING.md e colocar esses padrões por lá \o já traduzi minha mensagem aqui e vou arrumar o outro PR também
  2. Eu que agradeço pelo projeto, tava olhando os posts antigos aqui e achei ultra massa hahaha
  3. Vou pesquisar algumas referencias e proponho alguma coisa, tem algum espaço que vc ta usando pra discutir novas ideias?
  4. Vou dar uma olhada no scan depois!

@filipedeschamps
Copy link
Owner

  1. Perfeito!
  2. Masa 😍
  3. Hoje estamos usando primariamente o Github e o próprio TabNews (virou aquelas linguagens de programação que chegam num estágio de se auto-programar, sabe? 😂 )
  4. O "scan" é um termo que as vozes na minha cabeça inventaram. Mas é algo que acredito ser muito importante proteger na home e é basicamente a propriedade de você entrar numa página e ela ter o formato de consumo mais previsível (e por isso mais rápido) de informações.

@filipedeschamps
Copy link
Owner

@vercel
Copy link

vercel bot commented Aug 10, 2022

An owner of the TabNews Team on Vercel declined @omariosouto's request to join.

In order for their commit to be deployed, @omariosouto must push and request access again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants