feat: adiciona menu de outras cidades#50
Conversation
|
Looking good |
There was a problem hiding this comment.
Pull request overview
Adiciona ao site do Curitiba BitDevs uma nova entrada de menu “Outras Cidades” e atualiza a página /cities para listar comunidades BitDevs de outras cidades do Brasil, mantendo também um link separado para a lista global (BitDevs NYC).
Changes:
- Adiciona o item de menu Outras Cidades apontando para
/cities. - Reescreve
cities.mdcom conteúdo em português e lista de BitDevs brasileiros. - Inclui seção com link para a lista global de cidades mantida pelo BitDevs NYC.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
cities.md |
Substitui o conteúdo anterior por uma página “Outras Cidades” com links para BitDevs do Brasil e link para a lista global. |
_data/settings.yml |
Inclui “Outras Cidades” no menu principal apontando para /cities. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Thanks for the contribution, @brenorb. I liked it overall, but one thing is a bit unclear to me. On the NYC page, we have this version with a short single word that fits on one line:
Here, we are using two words, and the text breaks into a new line. The result feels a bit off visually. I would suggest either using a single word or adjusting the layout so the text does not wrap.
The logo and title are also wrapping, which is even more noticeable. In the original version, everything stays aligned on a single line:
|
|
@GustavoStingelin Sim, eu coloquei "outras cidades" para não ficar confuso, mas posso testar "cidades" e ver se a linha não quebra. Como Curitiba é bem maior que NYC talvez ainda fique esse problema. Como prefere? Fique à vontade para mexer e sugerir. Não tenho paixões pela forma exata final, a ideia é mais ter uma forma fácil de descobrir outros Bitdevs do Brasil (frequentemente me perguntam "tem bitdevs Tb na minha cidade?" e isso tornaria a descoberta muito mais fácil). |
|
Acabou acontecendo isso no site de São Paulo. Quando tiver um fix, replica lá pf @brenorb |
|
@brenorb dizem que o claudião é bom com essas coisas de front-end hahaha Pra mim, não quebrando o visual anterior que o @MiguelMedeiros fez, tá ótimo e dou ACK. |
|
É. Talvez pro de bsb, aqui no de Curitiba ainda tem o "sobre" no menu que deve piorar, tinha que ver individual por site talvez |
|
@GustavoStingelin se o conceito geral agrada, aí é só ajustar. Mando um print com o resultado para vc ver. @jaonoctus está suficientemente do seu agrado? |
|
TBH, desse jeito me agrada mais.
|
|
LGTM Vou até ver se funciona pro de Brasília. |
|
PF não quebrem meu layout, obrigado. |
|
@MiguelMedeiros dê sua opinião então nas imagens. |
|







Resumo
Outras Cidadesno menu principal/citiescom links para os outros BitDevs do BrasilValidação
git diff --checkCloses #49