approved! Did you come up with any idea about how you want to enable/disable this feature?
On Thu, Nov 19, 2009 at 7:34 PM, Max Carlson <[email protected]> wrote: > Change 20091118-maxcarlson-s by maxcarl...@bank on 2009-11-18 10:55:10 PST > in /Users/maxcarlson/openlaszlo/trunk-clean > for http://svn.openlaszlo.org/openlaszlo/trunk > > Summary: UPDATED: Improve CSS sprites > > Bugs Fixed: LPP-8622 - DHTML: Combine all css sprites into a single large > image > > Technical Reviewer: hminsky > QA Reviewer: ptw > > Details: Updated to fix issue Henry pointed out when compiling an empty > canvas. Added warning when master sprite can't be loaded. > > LzSprite.js - When quirks.use_css_master_sprite is on, store a path to the > master sprite and preload a copy of it. setResource() looks for a > spriteoffset, and where available uses it as the y offset in the master > sprite. Set the CSS y offset in __setBGImage() and __setFrame(). > > ImageMontageMaker - Refactor to build horizontal or vertical image strips. > Allow spacing between images to be compacted. > > DHTMLWriter - Track all resources added to the app, then build a single > master CSS sprite image. > > Tests: Now, only stretched resources need to be loaded at runtime. This > cuts the number of HTTP requests in half in many cases, e.g. webtop went > from ~144 to ~72 requests with an empty cache. Image retrieval time is also > improved, and the DOM is simpler with fewer img objects. > > Files: > M WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js > M WEB-INF/lps/server/src/org/openlaszlo/media/ImageMontageMaker.java > M WEB-INF/lps/server/src/org/openlaszlo/utils/FileUtils.java > M WEB-INF/lps/server/src/org/openlaszlo/compiler/DHTMLWriter.java > > Changeset: > http://svn.openlaszlo.org/openlaszlo/patches/20091118-maxcarlson-s.tar > -- Henry Minsky Software Architect [email protected]
_______________________________________________ Laszlo-reviews mailing list [email protected] http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews
