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 +1.
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 <[email protected]>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 >
