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
Google Chrome reloads dashboard twice #1561
Comments
In IE8 on Vista, I was getting a lot of console errors, "Object doesn't support this property or method". Disabling the PDFReports plugin seemed to fix this. |
This behaviour is somehow caused by the visitor country map plugin. Disabling it, fixes the problem. |
I can confirm that issue is caused with the Maps widget. Peter, good stuff. I'm curious, how did you find out the cause? |
This was not a big deal. I was able to reproduce the bug with the current trunk but not with the current install on a webserver. The only difference between these two installs was the World-Map plugin. The problem could be further tracked down to line 21 in worldmap.tpl
deleting it seems to fix the issue (please confirm). Further info to the browser detection of jquery: http://api.jquery.com/jQuery.browser/ Update: Now the JQuery.support-object should be used instead to determine if the browser supports w3c boxModel etc.: |
Attachment: Replaces the $.browser.chrome browser-detection hack with a short workaround. Please check, because I'm no JS hero! I tested in Chrome / IE8 / FF3 / Safari and the x-offset of the control-icons seemed ok. |
I had to introduce this line because the default jQuery browser detection makes no difference between Safari and Chrome but both browsers have different ways of calculating the width of a html select element. I don't think that this issue is connected to the box-model bug. There are serveral work-arounds for this:
|
The fix looks good, I'll commit this. Thanks a lot :) |
(In [2950]) added bug-fix from peterB (fixes #1561) |
Greg, I think there is also a Variable to be changed on line 71. |
Oh yes, I forgot to fix this because this is a very unnecessary line. The select elements width doesn't change as the browser window width changes. I'll remove it.. |
"{" and "}" are no valid strings to capsule js comments, please fix as it is in .diff file. Can be checked here: Update: |
I think these are Smarty comments which doesn't appear in the html output. |
When I go to http://demo.piwik.org using Google Chrome on windows XP, the dashboard first partly loads, then the dashboard visibly reloads itself. This is def not expected.
The text was updated successfully, but these errors were encountered: