Versão 1.1b1 do collective.cover quebra o tile de carrossel em brasil.gov.tiles 1.1rc1 (essa versão do cover foi adicionado em 1.1.4)
idgserpro opened this issue · 7 comments
Isso acontece porque aqui em brasil.gov.tiles faz referência aos recursos do galleria em
++resource++collective.cover/galleria-theme/galleria.cover_theme.jsMas o caminho na versão nova do cover mudou no commit collective/collective.cover@86a9972, ficando
++resource++collective.cover/js/galleria.cover_theme.js
Precisamos mudar todas as referências no pacote a esse caminho antigo pro novo e estudar uma forma disso não precisar ser manual como é hoje (se possível, resolver collective/collective.cover#613 para que fique tudo como um recurso js para não ter que ficar fazendo essas chamadas manuais).
Além disso, esse tile não tem um teste de robot, por isso esse erro não conseguiu ser pego na infra estrutura de testes, portanto ele também precisa ser criado.
no mesmo collective.cover podem ver o upgrade step que fizemos para consertar isso.
Há impacto em outras partes desse pacote, por exemplo no vídeo, ver https://github.com/plonegovbr/brasil.gov.tiles/search?utf8=%E2%9C%93&q=galleria
@hvelarde @rodfersou Prezados, seria o caso de removermos
https://github.com/plonegovbr/brasil.gov.tiles/blob/baa8547302291a9d4de19ead8eb8d9454f853cb5/src/brasil/gov/tiles/static/galleria.classic.css
e
uma vez que temos os mesmos em
e
e collective.js.galleria é dependência de collective.cover?
@idgserpro fiquem a vontade de consertar qualquer coisa que precisar conserto.
@hvelarde mesmo corrigindo o caminho do js, a tile de carrossel do brasil.gov.tiles não funciona porque o js ++resource++collective.cover/js/galleria.cover_theme.js do collective.cover, mudou bastante e já não funciona mais com o código do brasil.gov.tiles.
Estou querendo remover a customização do carrossel do brasil.gov.tiles e passar a utilizar carrossel do collective.cover . No entando, vejo a intenção de remover o carrossel do cover:
collective/collective.cover#485
Qual a sua opinião sobre esse assunto?
@idgserpro remover o tile de carrossel não é uma coisa trivial e não temos pressa por enquanto.
tal vez a melhor solução seja rescrever o tile do brasil.gov.tiles e utilizar o Cycle2 nele.
@idgserpro pode ver como foi feito na visão de albuns que usa cycle2 https://github.com/plonegovbr/brasil.gov.portal/tree/master/src/brasil/gov/portal/browser/album
@hvelarde @rodfersou a ideia não era remover completamente a tile de carrossel mas sim substituir pela tile do cover, uma vez o brasil.gov.title sobrescreve a tile de carrossel do cover. Isso pouparia esforços, uma vez que a tile de carrossel já está implementada no cover. Assim poderiamos concentrar os esforços na evolução da tile do cover.