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

Implementa página de Recentes #625

Merged
merged 2 commits into from
Aug 7, 2022
Merged

Implementa página de Recentes #625

merged 2 commits into from
Aug 7, 2022

Conversation

filipedeschamps
Copy link
Owner

Este PR implementa a página de /recentes que é uma cópia da Home, mas que utiliza a estratégia de new para listar as publicações mais recentes (independente das tabcoins). O link para acessar esta página foi incluída no menu superior.

Em paralelo, fiz um ajuste no DefaultLayout para sempre incluir o prefixo · TabNews caso o título da página não seja o título padrão usado na Home. Fiz isso, pois as únicas páginas que estavam com o prefixo · TabNews eram as que abriam algum content. Agora, todas páginas possuem esse prefixo, menos a Home.

@vercel
Copy link

vercel bot commented Aug 7, 2022

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

Name Status Preview Updated
tabnews ✅ Ready (Inspect) Visit Preview Aug 7, 2022 at 6:05PM (UTC)

@coffeeispower
Copy link
Contributor

coffeeispower commented Aug 7, 2022

Mas eu tambem questionaria ter uma rota separada só para isso. Porque da para passar uma query para a home e mudar a ordem.

@gabrielsozinho
Copy link
Contributor

gabrielsozinho commented Aug 7, 2022

Em celulares com maior zoom isso pode ficar cortado:
download-1

imaginei uma alteração, só um conceito, não que deverá ficar exatamente assim, quero sua avaliação:
IMG_20220807_122958

ou assim:
IMG_20220807_123137

existe alguma forma que não altere a proposta inicial, mas mesmo assim resolva esse problema? Por exemplo, tirar o "TabNews" e deixar só a logo na versão mobile?

@filipedeschamps
Copy link
Owner Author

Mas eu tambem questionaria ter uma rota separada só para isso. Porque da para passar uma query para a home e mudar a ordem.

@coffee-is-power show! Você diz uma query string? Acredito que dá para resolver tudo com query strings, mas não sei se a URL fica limpa. Por exemplo, daria também para fazer a página do usuário com querystring, onde ao invés disso:

https://www.tabnews.com.br/coffeeispower

Ficaria:

https://www.tabnews.com.br/?username=coffeeispower

Eu não sei, fica estranho. Fora que infelizmente o Next.js não gera novas páginas estáticas para query strings. Cheguei a conversar com o suporte e eles falaram que o framework se comporta assim de forma intencional.

Mas independente disso, sugiro sempre fazer URLs públicas terem informações no path o máximo que pudermos.

@filipedeschamps
Copy link
Owner Author

Em celulares com maior zoom isso pode ficar cortado:

Sensacional @gabrielsozinho !! Obrigado por esse ponto e por todas as ideias! Uma sugestão se possível seria desenhar elas usando os componentes do React Primer.

De qualquer forma, no Draft da Milestone 5 tem um item chamado Reorganizar menu e considerar responsividade em Mobile onde a idéia por trás disso é justamente atacar o ponto que você atacou ali que é pensar o que fazer com esse menu, principalmente em Mobile.

Então para esse PR eu vou fazer o seguinte: em Mobile eu vou esconder o texto "TabNews" para deixar mais espaço. E daí quando a issue de refatorar o Menu vier, vemos uma forma final de como lidar com isso 🤝

@filipedeschamps
Copy link
Owner Author

Feito! Bora agora pensar o que fazer com o menu por completo 🤝

RPReplay_Final1659895954.MP4

@filipedeschamps filipedeschamps merged commit 6ac24df into main Aug 7, 2022
@filipedeschamps filipedeschamps deleted the recents branch August 7, 2022 18:14
@filipedeschamps
Copy link
Owner Author

Merged! Let's gooooo!!

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.

3 participants