That was the first thing I tried, and it didn't make the extra scrollbars go away when there were oversized divs. Also it seems like the container needs position: absolute as well as overflow: hidden and a physical size - which is problematic if folks want their app to flow in the page - it should by default. I think it's something to do with the fact that 100% x 100% included the overflow, so it didn't do any good.
I'll look at your change before I spend any more time on this one. It will have to wait until tomorrow or late tonight :(. 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 > -- Regards, Max Carlson OpenLaszlo.org _______________________________________________ Laszlo-reviews mailing list [email protected] http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews
