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

Barra local deveria ser simplesmente o barra.js presente em https://barra.brasil.gov.br/barra.js na pasta static do pacote #30

Closed
idgserpro opened this issue May 25, 2016 · 11 comments
Assignees
Labels
Milestone

Comments

@idgserpro
Copy link
Member

idgserpro commented May 25, 2016

Hoje fazemos uma "reimplementação" da barra, pegando partes do html que é gerado dinamicamente pelo https://barra.brasil.gov.br/barra.js e colocando nessa template.

Bem lá no início da implementação da barra, isso até fazia sentido, pois a barra era apenas uma div sendo colocada no topo: hoje, adiciona uma imagem svg em formato base64 de libras na barra, adiciona um rodapé (com uma imagem de governo em formato svg) e ainda manipula fontes. Ter uma barra local que é uma "reimplementação" de um js que já está pronto é retrabalho desnecessário.

Assim, propomos:

Atendendo a esse issue, podemos fechar o #24, não sendo necessário mais lógica de fallback da imagem de governo, e precisamos criar um novo issue pra remover essa imagem do tema.

@hvelarde
Copy link
Member

a reimplementação da barra permitiu desativar os elementos invasivos inseridos nos sites a traves dela.

um exemplo é o site do Portal Brasil 2016 onde os mosquitos da campanha contra a dengue e a Zika eram uma péssima ideai nos sites em inglês e espanhol.

@idgserpro
Copy link
Member Author

Permitiu desativar mas não achamos que esse é o papel da barra local atual. Esse recurso de "desativar elementos invasivos" foi um efeito colateral da implementação atual da barra mas deveria ser resolvido no policy do portal, com o uso de jbot. Do jeito como está continua com o problema de ser uma reimplementação da barra oficial.

Exemplo: agora temos o recurso de libras na barra. Como fazer pra incorporar na barra local? Entrar no repositório da barra, vasculhar as modificações em svg, css, e reimplementar no brasil.gov.barra? Entrar em https://barra.brasil.gov.br/barra.js, copiar e colar apenas as partes importantes? Esse trabalho manual não só é improdutivo como aumenta a chance de erros e bugs.

Se a barra, que é a oficial, está aplicando os elementos invasivos, a reclamação deve ser com os responsáveis pela barra. Eles devem ser avisados do problema que é uma campanha invasiva em sites multilíngues do governo e chegar num consenso. Não devemos ficar implementando, em pacotes oficiais do IDG, formas de "burlar" algo que foi definido oficialmente pelo Ministério do Planejamento, ainda mais quando significa uma reimplementação.

A motivação desse relato é justamente a necessidade de, novamente, precisarmos trocar o fallback da imagem do rodapé, colocando a mais atual: se a barra local fosse o espelho do mais atual, além de resolver esse problema eliminaria o retrabalho de ficar refazendo essa barra. Seguir pelo caminho proposto nesse relato, a atualização é basicamente um wget da barra e colocar no pacote na pasta static, sem todo esse trabalho manual.

@caduvieira @rodfersou @ericof p/c

@rodfersou
Copy link
Member

Acredito que seja bom mudar a configuração do pacote para estar preparado para compilar o arquivo .sass da barra ao invés de utilizar o .css gerado, assim teremos somente uma cópia do conteúdo original da barra.

@idgserpro
Copy link
Member Author

Se formos por esse caminho @rodfersou, continuamos com a crítica da reimplementação: teria de estudar a lógica do .sass do https://github.com/govbr/barra.govbr, teríamos de colocar uma lógica de compilação de .sass no nosso pacote, acompanhar cada modificação dos arquivos .sass no repositório da barra, ver no .js o que teríamos de usar ou não pra replicar em nossas templates...

... ou podemos simplesmente colocar o .js já compilado, pronto e testado, do jeito que tem de rodar, em nossa pasta static, e, na template onde testa if local, ao invés de renderizar nossa implementação atual chama o .js como um ++resource++.

Criamos um teste que compara o .js do pacote com o original, assim cada build feita em brasil.gov.barra vai avisar se tem a última versão ou não da barra em javascript.

Envolve menos trabalho, menos reimplementação, menor chance de bugs e maior garantia de proximidade com a barra original. Acreditamos que em virtude da complexidade que se tornou a barra esse é o melhor caminho.

@hvelarde
Copy link
Member

eu acabo concordando cm vocês, @idgserpro, no final este pacote deveria só implementar a integração da barra com o Plone.

@caduvieira seria bom ter opções para desabilitar qualquer elemento intrusivo que se decida adicionar na barra.

@idgserpro
Copy link
Member Author

@hvelarde sobre elementos intrusivos sendo desabilitados em barras multilíngues, acho que podemos também discutir em #26

@hvelarde
Copy link
Member

acho que não: são problemas diferentes e o melhor seria abrir uma issue separada.

@caduvieira
Copy link
Contributor

Sobre desabilitar elementos intrusivos na barra isso deve ser discutido com a SECOM (Agora não sei com quem exatamente) mas o @isaiascoelho pode ter uma resposta.

Já tinha sugerido sobre isso para o pessoal da SECOM mas a escolha foi NÃO ter essa opção pois, no caso do mosquito da dengue, era para 'incomodar' mesmo assim como o mosquito 'incomoda' na vida real.

E as barras em outras línguas podem ser preparadas para não ter elementos que eventualmente não cabem no contexto.

@caduvieira
Copy link
Contributor

E sobre o fallback da imagem do rodapé eu continuo com a sugestão de não ter imagem de fallback conforme #24 (comment)

@idgserpro
Copy link
Member Author

idgserpro commented May 30, 2016

Sim @caduvieira, ao atender esse issue, o #24 (comment) poderá ser fechado e a imagem de fallback removida (ver plonegovbr/brasil.gov.temas#130). O relato #31 também poderá ser fechado uma vez que o css customizado da barra será removido.

@idgserpro idgserpro added this to the 1.1.5 milestone Aug 9, 2016
@idgserpro idgserpro self-assigned this Aug 9, 2016
@idgserpro idgserpro added the ready label Sep 2, 2016
idgserpro added a commit that referenced this issue Oct 25, 2016
Atende #30

A idéia é sempre baixar o barra.js oficial se quiser usar uma barra
local, ao invés de reimplementá-la aqui nesse pacote como era feito no
passado.
hvelarde pushed a commit that referenced this issue Oct 27, 2016
Atende #30

A idéia é sempre baixar o barra.js oficial se quiser usar uma barra
local, ao invés de reimplementá-la aqui nesse pacote como era feito no
passado.
@idgserpro
Copy link
Member Author

Atendido em #37.

@idgserpro idgserpro added ready and removed ready labels Oct 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants