It can be set to 100% and still have overflow: hidden. This is how the html wrapper works. It sets the body div to be 100% with overflow: hidden so there won't be any scrollbars.
I think you really need to review my change and let me get that in, because it is going to interact with whatever we come up with here. In order to let the app resize the canvas, I have to make the canvas change the size of the appenddiv and the canvas sets its sprite to 100% to fill the appendiv. On 2009-08-29, at 10:57EDT, Max Carlson wrote: > Because the appenddiv is used to measure the app size for percentage > width/height. For example it gets set to width: 100%; height: 100%... > > IE doesn't seem to need this extra layer - text links work fine with > the > current scheme. So I need to figure out one way that works in every > browser - nothing I've tried so far works. Worst case, I can > conditionalize with a quirk. > > P T Withington wrote: >> Why can't we just set overflow hidden on the appenddiv and take it >> off >> the canvas divs? I don't see the need to introduce another layer >> of divs. >> >> On 2009-08-28, at 21:30EDT, Max Carlson wrote: >> >>> Change 20090828-maxcarlson-V by [email protected] on 2009-08-28 >>> 18:10:43 PDT >>> in /Users/maxcarlson/openlaszlo/trunk-clean >>> for http://svn.openlaszlo.org/openlaszlo/trunk >>> >>> Summary: Fix link clickability >>> >>> 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: LzSprite - Create a separate container div with overflow: >>> hidden for canvas, click and context divs to prevent scrollbars. >>> Store a reference to the app container (LzSprite.__appContainer) to >>> measure the screen size. Remove overflow: hidden for lzcanvasdiv, >>> lzcanvasclickdiv and lzcanvascontextdiv. Don't set physical size >>> for >>> lzcanvasclickdiv and lzcanvascontextdiv. >>> >>> LzScreenKernel - Use LzSprite.__appContainer to measure screen size. >>> >>> Tests: Can inspect messages in the debugger again. Testcase from >>> LPP-8402 still passes. >>> >>> Files: >>> M WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js >>> M WEB-INF/lps/lfc/kernel/dhtml/LzScreenKernel.js >>> >>> Changeset: >>> http://svn.openlaszlo.org/openlaszlo/patches/20090828-maxcarlson-V.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
