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

#280 - Feat/change language button layout update #294

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

HenriqueRicardoFigueira
Copy link
Contributor

@HenriqueRicardoFigueira HenriqueRicardoFigueira commented Jan 23, 2025

Descrição de PR

Screencast.from.2025-01-24.10-27-08.webm

Customiza layout do botão de mudar de idioma.

Issue relacionado

Motivações

Melhorar o layout/usuabilidade do botão de mudar de idioma.

Informações adicionais

Copy link

netlify bot commented Jan 23, 2025

Deploy Preview for diciotech ready!

Name Link
🔨 Latest commit 270a5cc
🔍 Latest deploy log https://app.netlify.com/sites/diciotech/deploys/67991eb105d5dd00083baac5
😎 Deploy Preview https://deploy-preview-294--diciotech.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@george-gca
Copy link
Contributor

2 comentários:

  • ficou um ponto perdido ali onde antes tava o seletor
  • tem como fazer os dois botões ficarem do mesmo tamanho? Ficou meio estranho os 2 diferentes

@HenriqueRicardoFigueira
Copy link
Contributor Author

2 comentários:

  • ficou um ponto perdido ali onde antes tava o seletor
  • tem como fazer os dois botões ficarem do mesmo tamanho? Ficou meio estranho os 2 diferentes

Realmente ficou estranho kk
Ajustei já @george-gca

@george-gca
Copy link
Contributor

Acho que dá pra diminuir um pouquinho mais as margens laterais dos 2 botões, pra eles ficarem um pouco mais quadrados. Mas por mim já passa, finalmente esse botão vai tá condizente com o resto do site haha.

@HenriqueRicardoFigueira
Copy link
Contributor Author

HenriqueRicardoFigueira commented Jan 24, 2025

Acho que dá pra diminuir um pouquinho mais as margens laterais dos 2 botões, pra eles ficarem um pouco mais quadrados. Mas por mim já passa, finalmente esse botão vai tá condizente com o resto do site haha.

Diminui o tamanho dos botões da uma olhada @george-gca
Screencast from 2025-01-24 15-57-02.webm
Screencast from 2025-01-24 15-58-54.webm

@HenriqueRicardoFigueira
Copy link
Contributor Author

Acho que dá pra diminuir um pouquinho mais as margens laterais dos 2 botões, pra eles ficarem um pouco mais quadrados. Mas por mim já passa, finalmente esse botão vai tá condizente com o resto do site haha.

Realmente tava precisando mexer nesses caras.

@george-gca
Copy link
Contributor

Agora tem um _ embaixo da língua que não tinha antes, e os botões diminuíram mas o conteúdo deles também. Era pra ter diminuído só o botão, mantido a fonte. Tenta fazer os 2 botões ficarem no tamanho que tá o botão de mudar tema do site, ou próximo, mas mantendo o tamanho do conteúdo.

@HenriqueRicardoFigueira HenriqueRicardoFigueira force-pushed the feat/change-language-button-layout-update branch from 48269c0 to 270a5cc Compare January 28, 2025 18:15
@HenriqueRicardoFigueira
Copy link
Contributor Author

Agora tem um _ embaixo da língua que não tinha antes, e os botões diminuíram mas o conteúdo deles também. Era pra ter diminuído só o botão, mantido a fonte. Tenta fazer os 2 botões ficarem no tamanho que tá o botão de mudar tema do site, ou próximo, mas mantendo o tamanho do conteúdo.

Fala @george-gca ajustei novamente, pode dar uma olhada ?

Screencast from 2025-01-28 15-19-40.webm
Screencast from 2025-01-28 15-16-08.webm

@george-gca
Copy link
Contributor

Pode fazer pra quando tiver 3 ou + línguas a largura do dropdown ficar menor, de preferência com o mesmo espaço que tá de margem esquerda pra direita? Que ele tá chegando e até passando do botão de mudar o tema quando abre.

Outra coisa, quando passa o mouse em cima de uma opção no dropdown ele tá dando um highlight com borda quadrada, mas sem o highlight tá borda arredondada. Acho melhor padronizar, de preferência pra borda arredondada.

@HenriqueRicardoFigueira
Copy link
Contributor Author

Pode fazer pra quando tiver 3 ou + línguas a largura do dropdown ficar menor, de preferência com o mesmo espaço que tá de margem esquerda pra direita? Que ele tá chegando e até passando do botão de mudar o tema quando abre.

Outra coisa, quando passa o mouse em cima de uma opção no dropdown ele tá dando um highlight com borda quadrada, mas sem o highlight tá borda arredondada. Acho melhor padronizar, de preferência pra borda arredondada.

beleza

@levxyca levxyca self-requested a review January 29, 2025 12:05
@levxyca levxyca added design Improvements or addtions to design accessibility accessibility-related improvements or additions labels Jan 29, 2025
@levxyca levxyca added this to the v2 milestone Jan 29, 2025
@levxyca levxyca linked an issue Jan 29, 2025 that may be closed by this pull request
Copy link

netlify bot commented Jan 29, 2025

Deploy Preview for diciotech ready!

Name Link
🔨 Latest commit e276782
🔍 Latest deploy log https://app.netlify.com/sites/diciotech/deploys/679cd324a659490009985963
😎 Deploy Preview https://deploy-preview-294--diciotech.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Owner

@levxyca levxyca left a comment

Choose a reason for hiding this comment

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

Obrigada pela contribuição 🚀 @george-gca tem mais algum ponto aqui? se estiver ok já dou merge 🤩

@HenriqueRicardoFigueira HenriqueRicardoFigueira force-pushed the feat/change-language-button-layout-update branch from c5dc7af to 39ed4a7 Compare January 30, 2025 12:50
@george-gca
Copy link
Contributor

Eu não vou conseguir testar com mais um idioma agora. @HenriqueRicardoFigueira tem como gerar aqueles gifs de como tá agora?

@HenriqueRicardoFigueira
Copy link
Contributor Author

Eu não vou conseguir testar com mais um idioma agora. @HenriqueRicardoFigueira tem como gerar aqueles gifs de como tá agora?

Opa, claro @george-gca segue:
Screencast from 2025-01-30 20-10-58.webm

@george-gca
Copy link
Contributor

Cara, ainda tem umas quinas ficando em outra cor quando passa o mouse por cima da língua no dropdown. Consegue resolver isso, pra ele todo ficar com highlight?

@HenriqueRicardoFigueira
Copy link
Contributor Author

Cara, ainda tem umas quinas ficando em outra cor quando passa o mouse por cima da língua no dropdown. Consegue resolver isso, pra ele todo ficar com highlight?

@george-gca agora eu acho que agora ficou bom, eu realmente não tinha notado as quinas, foi mal 🥲
Screencast from 2025-01-31 10-42-10.webm

@george-gca
Copy link
Contributor

george-gca commented Feb 1, 2025

Por mim agora tá beleza @levxyca. Sugestão: aceita essa 1o, depois a #296, e depois a #292. Aí acho que o código vai ficar organizado, com o botão como precisa, e com todas as actions funcionando como deveriam. Provavelmente vai dar erro de merge na última, aí eu resolvo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility accessibility-related improvements or additions design Improvements or addtions to design
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

Estilizar o botão de mudar de idioma
3 participants