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

Reply via email to