<html>
<body>
Approved, but a couple of changes are needed.<br><br>
The setters should be marked private. You need to define setMaxlength and
setPattern methods that are marked deprecated. Look at setText() in
edittext for how it should look. Do not mark the method private like
setText is.<br><br>
<br><br>
<br>
<blockquote type=cite class=cite cite="">Change 20081023-laszlo-2 by
laszlo@T43-L3XEXMW on 2008-10-23 18:01:01 EDT<br>
in /home/laszlo/src/svn/openlaszlo/trunk-bigboss<br>
for
<a href="http://svn.openlaszlo.org/openlaszlo/trunk">
http://svn.openlaszlo.org/openlaszlo/trunk</a><br>
<br>
Summary: edittext maxlength and pattern attributes need formal
setters<br>
<br>
New Features:<br>
<br>
Bugs Fixed: LPP-7222 - edittext maxlength and pattern attributes need
formal setters<br>
<br>
Technical Reviewer: promanik<br>
QA Reviewer: henry<br>
Doc Reviewer: (pending)<br>
<br>
Documentation:<br>
<br>
Release Notes:<br>
<br>
Details: Implemented contributor Gregory Denson's<br>
suggested fix for the bug. Removed
setMaxlength<br>
and setPattern methods (since we're no longer<br>
using setFoo() syntax) and replaced them with<br>
setters for maxlength and pattern attributes.<br>
<br>
Tests: Run the following in DHTML and SWF:<br>
<br>
<canvas width="500" height="300"><br>
<simplelayout axis="y"/><br>
<edittext name="foo"
maxlength="50" width="300" height="200"
<br>
multiline="true"/><br>
<button
onclick="Debug.write(canvas.foo.maxlength);<br>
canvas.foo.setAttribute('maxlength', 100);
Debug.write(canvas.foo.maxlength)"/><br>
<button
onclick="Debug.write(canvas.foo.pattern);<br>
canvas.foo.setAttribute('pattern', '[a-z]*');
Debug.write(canvas.foo.pattern)"/><br>
</canvas><br>
<br>
Type into the field until you reach the limit.<br>
Click the first button, and note the changed values in the
debugger. Type into the field some more. It behaves as
expected. Click the second button. Type into the field.
Try using numbers and upper-case letters. It should behave as
expected.<br>
<br>
Files:<br>
M lps/components/lz/edittext.lzx<br>
<br>
Changeset: <br>
<a href="http://svn.openlaszlo.org/openlaszlo/patches/20081023-laszlo-2.tar">
http://svn.openlaszlo.org/openlaszlo/patches/20081023-laszlo-2.tar</a>
</blockquote></body>
</html>