Approved! I tested with webtop and it looks good! test/contextmenu/myapp.lzx is missing from this changeset. Actually, since it's a testcase, you can check it in independently of the LFC fix...
Henry Minsky wrote: > Note: there's a regression I'm looking at (LPP-8421) which causes the > default canvas context menu not to appear if you click > right on the canvas. That's independent of this change. > > > Change 20090825-hqm-h by [email protected] on 2009-08-25 17:34:30 EDT > in /Users/hqm/openlaszlo/trunk-clean > for http://svn.openlaszlo.org/openlaszlo/trunk > > Summary: fix for flickering swfs when right click menu is opened > > New Features: > > Bugs Fixed: LPP-8405 > > Technical Reviewer: ptw > QA Reviewer: max > Doc Reviewer: (pending) > > Documentation: > > Release Notes: > > Details: > > + when a right click is handled, instead of hiding the canvas content > and click divs, > we set their z-order to negative values, to place them behind the > contextmenu click div. > > > Tests: > > + test case from bug report, does not flicker. > + test/contextmenu/lpp-6980.lzx , check that proper context menu shows > up, regardless of whether other > opaque views are layered on top, check that embedded swf in <html> tag > does not flicker when > context menu is opened > > Files: > M test/contextmenu/lpp-6980.lzx > M WEB-INF/lps/lfc/kernel/dhtml/LzMouseKernel.js > > > Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20090825-hqm-h.tar -- Regards, Max Carlson OpenLaszlo.org _______________________________________________ Laszlo-reviews mailing list [email protected] http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews
