Yes, when a text element has a fixed width, it gets resize set to false - so I set overflow: hidden in this case.
__wpadding is back because the text size doesn't visually match the swf appearance - try a side-by-side with the testcase. The text measurement tests work out okay, so it should be fine - it's strictly a visual thing... P T Withington wrote: > I'm confused as to how this is working. You're turning off overflow > hidden, but when you have a text element with a set width, you want it > to clip the content, so, I guess you are turning hidden back on? But I > don't understand how the __wpadding came back into play, since it has > been off for a long time. I thought that it was covered by the padding > in the CSS style combined with the overflow setting? > > I guess I'm concerned that this change could affect a lot of other > delicately balanced things... > > On 2009-10-08, at 05:01, Max Carlson wrote: > >> Change 20091008-maxcarlson-o by [email protected] on 2009-10-08 >> 01:52:39 PDT >> in /Users/maxcarlson/openlaszlo/trunk-clean >> for http://svn.openlaszlo.org/openlaszlo/trunk >> >> Summary: UPDATED AGAIN: Turn off overflow: hidden for non-multiline >> DHTML text >> >> Bugs Fixed: LPP-8399 - Drop-shadow support for DHTML runtime, LPP-8538 >> - DHTML: text with set width appears different from swf >> >> Technical Reviewer: hminsky >> QA Reviewer: [email protected] >> >> Details: LzSprite - Remove overflow:hidden style defaults for text. >> Remove unneeded lztextcontainer_click style. >> >> LzTextSprite - Overflow: hidden should only be needed for multiline >> fields, fields with resize=false, or those with scrollEvents on. >> Explicitly turn off overflow: hidden when shadows are turned on. >> Correct sizing of widths to visually match Flash. >> >> Tests: test/dropshadow.lzx no longer clips in DHTML. test/lztest/ >> lztest-textheight.lzx runs as before. Testcase from LPP-8538 now >> looks consistent in swf and dhtml. >> >> Files: >> M WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js >> M WEB-INF/lps/lfc/kernel/dhtml/LzTextSprite.js >> >> Changeset: >> http://svn.openlaszlo.org/openlaszlo/patches/20091008-maxcarlson-o.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
