[Laszlo-checkins] r6392 - openlaszlo/branches/wafflecone/lps/components/incubator/rich-text

[ben@openlaszlo.org]

Author: ben
Date: 2007-09-06 13:49:27 -0700 (Thu, 06 Sep 2007)
New Revision: 6392

Modified:
   openlaszlo/branches/wafflecone/lps/components/incubator/rich-text/formatfontface.lzx
Log:
Change 20070906-ben-Q by ben at slim.local on 2007-09-06 13:20:31 PDT
    in /Users/ben/src/svn/openlaszlo/branches/wafflecone
    for http://svn.openlaszlo.org/openlaszlo/branches/wafflecone

Summary: Rich text editor: changing font to Verdana should behave properly

New Features:

Bugs Fixed: LPP-4556 Rich text: Can't change font back to Verdana

Technical Reviewer: crowley
QA Reviewer: (pending)
Doc Reviewer: (pending)

Documentation:
The formatfontface combo box needs to act as if it's changed, in order
to make sure that we tell the editor to change the font size. The superclass
(lzcombobox) doesn't seem to be setting it's "changed" instance variable
properly, so for now I'm just forcing changed to true when onselect is called. 
The lzcombobox potential bug is filed as LPP-4506. 

Release Notes:

Details:
    

Tests:
The test file for this is http://localhost:8080/wafflecone/lps/components/incubator/rich-text/test/richtexteditarea-driver.lzx?lzr=swf8
Make text of size 14, then you select it and try to change it to 18, and it will indeed 
change to size 18. 



Modified: openlaszlo/branches/wafflecone/lps/components/incubator/rich-text/formatfontface.lzx
===================================================================
--- openlaszlo/branches/wafflecone/lps/components/incubator/rich-text/formatfontface.lzx	2007-09-06 20:45:28 UTC (rev 6391)
+++ openlaszlo/branches/wafflecone/lps/components/incubator/rich-text/formatfontface.lzx	2007-09-06 20:49:27 UTC (rev 6392)
@@ -47,6 +47,7 @@
                 create an object representing just the change in format.
                 Give that object to the editor. --> 
             <method event="onselect"><![CDATA[
+                this.setChanged(true); // fix bug LPP-4556, Rich text: Can't change font back to Verdana
                 //------------------------------------------------------------
                 // make sure that we call into editor if this combobox has
                 // changed, i.e., it contains an initial value. -pk