brdevstreamers / brdevstreamers-ui

https://brstreamers.dev
Apache License 2.0
60 stars 20 forks source link

New Streamer card #53

Closed caio-emidio closed 1 year ago

caio-emidio commented 2 years ago

I was thinking about it... in my point of view, the card needs some improvements.

Actual Card: image

What i am thinking about: image

If you want to change contribute:

Figma File https://www.figma.com/file/MOHsNERLK9l1fzNyezFHDK/Untitled?node-id=0%3A1

lucasvsouza28 commented 1 year ago

woking on it.

caio-emidio commented 1 year ago

@lucasvsouza28 se possivel... acho que seria interessante ao carregar mover a tela levemente pra cima. Se vc ver o preview la no brstreamers.dev vc vai ver que a tela fica cortada pelo border radius ... Consegue dar uma olhada?

lucasvsouza28 commented 1 year ago

Bom dia, agora que eu reparei, no figma tem o card com border-radius e sem, mas não tava claro que era pra tirar o border-radius no hover. Você acha que fica melhor assim?

lucasvsouza28 commented 1 year ago

@lucasvsouza28 se possivel... acho que seria interessante ao carregar mover a tela levemente pra cima. Se vc ver o preview la no brstreamers.dev vc vai ver que a tela fica cortada pelo border radius ... Consegue dar uma olhada?

Fiz algumas alterações no PR #69.