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

jacine’s picture

This 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.

agerson’s picture

I 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

jacine’s picture

Wow! Really? Have you tried removing it?

avpaderno’s picture

That would explain why the problem is happening with Safari, and Chrome for Mac: both use WebKit.

agerson’s picture

Status: Active » Needs review

I think I got it! Removing

-webkit-box-shadow: 1em 1em 5em rgba(0, 0, 0, 0.5);

from #wrapper in style.css (line 22) dramatically improved my vertical scroll speed on Safari.

aimutch’s picture

I 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.

jacine’s picture

Adam!!! 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!

jacine’s picture

Status: Needs review » Reviewed & tested by the community
jacine’s picture

Status: Reviewed & tested by the community » Fixed

With 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 :)

-webkit-box-shadow: 1em 1em 1em rgba(0, 0, 0, 0.5);

Thanks guys! I really appreciate.

This will be up as 6.x-3.8 in a few.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.