@wedgeV
see http://dev.piwik.org/trac/ticket/4594#comment:10
this is pretty much a quick hack, but works fine for me. i don't have time to look into jqplot more.
@mattab
Cheers for the PR.. I will test it, but why did you change: libs/jqplot/jqplot-custom.min.js ? is it needed for this particular fix?
it's what the build script creates, I figured it's necessary to rebuild it.
feel free to ignore if that's not the case!
On May 7, 2014 8:35 PM, "Matthieu Aubry" notifications@github.com wrote:
Cheers for the PR.. I will test it, but why did you change:
libs/jqplot/jqplot-custom.min.js ? is it needed for this particular fix?—
Reply to this email directly or view it on GitHubhttps://github.com/piwik/piwik/pull/271#issuecomment-42501506
.
of course, silly me. Will test it now!
Does not work for me, using Chrome Emulate screen feature I still see the pie chart zoomed in.
I'll check again tomorrow, definitely worked on my retina mbp though.
On Wed, May 7, 2014 at 9:54 PM, Matthieu Aubry notifications@github.comwrote:
Does not work for me, using Chrome Emulate screen feature I still see the
pie chart zoomed in.—
Reply to this email directly or view it on GitHubhttps://github.com/piwik/piwik/pull/271#issuecomment-42505841
.
just checked again, works fine for me. see screenshot. anyone else with a retina that can test?
(just pushed a version that is rebased to the latest master)
oh, my bad, I didn't apply your patch properly.
beautiful, it works! Please keep the Pull requests coming :+1: