I think I have a fix as a part of my change... I need to test IE, and then I will send an update.
On 2009-08-29, at 11:15EDT, P T Withington wrote: > 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
