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

Melhorar a carga da seção "Em destaque" #403

Open
hvelarde opened this issue Nov 7, 2017 · 4 comments
Open

Melhorar a carga da seção "Em destaque" #403

hvelarde opened this issue Nov 7, 2017 · 4 comments
Assignees

Comments

@hvelarde
Copy link
Member

hvelarde commented Nov 7, 2017

Aparentemente a mudança introduzida em plonegovbr/brasil.gov.temas@2e778c7 para corrigir a visualização da tela após login teve um impacto não esperado na seção "Em destaque" descrito em #401.

Isso foi corrigido em #402 mas de uma forma não optimizada, pois estamos carregando a página completa ao invés do #content só.

Precisamos avaliar se existe alguma forma de melhorar isso.

@hvelarde hvelarde added this to the 1.5 milestone Nov 7, 2017
@caduvieira
Copy link
Contributor

caduvieira commented Nov 9, 2017

@brunobbbs você tinha corrigido isso aqui no governoeletronico de outra forma?

Edit
Foi usado isso?
https://colab.interlegis.leg.br/archives/thread/plonegov-br/falha-no-css-ao-realizar-login#msg-159275

@hvelarde
Copy link
Member Author

hvelarde commented Nov 9, 2017

isso aqui está também relacionado com #333.

@hvelarde hvelarde removed this from the 1.5rc1 milestone Nov 13, 2017
@brunobbbs
Copy link
Member

brunobbbs commented Nov 14, 2017

Isso mesmo @caduvieira, usei a solução postada. Se remover o trecho de código abaixo do template login_form o css não quebra mais após login. Se não me engano esse campo é usado para quando queremos fazer login através de uma janela modal, coisa que não é usada no IDG. Utilizei essa solução em dois clientes sem efeitos colaterais.

<input type="hidden"
            name="ajax_load"
            value=""
            tal:attributes="value request/ajax_load|nothing" />

Acredito que seja uma boa ideia adicionar essa customização aqui, então podemos desfazer plonegovbr/brasil.gov.temas@2e778c7 e #402 e fechar o #333. Se não houverem objeções posso submeter um PR.

@brunobbbs brunobbbs mentioned this issue Nov 16, 2017
21 tasks
@brunobbbs
Copy link
Member

Feito,

Criei um PR no brasil.gov.temas readicionando a diretiva notheme if ajax_load e adicionei a customização no template de login para evitar a quebra.

plonegovbr/brasil.gov.temas#147

Também criei um PR nesse projeto revertendo o commit 9398317 que removia o ajax_load dos parâmetros. Esse só deve ser aceito após incorporadas as mudanças do PR acima.

#423

Agora é com vocês.

Abs.

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