search

plonegovbr / brasil.gov.paginadestaque

Complemento ao Portal Padrao para criacao de microsites e campanhas
https://plone.org.br/gov/
1 stars 4 forks source link

Corrige Tema Cinza para funcionar com nova versao do brasil.gov.barra. #11

Closed shogunbr closed 7 years ago

shogunbr commented 7 years ago

Corrige Tema Cinza para funcionar com nova versao do brasil.gov.barra reportado em #8

idgserpro commented 7 years ago

@shogunbr sua branch está desatualizada. Favor fazer um rebase com o master.

shogunbr commented 7 years ago

@idgserpro foi

idgserpro commented 7 years ago

Além de src/brasil/gov/paginadestaque/themes/destaques-cinza/img/acesso-a-infornacao.png, src/brasil/gov/paginadestaque/themes/destaques-cinza/img/brasil.png também pode ser removido nesse mesmo commit/PR (ambos foram removidos no PR da adaptação do brasil.gov.temas para a nova barra > 1.2).

Além disso, fica um pequena margem (bem sutil) maior cinza entre a barra e o tema (clique nas imagens para que fique num tamanho maior no browser):

selecao_005_edited

Que não ocorre nos demais temas.

selecao_003_edited

Há outras "inconsistências" no layout com relação aos demais temas e outros arquivos que também podem ser removidos, mas acho que não são o escopo desse PR e portanto abrimos um outro issue https://github.com/plonegovbr/brasil.gov.paginadestaque/issues/12.

shogunbr commented 7 years ago

@idgserpro inicialmente eu achava que essa borda extra tbm não estava no escopo, pois já estava nas versões anteriores. Mas como esse PR corrige o estilo referente a barra do governo, não vejo problemas em ser tratatado aqui.

idgserpro commented 7 years ago

Seu PR está quase pronto! Agora falta um passo que parece "burocrático" mas necessário, chamado "upgradeStep". Como você alterou o css, é uma boa prática refazer os bundles que o Plone faz para que, ao aplicar essa nova versão num portal, funcione automaticamente. Isso é necessário porque sua correção funciona perfeitamente para portais criados do zero, mas aqueles já em produção continuarão com o erro.

Você precisa, basicamente:

Qualquer dúvida pode questionar por aqui mesmo.

shogunbr commented 7 years ago

@idgserpro o teste não passou, eu fiz algo errado?

idgserpro commented 7 years ago

Você já pode se familiarizar com o processo e ver o log da build que quebrou. Lá indica a linha que deu problema, de forma mais específica:

https://github.com/plonegovbr/brasil.gov.paginadestaque/blob/d6916ab3ed2738f07bb9a942ccad2efd3229f24c/src/brasil/gov/paginadestaque/tests/test_setup.py#L124

Você precisa alterar o 1001 para 1002 nesse teste, pois é o último profile que você adicionou. Aproveite e adicione testes no mesmo molde do upgrade anterior:

    def test_to1002_available(self):
        step = self.list_upgrades(u'1001', u'1002')
        self.assertEqual(len(step), 1)

    def test_to1002_execution(self):
        self.execute_upgrade(u'1001', u'1002')
shogunbr commented 7 years ago

Entendi. Mas só vou poder a voltar a mexer nisso semana que vem, pois só tenho montado o ambiente de desenvolvimento na minha máquina do trabalho.

idgserpro commented 7 years ago

Tudo bem. Qualquer dúvida estamos aí.

shogunbr commented 7 years ago

@idgserpro

idgserpro commented 7 years ago

@shogunbr Pra mim está ok. Demais questionamentos de layout ou melhoria faremos em https://github.com/plonegovbr/brasil.gov.paginadestaque/issues/12.

Não temos previsão para essa correção ser incorporada num novo release IDG, converse com o @hvelarde se necessitar de um release de brasil.gov.paginadestaque nesse interim: você precisará pinar esse pacote na sua infra-estrutura.

Se tem interesse em mexer em outros relatos de css no IDG fique à vontade :laughing: