Closed (fixed)
Project:
Sky
Version:
6.x-3.7
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
5 Aug 2009 at 22:47 UTC
Updated:
5 Sep 2009 at 19:50 UTC
Running sky on Drupal 6.13. Issue is scroll/resize speed. System is fresh install with minimal page content. Result:
Safari 4.01 Mac: very slow scroll/resize
Chrome 3.0.196.0 Mac: very slow scroll/resize
Webkit Nightly (r46809) Mac: very slow scroll/resize
Firefox 3.0.13 Mac: fast scroll/resize
Chrome Windows (not sure version): fast scroll/resize
On Safari/Chrome/Webkit Mac, high CPU usage during scroll/resize operations.
Comments
Comment #1
jacineThis has been brought to my attention, and I've experienced it too. I didn't realize it was an issue with the theme until someone brought it to my attention. The problem is - I have no idea what's causing it and can't figure out to fix.
Hopefully someone will help with this one.
Comment #2
agerson commentedI found a bug reported to webkit that may explain it. Perhaps Sky's full page background gradient is hitting this WebKit memory bug:
GeneratedImage::drawPattern() appears to be using more memory than necessary.
GeneratedImage::drawPattern() creates an ImageBuffer the size of a tile, then
creates an Image of the same size, which results in another copy of the data
being made. For large tiles (such as a gradient whose height is the height of
the body element), those are very large allocations. On the CG platforms, the
code can be re-worked to utilize CGPattern, with the generator being called
from the pattern callback and CG managing the memory allocation.
Experimentation (as part of investigating bug 24130) has shown that this
results in much faster painting and reduced memory use.
https://bugs.webkit.org/show_bug.cgi?id=24178
Comment #3
jacineWow! Really? Have you tried removing it?
Comment #4
avpadernoThat would explain why the problem is happening with Safari, and Chrome for Mac: both use WebKit.
Comment #5
agerson commentedI think I got it! Removing
from #wrapper in style.css (line 22) dramatically improved my vertical scroll speed on Safari.
Comment #6
aimutch commentedI would recommend pulling that from the code. It has a nice effect for the browsers that support it. But it sounds like the performance hit isn't worth the benefit.
Comment #7
jacineAdam!!! Thank you ;)
This was really bugging me and wasn't sure where to turn. I sooo appreciate you tracking this down and will get rid of it ASAP!
Comment #8
jacineComment #9
jacineWith some help from dvessel, we discovered that the size of the gradient is the problem. After changing the size to 1em instead of 5em, I don't even notice a hit anymore and we still get to keep the gradient :)
Thanks guys! I really appreciate.
This will be up as 6.x-3.8 in a few.