This version of the change has the following behavior changes
1) if width arg is explicitly supplied, resize gets set to false so the text won't resize to the textwidth 2) multiline text defaults to parent width if no width is specified 3) empty text defaults to zero width (consistent with "resize=true" behavior) 4) empty input text default to 100 width (consistent with current behavior) I tweaked the regression test to account for these changes Change 20070607-hqm-4 by [EMAIL PROTECTED] on 2007-06-07 13:55:40 EDT in /cygdrive/c/users/hqm/openlaszlo/legals3 for http://svn.openlaszlo.org/openlaszlo/branches/legals Summary: fix/changes to lztext width calculation New Features: multiline text defaults to parent's width empty text defaults to zero width Bugs Fixed: LPP-2958 Technical Reviewer: max (pending) QA Reviewer: (pending) Doc Reviewer: (pending) Documentation: Release Notes: Details: Tests: test/lztest/lztest-textheight.lzx test/lztest/lztest-lpp-2958.lzx amazon lzpix examples/components/style_example.lzx Files: M test/lztest/lztest-textheight.lzx M WEB-INF/lps/lfc/kernel/swf/LzTextSprite.as M WEB-INF/lps/lfc/views/LzInputText.lzs M WEB-INF/lps/lfc/views/LzText.lzs Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20070607-hqm-4.tar On 6/5/07, Max Carlson <[EMAIL PROTECTED]> wrote:
This change causes some regressions in the following tests: http://localhost:8080/legals-clean/test/lztest/lztest-textheight.lzx?lzr=swf7&debug=true http://localhost:8080/legals-clean/test/lztest/lztest-textheight.lzx?lzr=dhtml&debug=true Also, the new test doesn't pass in DHTML: http://localhost:8080/legals-clean/test/lztest/lztest-lpp-2958.lzx?debug=true&lzr=dhtml Henry Minsky wrote: > Change 20070530-hqm-0 by [EMAIL PROTECTED] on 2007-05-30 09:23:10 EDT > in /cygdrive/c/users/hqm/openlaszlo/legals3 > for http://svn.openlaszlo.org/openlaszlo/branches/legals > > Summary: make sure swf sprite text-width matches LzView text-width > > New Features: > > Bugs Fixed: LPP-2958 > > Technical Reviewer: max (pending) > QA Reviewer: (pending) > Doc Reviewer: (pending) > > Documentation: > > Release Notes: > > Details: > > swf kernel textsprite was setting it's width in some cases without syncing > up it's > LzView owner. > > + I removed the default 100 width on LzText, but left it for LzInputText. > > > > > > Tests: > > test/lztest/lztest-lpp-2958.lzx > > Files: > M WEB-INF/lps/lfc/kernel/swf/LzTextSprite.as > M WEB-INF/lps/lfc/views/LzInputText.lzs > M WEB-INF/lps/lfc/views/LzText.lzs > A test/lztest/lztest-lpp-2958.lzx > > > > -- Regards, Max Carlson OpenLaszlo.org
-- Henry Minsky Software Architect [EMAIL PROTECTED]
