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

Feat: Adiciona espaçamento e icone questionador ao plugin Header com Img e Icone #184

Merged
merged 2 commits into from
Mar 8, 2024

Conversation

miguelzinh3
Copy link
Contributor

@miguelzinh3 miguelzinh3 commented Mar 6, 2024

Contexto

Adiciona estilos novos no icone de Header com Img e Icone.

Link da Tarefa/Issue

Screenshots

image

Como testar?

  • Adicione um plugin Header com Img e Icon na página
  • Adicione um Text como Titulo 1
  • Para definir o quão longe o título estará do icone, adicione uma linha acima de seu título ou adicione o espaçamento top ou top & bottom de 5

@miguelzinh3 miguelzinh3 self-assigned this Mar 6, 2024
@igr-santos
Copy link
Contributor

@miguelzinh3 eu acredito que o problema é outro, tem haver com os diferentes tamanhos de imagens utilizadas dentro do Header, você chegou a fazer um teste com diferentes tamanhos de imagem?

@miguelzinh3
Copy link
Contributor Author

miguelzinh3 commented Mar 7, 2024

@miguelzinh3 eu acredito que o problema é outro, tem haver com os diferentes tamanhos de imagens utilizadas dentro do Header, você chegou a fazer um teste com diferentes tamanhos de imagem?

sim! eu fiz testes com imagens pequenas na vertical e grandonas na horizontal e o problema persiste pois é no espaçamento abaixo do plugin de header com img e icon!

@miguelzinh3 miguelzinh3 force-pushed the feature/header-with-icon-refactor branch from c7add44 to ac18bb7 Compare March 7, 2024 19:34
@igr-santos igr-santos merged commit cf52869 into release/site-nossas Mar 8, 2024
1 check passed
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.

2 participants