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

refactor: adjust input media aesthetics details #599

Merged
merged 3 commits into from
Aug 29, 2024

Conversation

JpBurgarelli
Copy link
Contributor

@JpBurgarelli JpBurgarelli commented Aug 22, 2024

Closes #542

Feature

Antes de tudo, é importante destacar que, segundo o Ale, não há necessidade de seguir à risca a consistência do layout, pois, futuramente, esse componente passará por alterações. Foram realizadas modificações para que o campo de entrada de mídia tenha um tamanho padrão e consistente. Além disso, o "X", que é responsável por remover uma mídia já adicionada, foi ajustado, já que anteriormente ele apresentava um formato oval em vez de circular.

Visual evidences 🖼️

Estava:
https://github.com/user-attachments/assets/82cfce4b-455e-43b2-a774-9e371898297f

Ficou/esta:
https://github.com/user-attachments/assets/fd200a32-6aa1-47e5-bed2-e2ff53bd239c

Checklist
  • Issue linked
  • Build working correctly
  • Tests created

@JpBurgarelli JpBurgarelli self-assigned this Aug 22, 2024
Copy link
Contributor

@juliaam juliaam left a comment

Choose a reason for hiding this comment

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

o layout não está de acordo com o do figma
image

voce conseguiria talvez escurecer um pouco o hover de quando não tem nenhuma media selecionada? dá uma checada no layout pra verificar isso

@JpBurgarelli JpBurgarelli force-pushed the refactorInputMedia/#542 branch from e3a62be to 3a60fae Compare August 23, 2024 11:07
@aalmeida00 aalmeida00 merged commit 45ba097 into devhatt:master Aug 29, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Ajuste de Layout do MediaInput
4 participants