Change 20100308-maxcarlson-n by maxcarl...@bank on 2010-03-08 16:41:43 PST
    in /Users/maxcarlson/openlaszlo/trunk-clean
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: UPDATED: Add clearRect() and strokeRect() to drawview

Bugs Fixed: LPP-8817 - Add strokeRect() and clearRect() to drawview

Technical Reviewer: hminsky
QA Reviewer: ptw

Details: Updated to address Tucker's concerns:
+ It seems to me that strokeRect will only draw 3 sides of the rectangle.  Am I 
confused?  Wouldn't there need to be 4 linteTo's?  

Yes, that was silly of me :)

+ Also, can you fill the comment in front of strokeRect so that it is not one 
long line?

Done.

In addition, I added support for gradient fills with fillRect() and refactored 
the box drawing routine to a separate __drawRect() method shared by fillRect() 
and strokeRect().

Otherwise the same:
Rename swf __updateStyles() to __updateLineStyle().  Add implementations of 
clearRect() and strokeRect() for DHTML and Flash.

Release Notes: drawview now supports clearRect() and strokeRect().

Tests: test/drawing/drawing.lzx runs consistently in dhtml and all versions of 
Flash.

Files:
M       lps/components/extensions/drawview.lzx

Changeset: 
http://svn.openlaszlo.org/openlaszlo/patches/20100308-maxcarlson-n.tar

_______________________________________________
Laszlo-reviews mailing list
[email protected]
http://www.openlaszlo.org/mailman/listinfo/laszlo-reviews

Reply via email to