I haven't had a chance to look at the details, but does this extension
follow the (property name and value) definitions found in CSS3 Backgrounds
and Borders [4]? If it doesn't, then my vote is -1; otherwise, I would vote

However, I also notice that the fox property name proposed in [1] contains
uppercase (fox:border-BLOCK-radius-INLINE). That is a definite no-no, and
thus warrants a -1 vote until changed to LC. All LC please!

In general, the property names and values should exactly match what are
proposed in CSS3 unless there is *a really good reason* why that shouldn't
be the case.

[4] http://www.w3.org/TR/css3-background/#corners

On Fri, Oct 12, 2012 at 5:40 PM, Peter Hancock <peter.hanc...@gmail.com>wrote:

> Hi,
> Luis Benardo and Myself have just done some clean up to the branch
> Temp_RoundedCorners.  This branch implements support for 'fox'
> extension properties  for specifying borders with rounded corners.
> Please refer to [1] and [2] for details.
> There is an example fo [3] that demonstrates the feature.
> Currently we are supporting:
>   PDF, PS and AFP outputs
>   'border-style' property with values of  'solid', 'none', 'hidden'
> and, to a limited degree, 'dashed'
> I would like to start a vote to merge feature branch to trunk, with my +1.
> Thanks,
> Peter
> [1] http://wiki.apache.org/xmlgraphics-fop/RoundedBorders
> [2] http://xmlgraphics.staging.apache.org/fop/trunk/extensions.html
> [2] examples/fo/advanced/rounded-corners.fo in the Temp_RoundedCorners
> branch

Reply via email to