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

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

Closed
idgserpro opened this issue Nov 18, 2015 · 8 comments

Comments

@idgserpro
Copy link
Member

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
Copy link
Member

Me parece que ele está no override

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

@idgserpro
Copy link
Member Author

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 idgserpro changed the title Tile embed existe no código, mas não está no configure.zcml. Tile embed existe no código, mas não está no configure.zcml e sim no overrides.zcml. Nov 18, 2015
@idgserpro
Copy link
Member Author

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
Copy link
Member

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

@rodfersou
Copy link
Member

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
Copy link
Member Author

"Contaminamos" demais esse relato, abrimos outro pra resolver essa questão do icon em #140

@hvelarde
Copy link
Member

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants