Review Board 1.5.4

Issue 588 - LazyLoading

Updated 1 year, 5 months ago

David Pollak Reviewers
dchenbecker, dpp, timperrett, marius, charles, heiko, joni, atsuhiko, jorge, naftoli, kris, indrajit, alexb, dlouwers, mstarzyk, rmellgren, jhoffman, probinett, jmadsen, mhartmann, jstrachan, jgoday
None LiftWeb-archive
https://liftweb.assembla.com/spaces/liftweb/tickets/588-multiple--lift-lazy-load--in-the-same-page-does-not-work-as-expected#last_comment

dpp_issue_588

LazyLoading had a number of issues.  It would not support multiple lazy loads on a page, the execution of the lazy snippets was serial, the wrong render version was posted for the functions created.

I fixed some underlying issues with thread pooling (the thread pools grow and shrink and the thread pools used for ActorPing support multiple threads.

I created a generic method on LiftSession that will capture state for deferred processing.

I updated the lazy loading and parallel snippet code to use the deferred processing code.

I added an attribute to lazy loading to allow the embedding of a template and if the attribute isn't specified, the spinning loader image is used by default.

I created examples of both lazy loading and parallel snippets in example.
Smoke testing against example code.
Ship it!
Posted 1 year, 5 months ago (August 26th, 2010, 8:28 a.m.)
Looks great!