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

Adicionando Botão de Compartilhar #38

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

alex-custodio
Copy link
Member

Ainda não está funcional, mas é melhor ir incrementando aos poucos. Aceito sugestões de design pra colocar esse botão também.
Essa PR também corrige um espaçamento indevido da ilustração da menina em telas grandes, que observei enquanto usava os computadores do colégio.

image

Copy link
Collaborator

@danielfireman danielfireman left a comment

Choose a reason for hiding this comment

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

Não é legal colocar em produção um botão sem função. Você pode até colocá-lo, mas deve escondê-lo usando um parâmetro de URL, por exemplo. Essa é uma técnica bem documentada chamada feature flags.

Outra coisa importante é a localização do botão. Ela está assim no design? É estranho ver o botão de compartilhamento não seguir nenhum ícone comum para tal, além do que, ele compete de igual para igual com um dos usos principais do site que é navegação por município.

Daí você pode separar a correção do problema do botão e passar a correção primeiro.

@JezzDiego
Copy link
Collaborator

Uma dica para o botão de compartilhar é usar a biblioteca react-share. Ela é bem leve, não requer scripts externos e abre uma janela popup para o compartilhamento nas redes sociais, assim o usuário não sai da página na qual estava navegando.

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