Hm. I'm not 100% copacetic with this. 1) Since there is a rumor that IE screws up the commas, and the commas are purposely optional in CSS because of this, I vote for leaving out the commas.
2) Because this is an IE-only bug, I'd like that inside a quirk, because of the following suspicion: We are trying as much as possible _not_ to put styles on individual divs if we can avoid that, on the theory that putting styles on individual divs has to slow the rendering speed of those divs. So, if possible, we put the most common style (if it is not already the CSS default) on a div class, and try to only override on a per-div basis for exceptional cases. In this case, clipping is the exception (it is only on for a sprite that represents a view with scrollevents enabled), and not clipping is the default. Once again, IE totally sucks. In every other browser, setting a CSS style property to the empty string removes that style property from the div and the default value prevails. Leave it to IE to get that wrong... On 2009-10-13, at 14:41, Henry Minsky wrote: > Change 20091013-hqm-p by [email protected] on 2009-10-13 14:38:37 EDT > in /Users/hqm/openlaszlo/trunk6 > for http://svn.openlaszlo.org/openlaszlo/trunk > > Summary: error in LzTextSprite setScrolling in IE7/DHTML > > New Features: > > Bugs Fixed: LPP-8415 > > Technical Reviewer: max > QA Reviewer: (pending) > Doc Reviewer: (pending) > > Documentation: > > Release Notes: > > Details: > > + In DHTML dbeug mode IE7 if you click to set the debugger to > multiline, you get an 'invalid argument' > error from Javascript, due to setting the clip.style to ''. > > Changed to rect(auto,auto,auto,auto), which is supposed to be the > default behavior. There was > some comment on Microsoft's site about how using commas wasn't > supported by IE7, but it seems to work > for me. > > > Tests: > > test case below should not get an error in IE7/DHTML > <canvas> > <view height="120" bgcolor="black" width="2" /> > <inputtext id="foo" width="400" x="6" > multiline="false" > text="${canvas.runtime}" > bgcolor="#ccffcc" > oninit="setAttribute('multiline', true); > this.setAttribute('height', 120)" /> > </canvas> > > > Files: > M WEB-INF/lps/lfc/kernel/dhtml/LzTextSprite.js > > > Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20091013-hqm-p.tar > _______________________________________________ > 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
