Because when we make the canvas different we get bug reports like the one to make the canvas resizable?
We should strive to make it similar as possible. I understand making the div clip, I don't understand making the clickcontainer and the clickdiv clip. On Aug 22, 2009, at 15:07, Max Carlson <[email protected]> wrote: > Why should canvas divs use the same policy as their children? I see > them as a special case - we explicitly want the canvas to be > clipped/contained. I changed this because before, we had overflow: > hidden on the body, which won't work for oversized apps... > > It seems like less overall computation to keep non-canvas containers 0 > height/width and allow overflow. So, I'm in favor of keeping things > the > way they are. > > P T Withington wrote: >> I'd like to see the canvas be more in parallel with views. See >> >> http://jira.openlaszlo.org/jira/browse/LPP-8007 >> >> where Henry had to make the click containers _not_ overflow hidden >> (presumably because they have 0 size). You just commented on my >> change >> that I should _not_ have to size the click container, just the actual >> click div. >> >> So, we ought to have one policy for the canvas and the views, >> either the >> click (and context) containers have 0 size and allow overflow, or >> they >> have overflow:hidden but have to be resized. >> >> On 2009-08-21, at 21:12EDT, Max Carlson wrote: >> >>> Change 20090821-maxcarlson-Y by maxcarl...@bank on 2009-08-21 >>> 17:33:09 >>> PDT >>> in /Users/maxcarlson/openlaszlo/trunk-clean >>> for http://svn.openlaszlo.org/openlaszlo/trunk >>> >>> Summary: Eliminate scrollbars for oversized divs >>> >>> Bugs Fixed: LPP-8402 - DHTML: Extra scrollbar width on Safari and >>> windows firefox and IE 7 when wrapper has overflow: hidden removed >>> >>> Technical Reviewer: ptw >>> QA Reviewer: hminsky >>> >>> Details: html-response - Remove overflow: hidden from default CSS. >>> >>> LzSprite.js - Add overflow: hidden to lzcanvasdiv, lzcanvasclickdiv >>> and lzcanvascontextdiv CSS declarations. Ensure lzcanvasclickdiv >>> qnd >>> lzcanvascontextdiv have a width and height so overflow: hidden >>> doesn't >>> hide clickable areas. >>> >>> Tests: See LPP-8402, also try calendar with ?lzr=dhtml&lzt=html in >>> one >>> of the affected browsers. >>> >>> Files: >>> M WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js >>> M WEB-INF/lps/templates/html-response.xslt >>> >>> Changeset: >>> http://svn.openlaszlo.org/openlaszlo/patches/20090821-maxcarlson-Y.tar >>> _______________________________________________ >>> Laszlo-reviews mailing list >>> [email protected] >>> http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews >> > > -- > Regards, > Max Carlson > OpenLaszlo.org > _______________________________________________ > Laszlo-reviews mailing list > [email protected] > http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews _______________________________________________ Laszlo-reviews mailing list [email protected] http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews
