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

Criei um simples ZOOM nos avatars. #49

Merged
merged 5 commits into from
Jun 24, 2021
Merged

Criei um simples ZOOM nos avatars. #49

merged 5 commits into from
Jun 24, 2021

Conversation

vitoropereira
Copy link
Contributor

Criei em CSS um simples zoom para que ao passar o mouse em cima podermos ver melhor as imagens.

@vercel
Copy link

vercel bot commented Jun 24, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/filipedeschamps/tabnews/DhCmhgUWse3smgvMzvPj5YY6akxj
✅ Preview: https://tabnews-git-vop1234-filipedeschamps.vercel.app

@filipedeschamps
Copy link
Owner

hahah que sensacionaaaal 😍 deu um charme muito maior, gostei bastante 👍 🤝

Só notei que você adicionou sem querer umas linhas a mais logo antes do fechamento da tag style. Topa mandar um commit removendo? Que depois disso esse PR vai ser feito merge na certa 😍

Copy link
Contributor Author

@vitoropereira vitoropereira left a comment

Choose a reason for hiding this comment

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

Ajustes nas linhas. Era isso?

@MichelSilvas
Copy link
Contributor

MichelSilvas commented Jun 24, 2021

Criei em CSS um simples zoom para que ao passar o mouse em cima podermos ver melhor as imagens.

Eu fiz um teste aqui com transition: 600ms; na hora que ocorre o zoom do avatar.
Achei que ficou legal. Até com um tempo menor.
O que acham de adicionar isso?

@filipedeschamps
Copy link
Owner

Ajustes nas linhas. Era isso?

@vop1234 isso!

Eu fiz um teste aqui com transition: 600ms; na hora que ocorre o zoom do avatar.
Achei que ficou legal. Até com um tempo menor.
O que acham de adicionar isso?

@MichelSilvas não se acanhe, mande um PR em paralelo a esse :) e sugiro usar algum easing na animação, pois geralmente fica mais natural do que ter uma velocidade linear 👍

Adicionando uma pequena animação no zoom
@rodrigoKulb
Copy link
Contributor

rodrigoKulb commented Jun 24, 2021

@vop1234 achei animal, @filipedeschamps já adicionei a animação antes de você falar hahaha!!!

@MichelSilvas ficou bem natural mesmo!

https://tabnews-p38fwjdx5-filipedeschamps.vercel.app/pocs/construction-02

@filipedeschamps como fiz antes de ler os comentários se quiser coloco em uma PR paralela.

@geovani-brito
Copy link
Contributor

Que ótima ideia! Sempre que vou procurar a minha foto é quase um jogo de esconde-esconde. rsrs Acho que isso ajuda bastante.

@rodrigoKulb
Copy link
Contributor

Olha que legal quando a foto é um PNG:
Captura de tela em 2021-06-24 15-43-12

@filipedeschamps filipedeschamps merged commit afec369 into main Jun 24, 2021
@filipedeschamps filipedeschamps deleted the vop1234 branch June 24, 2021 18:45
@filipedeschamps
Copy link
Owner

ahhahahahah vocês são sensacionaaaaaais!!!! 😍 😍 😍 😍

E agora a pessoa quando for tirar o seu print pode fazer com sua imagem maximizada!!

Merged totaaaal!! Let's goooooo!!!!

@cassioliveira
Copy link

Ah cara, que legal essa questão aí do PNG no zoom mas me diz, como essas imagens foram capturadas? Eu não tive tempo de procurar ainda. Todas as imagens de perfil do github, se colocar um .png no final não baixa nesse formato? Seria legal ter esse zoom aí mas na minha foto, por exemplo, não vai.

@rodrigoKulb
Copy link
Contributor

Olá @cassioliveira já está no ar com zoom na sua foto, conforme print:
Captura de tela em 2021-06-25 17-18-27
Acesse: https://www.tabnews.com.br/

@cassioliveira
Copy link

Olá @cassioliveira já está no ar com zoom na sua foto, conforme print:
Captura de tela em 2021-06-25 17-18-27
Acesse: https://www.tabnews.com.br/

Ah show de bola! Eu não vi ainda o vídeo de hoje e não sabia que já estava no ar. Obrigado!!

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.

None yet

6 participants