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

Fix: Cards e Share #191

Merged
merged 1 commit into from
Oct 11, 2024

Conversation

ramoncordini
Copy link
Collaborator

Fix: Remoção do atributo autoheight nos slides pois foi feita uma correção no ds que corrige a necessidade deste.
Fix: Inclusão do componente addtoany para elementos share.
Fix: Correção de layout para impedir quebra de layout ao abrir dropdown share na home do periódico no mobile.

O que esse PR faz?

Corrige a necessidade da função javascript para redimensionar os cards.
Insere componentes addtoany ao template share, possibilitando uma grande variedade de redes sociais para se compartilhar.

Onde a revisão poderia começar?

Carregue os arquivos e navegue no sistema. Todos os dropdowns share devem exibir os novos elementos do addtoany. Ver print.
Os cards serão redimensionados automaticamente após a atualização do design system.

Como este poderia ser testado manualmente?

Siga os passos descritos anteriormente.

Algum cenário de contexto que queira dar?

O ajuste referente aos cards depende da atualização do design system. Essa atualização remove o atributo height: 100$ do elemento .slick-slide. Com isso, os cards obtém a altura do maior elemento de forma padronizada e global.

Screenshots

Screen Shot 2024-10-09 at 12 12 24
Screen Shot 2024-10-09 at 12 11 27
Screen Shot 2024-10-09 at 12 11 02
Screen Shot 2024-10-09 at 12 10 46

Quais são tickets relevantes?

Referências

Indique as referências utilizadas para a elaboração do pull request.

…reção no ds que corrige a necessidade deste.

Fix: Inclusão do componente addtoany para elementos share.
Fix: Correção de layout para impedir quebra de layout ao abrir dropdown share na home do periódico no mobile.
@robertatakenaka robertatakenaka merged commit 75c5316 into scieloorg:master Oct 11, 2024
2 checks 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