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: Adiciona CarouselPlugin editável no CMS #136

Merged
merged 2 commits into from
Oct 29, 2023

Conversation

miguelzinh3
Copy link
Contributor

@miguelzinh3 miguelzinh3 commented Oct 19, 2023

Contexto

Refatoração do plugin de Carousel genérico para editável.

Melhorias futuras: Adicionar edição de parametros de quantidade de slides por página do Swiper, Adicionar edição de breakpoints do Swiper e edição de possibilidade de alinhamento e tamanho máximo das imagens/texto.

Link da Tarefa/Issue

Requisitos

  • Cria um novo plugin CarouselPlugin a partir de EleicaoCarouselPlugin
  • Correlaciona os itens de CarouselItem e CarouselPlugin na model
  • Adiciona template com image, text e default_alt_text

Screenshots

image

@miguelzinh3 miguelzinh3 self-assigned this Oct 19, 2023
@miguelzinh3 miguelzinh3 force-pushed the refactor/carousel-plugin branch from e65fb58 to c82d091 Compare October 19, 2023 22:05
@miguelzinh3 miguelzinh3 force-pushed the refactor/carousel-plugin branch 2 times, most recently from 719dc51 to 78a1cf8 Compare October 20, 2023 20:43
@miguelzinh3 miguelzinh3 marked this pull request as ready for review October 20, 2023 20:44
app/contrib/frontend/landpage/cms_plugins.py Outdated Show resolved Hide resolved
@miguelzinh3 miguelzinh3 force-pushed the refactor/carousel-plugin branch from 78a1cf8 to f331bb8 Compare October 29, 2023 19:22
@miguelzinh3 miguelzinh3 merged commit 9fad683 into main Oct 29, 2023
1 check passed
@miguelzinh3 miguelzinh3 deleted the refactor/carousel-plugin branch October 29, 2023 19:29
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