myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeanne Waldman <jeanne.wald...@oracle.com>
Subject Re: [Trinidad][Skinning] Introduce include-Property in CSS
Date Thu, 14 Jan 2010 23:45:03 GMT
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
What is 'name'? Is that an alias? We distinguished between selector and
name in XSS but we don't in the CSS format. We still do in the code,
but the person working with the css shouldn't know the difference.<br>
<br>
Yes, the con of my suggestion is that order matters and the user needs
to know what the order means .I can see them not knowing is color what
I'm setting or is color what I'm retrieving? That is what I think the
con is for the CSS syntax like <i>padding: 0px 2px 3px 4px</i><br>
<br>
<br>
Regarding your link, I haven't had time to look at that yet. <br>
<br>
Jeanne<br>
<br>
Marius Petoi wrote, On 1/12/2010 11:46 PM PT:
<blockquote
 cite="mid:16f5365c1001122346v3b6dbe79sb91e6442e6543b7b@mail.gmail.com"
 type="cite">Hi Jeanne,<br>
  <br>
Thank you for the answer! Like in the situation of -tr-rule-ref, the
list of properties is comma separated. I don't understand what you mean
by camel-case...The name of the new property can be whatever the user
wishes for. Afterwards, it will be treated like all the other
properties in the CSS. Regarding the new syntax you suggested, first of
all, we may have a selector or a name, in which case "selector" is
replaced with "name". Also, another problem is the order in which they
appear; with this syntax it can be any order.<br>
  <br>
How about <a moz-do-not-send="true"
 href="http://markmail.org/search/?q=skinning%20list%3Aorg.apache.myfaces.dev#query:skinning%20list%3Aorg.apache.myfaces.dev%20order%3Adate-backward+page:3+mid:3au5ilvrrpbxopgx+state:results">http://markmail.org/search/?q=skinning%20list%3Aorg.apache.myfaces.dev#query:skinning%20list%3Aorg.apache.myfaces.dev%20order%3Adate-backward+page:3+mid:3au5ilvrrpbxopgx+state:results</a>
? Did you have the time to look over it too? <br>
  <br>
Regards,<br>
Marius<br>
  <br>
  <div class="gmail_quote">On Tue, Jan 12, 2010 at 11:58 PM, Jeanne
Waldman <span dir="ltr">&lt;<a moz-do-not-send="true"
 href="mailto:jeanne.waldman@oracle.com">jeanne.waldman@oracle.com</a>&gt;</span>
wrote:<br>
  <blockquote class="gmail_quote"
 style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left:
1ex;">
    <div bgcolor="#ffffff" text="#000000">
    <pre><code>Another idea for the syntax comes from the rgb color syntax -
color: rgb(100%, 0%, 0%)

You could use this syntax, and not specify what each of the properties is for:
</code>-tr-include-property:
property("af|foo", "color", "background-color")
or
property(af|foo, color, background-color)

I like this because it's shorter, but I don't like it since they will have to look up which
is which, something I have to do when I use the border: 0px 3px 2px 1px syntax - which is
right, left, top, bottom.


Jeanne 
    
<code>
</code></pre>
    <br>
    <br>
Jeanne Waldman wrote, On 1/12/2010 10:21 AM PT:
    <div>
    <div class="h5">
    <blockquote type="cite"> Hi,<br>
Thanks for this patch.<br>
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.<br>
Like, is the comma standard, or should it be space-separated? Is the
camel-case standard, or should it be '-'s.<br>
I think it looks good, but I'll have to look at it closer before I vote.<br>
      <br>
Jeanne<br>
      <br>
Marius Petoi wrote, On 1/11/2010 4:52 AM PT:
      <blockquote type="cite">Is there anyone who has already reviewed
this or added
them on his/her TODO list? Thank you in advance!<br>
        <br>
Marius<br>
        <br>
        <div class="gmail_quote">On Fri, Jan 8, 2010 at 11:35 AM,
Marius
Petoi <span dir="ltr">&lt;<a moz-do-not-send="true"
 href="mailto:marius.petoi@codebeat.ro" target="_blank">marius.petoi@codebeat.ro</a>&gt;</span>
wrote:<br>
        <blockquote class="gmail_quote"
 style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left:
1ex;">Hello,<br>
          <br>
I created a new JIRA task for this issue (<a moz-do-not-send="true"
 href="https://issues.apache.org/jira/browse/TRINIDAD-1680"
 target="_blank">https://issues.apache.org/jira/browse/TRINIDAD-1680</a>)
and I added a patch for it.<br>
          <br>
I introduced a new -tr property: "-tr-include-property". The syntax of
this is: <br>
          <br>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;-tr-include-property:
property(selector="af|foo",propertyName="color",
localPropertyName="background-color") <br>
          <br>
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. <br>
          <br>
I also introduced the new feature in the documentation.<br>
          <br>
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.<br>
          <br>
Regards,<br>
          <font color="#888888">Marius<br>
          </font></blockquote>
        </div>
        <br>
      </blockquote>
    </blockquote>
    </div>
    </div>
    </div>
  </blockquote>
  </div>
  <br>
</blockquote>
</body>
</html>

Mime
View raw message