search

plonegovbr / brasil.gov.tiles

Blocos de Conteúdo (Tiles) para sites do Governo Brasileiro
https://plone.org.br/gov/
3 stars 12 forks source link

Tile embed existe no código, mas não está no configure.zcml e sim no overrides.zcml. #134

Closed idgserpro closed 6 years ago

idgserpro commented 8 years ago

Temos o código de implementação do tile embed, mas ele não está disponível no configure.zcml e sequer no registry.xml. Esse tile é referenciado no manual de gestão de conteúdo.

No registro de configuração de um novo portal IDG 1.0.5, temos os seguintes tiles registrados (veja que não temos o embed):

[u'agenda', u'audio', u'audiogallery', u'collective.cover.banner', u'collective.cover.carousel', u'collective.cover.collection', u'collective.cover.list', u'collective.cover.richtext', u'em_destaque', u'mediacarousel', u'nitf', u'social', u'standaloneheader', u'video', u'videogallery', u'banner_rotativo']

Qual foi a motivação desse overrides? @rodfersou alguma idéia? Aos demais, alguma idéia do porque ele não aparece registrado?

agnogueira commented 8 years ago

Me parece que ele está no override

Esse tile é padrão do cover, interessante descobrir porque é duplicado aqui.

idgserpro commented 8 years ago

Deve ter ocorrido algum desenvolvimento "em paralelo" na época, assim como algumas coisas do collective.nitf presentes em brasil.gov.portal. Vou mudar o título da issue e complementar com relação ao registro.

idgserpro commented 8 years ago

Por causa desse overrides, a nova versão a13 do collective.cover (que não saiu ainda) quebra os ícones dos que tem overrides, pois agora temos o prefixo /img para o ícone dos tiles mas o overrides.zcml não.

Versão atual:

selecao_130

Com o pre-release a13:

selecao_131

Para testar: pegue o último commit do brasil.gov.tiles e o último de collective.cover.

Novamente, @rodfersou: lembra da motivação de porque esse overrides.zcml existe? Isso é importante saber para determinarmos o direcionamento da correção. Se o overrides.zcml não é mais necessário, ele pode ser removido junto com as customizações e esse erro do ícone resolve; se ele é necessário, teremos de alterá-lo com essas novas referências quando for atualizar o collective.cover.

rodfersou commented 8 years ago

Os overrides existem para customizar tiles do collective.cover quando necessário.

Não me lembro da motivação para cada override, mas podemos procurar isso no histórico do github.

rodfersou commented 8 years ago

Algumas vezes não podiamos esperar por um release do collective.cover, então tivemos que alterar o tile no nosso pacote.

rodfersou commented 8 years ago

Sugiro que seja revisado caso a caso o que o collective.cover ja corrige para desfazer aos poucos essas customizações. Em alguns casos ainda abrir issues no collective.cover para corrigir diretamente la.

idgserpro commented 8 years ago

"Contaminamos" demais esse relato, abrimos outro pra resolver essa questão do icon em https://github.com/plonegovbr/brasil.gov.tiles/issues/140

hvelarde commented 6 years ago

o overrides do tile de embed foi removido no IDG v2 por tanto isso aqui não é mais válido (refs. #210).