Author: ben
Date: 2007-08-21 21:26:21 -0700 (Tue, 21 Aug 2007)
New Revision: 6176
Modified:
openlaszlo/branches/wafflecone/lps/components/incubator/rich-text/test/formatfontsize.lzx
openlaszlo/branches/wafflecone/lps/components/incubator/rich-text/test/richtexteditarea-test.lzx
Log:
Change 20070821-ben-d by [EMAIL PROTECTED] on 2007-08-21 17:53:53 PDT
in /Users/ben/src/svn/openlaszlo/branches/wafflecone
for http://svn.openlaszlo.org/openlaszlo/branches/wafflecone
Summary: Eliminate warnings in richtext tests.
New Features:
Bugs Fixed:
Technical Reviewer: jcrowley
QA Reviewer: mkratt (pending)
Doc Reviewer: (pending)
Documentation:
Several warnings and errors were being emitted at startup of the rich text
edit area test and driver files. These were because the linking up of the
editor to the formatting menus was fuzzy; I've made the linking up
explicit, and now we don't get these warnings, but the functionality
still works once we are all hooked up.
Release Notes:
Details:
Tests:
http://localhost:8080/wafflecone/lps/components/incubator/rich-text/test/richtexteditarea-test.lzx?lzr=swf8&debug=true&lzbacktrace=true
http://localhost:8080/wafflecone/lps/components/incubator/rich-text/test/richtexteditarea-driver.lzx?lzr=swf8&debug=true&lzbacktrace=true
..runs without warnings or errors.
Modified:
openlaszlo/branches/wafflecone/lps/components/incubator/rich-text/test/formatfontsize.lzx
===================================================================
---
openlaszlo/branches/wafflecone/lps/components/incubator/rich-text/test/formatfontsize.lzx
2007-08-22 01:51:46 UTC (rev 6175)
+++
openlaszlo/branches/wafflecone/lps/components/incubator/rich-text/test/formatfontsize.lzx
2007-08-22 04:26:21 UTC (rev 6176)
@@ -48,7 +48,7 @@
- <attribute name="editor" value="$once{this.parent.editor}" />
+ <attribute name="editor" />
<attribute name="toolbar" value="$once{this.parent}" />
@@ -70,8 +70,6 @@
toolbar.registerTool(this,"size");
- this.setValue(this.editor.defaultfontsize, true);
-
</method>
<!--- @keywords private
@@ -99,17 +97,23 @@
//------------------------------------------------------------
if (this.changed && this.toolbar._passthruchanges) {
+ if ("editor" in this) {
+ this.editor.setFormatAttribute("size",
this.getValue());
+ this.editor.delegateRestoreSelection();
+ }
- this.editor.setFormatAttribute("size", this.getValue());
-
- this.editor.delegateRestoreSelection();
-
}
]]>
</method>
+ <!-- When we get an editor to hook up to, make this menu's value
+ match up to the default font of the editor. -->
+ <method event="oneditor" args="e">
+ this.setValue(this.editor.defaultfontsize, true);
+ </method>
+
<!--- method broadcast out by the toolbar for clearing tool's state
-->
Modified:
openlaszlo/branches/wafflecone/lps/components/incubator/rich-text/test/richtexteditarea-test.lzx
===================================================================
---
openlaszlo/branches/wafflecone/lps/components/incubator/rich-text/test/richtexteditarea-test.lzx
2007-08-22 01:51:46 UTC (rev 6175)
+++
openlaszlo/branches/wafflecone/lps/components/incubator/rich-text/test/richtexteditarea-test.lzx
2007-08-22 04:26:21 UTC (rev 6176)
@@ -26,9 +26,9 @@
<formatbtn name="boldbtn" id="gBoldBtn" format="bold" iconname="bold"
icony="4" />
<formatbtn name="underlinebtn" id="gUnderlineBtn" format="underline"
iconname="underline" />
<formatbtn name="italicbtn" id="gItalicBtn" format="italic"
iconname="italic" icony="4" />
- <formatfontface name="fontfacecb" y="${(parent.height-this.height)/2}"/>
- <formatfontcolor name="fontcolorcb" y="${(parent.height-this.height)/2}"/>
- <formatfontsize name="fontsizecb"
y="${Math.round((parent.height-this.height)/2)}"/>
+ <formatfontface name="fontfacecb" id="gFontFaceCB"
y="${(parent.height-this.height)/2}"/>
+ <formatfontcolor name="fontcolorcb" id="gFontColorCB"
y="${(parent.height-this.height)/2}"/>
+ <formatfontsize name="fontsizecb" id="gFontSizeCB"
y="${Math.round((parent.height-this.height)/2)}"/>
</richtexttoolbar>
@@ -42,6 +42,16 @@
defaultfontcolor="0x336699"
border="10" text="For reasons which many persons thought
ridiculous, Mrs. Lightfoot Lee decided to pass the winter in Washington. She
was in excellent health, but she said that the climate would do her good. In
New York she had troops of friends, but she suddenly became eager to see again
the very small number of those who lived on the Potomac. It was only to her
closest intimates that she honestly acknowledged herself to be tortured by
ennui. Since her husband's death, five years before, she had lost her taste for
New York society; she had felt no interest in the price of stocks, and very
little in the men who dealt in them; she had become serious. What was it all
worth, this wilderness of men and women as monotonous as the brown stone houses
they lived in? In her despair she had resorted to desperate measures. She had
read philosophy in the original German, and the more she read, the more she was
disheartened that so much culture should lead to nothing... not!
hing.">
+ <method name="init">
+ super.init();
+ gBoldBtn.setAttribute("editor", this);
+ gUnderlineBtn.setAttribute("editor", this);
+ gItalicBtn.setAttribute("editor", this);
+ gFontFaceCB.setAttribute("editor", this);
+ gFontSizeCB.setAttribute("editor", this);
+ gFontColorCB.setAttribute("editor", this);
+ </method>
+
</richtexteditarea>
</view>
</view>
_______________________________________________
Laszlo-checkins mailing list
[email protected]
http://www.openlaszlo.org/mailman/listinfo/laszlo-checkins