Change philip-20110608-Alx by philip@PHILIP-I7 on 2011-06-08 18:47:31 EDT
    in /cygdrive/c/clients/laszlo/svn/openlaszlo/trunk
    for http://svn.openlaszlo.org/openlaszlo/trunk

Summary: Support 8 border radius values in the kernel

New Features:

Bugs Fixed: LPP-9909

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

Documentation:

Release Notes:

Overview:
lz.view and the kernel only supported 4 border radii, rather than 8 radii that are defined by css. With this changeset, you can specify the border radii in a few ways. These are all equivalent:

cornerradius="4"
cornerradius="4 4 4 4"
cornerradius="4 4 4 4 / 4 4 4 4"

The first case specifies a single corner radius to use at each corner.
The second version specifies a corner radius to use at each corner.
The third version specifies separate horizontal and vertical radii to use at each corner. The first set of 4 values is the horizontal radii, and the second set of 4 values is the vertical radii. More information can be found here: http://www.w3.org/TR/css3-background/#the-border-radius


Details:
LaszloView.lzs
- Parse 1, 4 or 8 radii the from cornerradius attributes.

LzSprite.js
- IE9 supports cornerradius.
- Parse cornerradius and store in two formats. One format is used for browsers where all corner radii can be specified in a single line. The other is for browsers where each radii is specified separately.

LzSprite.as
- Added drawRoundRect() method. This is needed because a circle is not drawn if you draw four curved lines for the cornerradius. When a circle is detected, drawCircle() is called. To insure that a circle is shown, the detection of a circle is done to one decimal place.
- Calls to LzKernelUtils.rect changed to LzKernelUtils.roundrect.

Tests:
Test case in LPP-9909 works properly with changeset applied.

Files:
M       WEB-INF/lps/lfc/kernel/dhtml/LzSprite.js
M       WEB-INF/lps/lfc/kernel/swf9/LzSprite.as
M       WEB-INF/lps/lfc/views/LaszloView.lzs

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/philip-20110608-Alx.tar

Reply via email to