Re: CSS metadata boilerplate

2014-01-09 Thread John Hendrikx
Noticed a slight bug in the code. The line: cssMetaDataByName.put(cssName, cssMetaData); ...needs to be added in the if block where the CssMetaData is created. --John

Re: CSS metadata boilerplate

2014-01-09 Thread David Grieve
Thanks, John. It would be helpful if you could create an issue in jira to track this. On Jan 8, 2014, at 4:18 PM, John Hendrikx hj...@xs4all.nl wrote: On 8/01/2014 14:28, David Grieve wrote: The reason things are as they are is because most properties in the core classes are lazily created

Re: CSS metadata boilerplate

2014-01-08 Thread Tomas Mikula
+1. I ended up creating such property subclasses myself, too. And I don't provide the static getClassCssMetaData() either. Tomas On Wed, Jan 8, 2014 at 7:05 AM, Tom Eugelink t...@tbee.org wrote: Yes, I had similar considerations. I was thinking about providing exactly such extended Property

Re: CSS metadata boilerplate

2014-01-08 Thread David Grieve
The reason things are as they are is because most properties in the core classes are lazily created and I didn't want cause the property to be created just to see if it was bound. But that is a particular concern for the core classes and not so much for controls. I'm not sure if you are aware

Re: CSS metadata boilerplate

2014-01-08 Thread John Hendrikx
On 8/01/2014 14:28, David Grieve wrote: The reason things are as they are is because most properties in the core classes are lazily created and I didn't want cause the property to be created just to see if it was bound. But that is a particular concern for the core classes and not so much for

CSS metadata boilerplate

2014-01-07 Thread John Hendrikx
Hi List, I'm in the process of adding CSS metadata to a new control, and I noticed there is a lot of boilerplate. In order to reduce this, I've created some custom classes StyleableProperty classes (SimpleStyleableXXXProperty), which reduces the boilerplate significantly without sacrificing

Re: CSS metadata boilerplate

2014-01-07 Thread Tom Eugelink
Yes, I had similar considerations. I was thinking about providing exactly such extended Property classes in JFXtras to remove as much of the boilerplate. Tom On 2014-1-8 3:34, John Hendrikx wrote: Hi List, I'm in the process of adding CSS metadata to a new control, and I noticed there is