Make colour ramp have quantile bins about pct-shiny HOT 9 CLOSED

Made some progress on this in dbd2810 , is this what you were after?

from pct-shiny.

Code looks good but it's not working - think my changes may have broken it. Sorry! Let me know when this up-and-running!

from pct-shiny.

Sure, I think it was when you reverted the colour ramp changes that it re-introduced some old code, easily done. I'm not totally sure I've implemented the quartiles how you wanted but now it works can you have a look and close this if it is?

from pct-shiny.

Looks good.

from pct-shiny.

@nikolai-b
Thanks for the changes in the legend. It looks really nice.
One thing I've noticed, however, that the y-axis doesn't correspond to the underlying values for the selected zone attribute (the range doesn't correspond to the range of the y-axis). Is this something intentionally introduced?

from pct-shiny.

@usr110 I don't think this is intended. Please try to fix if you can...

Look forward to catching up!

Robin

from pct-shiny.

@Robinlovelace I have fixed the errors, I believe - also the dependency with the inputs, for the legend. Could you please update the code on the server? I will then test it out.

from pct-shiny.

I think it is updated - please test:
http://geo8.webarch.net/pct-shiny/master/
Fantastic work Ali, many thanks.

from pct-shiny.

re #15 (comment), sorry that was my mistake. I didn't realize that using isolate in render plot would mean the value never got updated (maybe because as render plot itself is only called once but it is re-evaluated constantly - the Shiny way). Closing now as this seems done.

from pct-shiny.