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 ? If it doesn't, then my vote is -1; otherwise, I would vote +1.
However, I also notice that the fox property name proposed in  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.  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  and  for details. > > There is an example fo  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 > >  http://wiki.apache.org/xmlgraphics-fop/RoundedBorders >  http://xmlgraphics.staging.apache.org/fop/trunk/extensions.html >  examples/fo/advanced/rounded-corners.fo in the Temp_RoundedCorners > branch >