Closed ltartari closed 11 years ago
@ltartari Tá feito!
Dá uma olhada na cor. Se você preferir outra, basta mudar em https://github.com/okfn-brasil/orcamento.inesc.org.br/blob/master/app/assets/javascripts/angular_app/directives/treemap.js.coffee#L3.
Depois disso, só fechar a issue. Ainda temos o problema de não conseguir voltar um nível, mas tá valendo... :smile:
Gostei! Só mudaria para verde pra refletir o box azul, visto que o valor apresentado é relativo ao orçamento autorizado. Podemos mudar a fonte, também?
On Jul 20, 2013, at 11:46, Vitor Baptista notifications@github.com wrote:
@ltartari Tá feito!
Dá uma olhada na cor. Se você preferir outra, basta mudar em https://github.com/okfn-brasil/orcamento.inesc.org.br/blob/master/app/assets/javascripts/angular_app/directives/treemap.js.coffee#L3.
Depois disso, só fechar a issue. Ainda temos o problema de não conseguir voltar um nível, mas tá valendo...
— Reply to this email directly or view it on GitHub.
Não podemos ter um botão voltar que gera novamente o gráfico (voltando assim para a representação inicial)?
On Jul 20, 2013, at 11:46, Vitor Baptista notifications@github.com wrote:
@ltartari Tá feito!
Dá uma olhada na cor. Se você preferir outra, basta mudar em https://github.com/okfn-brasil/orcamento.inesc.org.br/blob/master/app/assets/javascripts/angular_app/directives/treemap.js.coffee#L3.
Depois disso, só fechar a issue. Ainda temos o problema de não conseguir voltar um nível, mas tá valendo...
— Reply to this email directly or view it on GitHub.
Mudei a cor e consegui alterar a fonte. : )
Ficou bem melhor!
Acabei reduzindo o tamanho do treemap, pois estava ficando ruim para clicar os boxes pequenos da direita que ficavam escondidos pela barra de rolagem.
Quanto à navegação, dá pra voltar usando o seletor de ano no header, porém não é intuitivo. Se tiver como replicar essa funcionalidade em um botão que só aparece quando o iframe está mostrando um detalhamento (nível subfunção) ficaria perfeito.
Sim, preferi ele menor também.
Vou tentar adicionar o link. Não deve ser muito difícil...
@ltartari Feito! Adicionei o botão pra voltar, mas ele tá estranho ainda... coloquei ele sempre aparecendo, mas disabled/enabled. Se você achar melhor ele só aparecer quando pode ser usado, é só trocar o ng-disabled por ng-hide em https://github.com/okfn-brasil/orcamento.inesc.org.br/blob/master/app/assets/javascripts/angular_app/directives/treemap.js.coffee#L32.
Ou, melhor ainda, faz como com as abas: deixa ele com display: none por padrão no css, e coloca o ng-class="{visible: drilledDown}"
, assim evitamos dele piscar na tela quando a página está carregando.
Revisa lá. Tou dando como encerrado.
:+1:
Só fiz tirar o ng-disabled que não precisa mais, e limpo o div do treemap assim que o usuário clica em Voltar
. Não precisava, mas achava ruim o delay entre clicar em voltar e o treemap terminar de carregar.
Seria bom [se possível] remover a borda e a sombra que o canvas coloca quando executando o mouse over. A cor eu já alterei.
Ah, e a animação de zoom só faz sentido na primeira carga… depois ela não serve pra nada :P
Aí você já quer demais... :P
Mas concordo plenamente. Infelizmente, isso já seria um patch pro openspendingjs, que não deve rolar agora.
Vítor Baptista
Developer | http://vitorbaptista.com | LinkedInhttp://www.linkedin.com/in/vitorbaptista| @vitorbaptista http://twitter.com/vitorbaptista
The Open Knowledge Foundation http://okfn.org
Empowering through Open Knowledge
http://okfn.org/ | @okfn http://twitter.com/okfn | OKF on Facebookhttps://www.facebook.com/OKFNetwork | Blog http://blog.okfn.org/ | Newsletterhttp://okfn.org/about/newsletter/
Em 21 de julho de 2013 19:43, Leonardo Tartari notifications@github.comescreveu:
Ah, e a animação de zoom só faz sentido na primeira carga… depois ela não serve pra nada :P
— Reply to this email directly or view it on GitHubhttps://github.com/okfn-brasil/orcamento.inesc.org.br/issues/28#issuecomment-21318549 .
Ver Issue #643 no OpenSpending.