Re: [Trinidad][Skinning] Introduce include-Property in CSS

[Jeanne Waldman]

Hi, Thanks for this patch. I will have to look at the CSS spec to see if this syntax conforms to other CSS syntaxes. This is what I usually do when I try to come up with a new skinning api. Like, is the comma standard, or should it be space-separated? Is the camel-case standard, or should it be '-'s. I think it looks good, but I'll have to look at it closer before I vote. Jeanne Marius Petoi wrote, On 1/11/2010 4:52 AM PT: Is there anyone who has already reviewed this or added them on his/her TODO list? Thank you in advance! Marius On Fri, Jan 8, 2010 at 11:35 AM, Marius Petoi <marius.pe...@codebeat.ro> wrote: Hello, I created a new JIRA task for this issue (https://issues.apache.org/jira/browse/TRINIDAD-1680) and I added a patch for it. I introduced a new -tr property: "-tr-include-property". The syntax of this is:      -tr-include-property: property(selector="af|foo",propertyName="color", localPropertyName="background-color") In SkinStyleSheetParserUtils, when the selectors are parsed, similar to the -tr-rule-ref, I introduced a list of includedProperties. The rules defined with -tr-include-property are parsed and the list of includedProperties is filled up. In the end, when the StyleNode is created, this list is passed to the constructor. I also introduced the new feature in the documentation. Is the syntax of the new rule ok? And if so, please have a look over the patch and tell me whether I should modify anything. Regards, Marius