Skip to content
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

Scrolling a very big image list is laggy #141

Closed
ngyikp opened this issue Mar 26, 2016 · 4 comments
Closed

Scrolling a very big image list is laggy #141

ngyikp opened this issue Mar 26, 2016 · 4 comments

Comments

@ngyikp
Copy link

ngyikp commented Mar 26, 2016

I optimised an assets folder with about ~3600 images, and scrolling the list is very laggy even after the optimisation is done.

Video: https://dl.dropboxusercontent.com/u/1286251/images/imageoptim%20lag.mp4

ImageOptim version 1.6.1
Mac OS 10.11.4

@kornelski
Copy link
Member

Yes, I can confirm this. There's some awful performance regression in OS X that makes drawing of images in the list incredibly slow. System spends 99% of time in the argb32_image_mark_argb32 function, burning cycles.

I'm not sure what to do about it from ImageOptim's perspective, because I'm only using very high-level concepts like imageNamed and image table cell.

@MiroHofo
Copy link

Could something be done about this?
UI is very very laggy, it is pain to use it, sort it, scroll. There has to be a way how to optimize it, please :)

@MiroHofo
Copy link

Hey guys!
I have found what is causing issue and got a fix. It was all caused by opacity at loupe button, so whan I removed it, scrolling, sorting, selecting items is fast as lightning. Modified code in RevealButtonCell.m (if switch can be removed completely):
float opacity = iMouseHoveredInInfoButton ? 1.0f : ([self isHighlighted] ? 1.0f : 1.0f); [image drawInRect:infoButtonRect fromRect:NSZeroRect operation:NSCompositeSourceOver fraction:opacity respectFlipped:YES hints:nil]; }
First time using Xcode and git and don't know how to submit this. @pornel could you please submit this fix and build new version?

RevealButtonCell.m.zip

@kornelski
Copy link
Member

Oh, brilliant! I wrongly assumed that image rendering was from the status image column, and didn't think of the loupe.

Hopefully fixed in 6ee2893

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants