Skip to content

feat: adiciona menu de outras cidades#50

Open
brenorb wants to merge 2 commits into
curitibabitdevs:masterfrom
brenorb:feat/curitiba-other-cities
Open

feat: adiciona menu de outras cidades#50
brenorb wants to merge 2 commits into
curitibabitdevs:masterfrom
brenorb:feat/curitiba-other-cities

Conversation

@brenorb
Copy link
Copy Markdown

@brenorb brenorb commented May 17, 2026

Resumo

  • adiciona a aba Outras Cidades no menu principal
  • cria/atualiza a página /cities com links para os outros BitDevs do Brasil
  • mantém um link separado para a lista global do BitDevs NYC

Validação

  • git diff --check

Closes #49

@GustavoStingelin GustavoStingelin requested a review from Copilot May 18, 2026 20:51
@GustavoStingelin
Copy link
Copy Markdown
Member

Looking good

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

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.md com 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.

Comment thread cities.md
@GustavoStingelin
Copy link
Copy Markdown
Member

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:

image

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.

image

The logo and title are also wrapping, which is even more noticeable. In the original version, everything stays aligned on a single line:

image

Copy link
Copy Markdown
Member

@GustavoStingelin GustavoStingelin left a comment

Choose a reason for hiding this comment

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

Commented above.

@brenorb
Copy link
Copy Markdown
Author

brenorb commented May 21, 2026

@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).

@jaonoctus
Copy link
Copy Markdown

jaonoctus commented May 21, 2026

#50 (comment)

Acabou acontecendo isso no site de São Paulo. Quando tiver um fix, replica lá pf @brenorb

@GustavoStingelin
Copy link
Copy Markdown
Member

@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.

@brenorb
Copy link
Copy Markdown
Author

brenorb commented May 21, 2026

Gepeto que faz todo o trabalho pesado aqui.
Desconsiderem a sujeira de encoding. Assim tá aceitável? Opiniões?

Desktop:
image

Mobile:
image

@GustavoStingelin
Copy link
Copy Markdown
Member

É. Talvez pro de bsb, aqui no de Curitiba ainda tem o "sobre" no menu que deve piorar, tinha que ver individual por site talvez

@brenorb
Copy link
Copy Markdown
Author

brenorb commented May 21, 2026

@GustavoStingelin se o conceito geral agrada, aí é só ajustar. Mando um print com o resultado para vc ver.

@jaonoctus está suficientemente do seu agrado?

@GustavoStingelin
Copy link
Copy Markdown
Member

TBH, desse jeito me agrada mais.
#51

image

@brenorb
Copy link
Copy Markdown
Author

brenorb commented May 22, 2026

LGTM
#51 uma boa ideia

Vou até ver se funciona pro de Brasília.

@MiguelMedeiros
Copy link
Copy Markdown
Collaborator

PF não quebrem meu layout, obrigado.

@brenorb
Copy link
Copy Markdown
Author

brenorb commented May 22, 2026

@MiguelMedeiros dê sua opinião então nas imagens.

@MiguelMedeiros
Copy link
Copy Markdown
Collaborator

@MiguelMedeiros dê sua opinião então nas imagens.

image

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.

Adicionar menu "Outras Cidades" com links para outros BitDevs do Brasil

5 participants