<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://www.w3.org/Bugs/Public/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4"
          urlbase="https://www.w3.org/Bugs/Public/"
          
          maintainer="sysbot+bugzilla@w3.org"
>

    <bug>
          <bug_id>1269</bug_id>
          
          <creation_ts>2005-04-24 20:18:23 +0000</creation_ts>
          <short_desc>color: rgb(real, int, int) should be reported as an error</short_desc>
          <delta_ts>2005-08-29 12:41:07 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>CSSValidator</product>
          <component>CSS 1.0</component>
          <version>CSS Validator</version>
          <rep_platform>PC</rep_platform>
          <op_sys>Windows XP</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>http://www.gtalbot.org/BrowserBugsSection/MSIE6Bugs/ParsingRGB.html</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Gérard Talbot">info</reporter>
          <assigned_to name="Olivier Thereaux">ot</assigned_to>
          
          
          <qa_contact name="qa-dev tracking">www-validator-cvs</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>3611</commentid>
    <comment_count>0</comment_count>
    <who name="Gérard Talbot">info</who>
    <bug_when>2005-04-24 20:18:23 +0000</bug_when>
    <thetext>&quot;illegal values, *_or values with illegal parts_*, are treated as if the
declaration weren&apos;t there at all&quot; 
CSS 1, section 7.1 Forward-compatible parsing
http://www.w3.org/TR/CSS1#forward-compatible-parsing

&quot;Illegal values. User agents must ignore a declaration with an illegal value.&quot;
CSS 2.1, section 4.2 Rules for handling parsing errors
http://www.w3.org/TR/CSS21/syndata.html#parsing-errors

&quot;Unsupported ValuesIf a UA does not support a particular value, it should ignore
that value when parsing stylesheets, as if that value was an illegal value.&quot;
CSS3, section 5. Rules for handling parsing errors or unsupported features 
http://www.w3.org/TR/2003/WD-css3-syntax-20030813/#error-handling

Actual results:
The CSS validator truncates the illegal part and does not report any parsing error.

Expected results:
The CSS validator should report the error and notify accordingly that only
integer value in the 0 - 255 range are valid.

Note: Safari 1.2+ and Mozilla-based browsers will parse a real value as an error
and will apply error condition to the declaration.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>5505</commentid>
    <comment_count>1</comment_count>
    <who name="Yves Lafon">ylafon</who>
    <bug_when>2005-08-26 14:22:19 +0000</bug_when>
    <thetext>Fixed, see http://qa-dev.w3.org:8001/css-validator/</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>5543</commentid>
    <comment_count>2</comment_count>
    <who name="Gérard Talbot">info</who>
    <bug_when>2005-08-26 17:07:32 +0000</bug_when>
    <thetext>Yves, the parsing error is detected and reported but the error message is rather
vague in my opinion. &quot;Invalid RGB function&quot; will certainly confuse a few people
and does not help as best as one could. 

1- Would it be possible to refine the error message? Just asking...

2- If one submits the file
http://www.hixie.ch/tests/evil/css/css21/tests/t040306-syntax-01-f.htm
the qa-dev.w3.org:8001/css-validator CSS validator reports that 

p#percentpercentpercent1  {

     color : rgb(0.0, 50.0, 0.0);

}
p#percentpercentpercent2 {

     color : rgb(0.0, 49.99, 0.0)

are valid info when the submitted rules were

    p#percentpercentpercent1 { color: rgb(0%, 50%, 0%) }
    p#percentpercentpercent2 { color: rgb(0%, 49.99%, 0%) }

Notice that the &quot;%&quot; is lost. Small glitch. Should I open a bug for that?

http://qa-dev.w3.org:8001/css-validator/validator?uri=http%3A%2F%2Fwww.hixie.ch%2Ftests%2Fevil%2Fcss%2Fcss21%2Ftests%2Ft040306-syntax-01-f.htm</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>5546</commentid>
    <comment_count>3</comment_count>
    <who name="Bj">bjoern</who>
    <bug_when>2005-08-26 17:39:54 +0000</bug_when>
    <thetext>Please open a new one.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>5559</commentid>
    <comment_count>4</comment_count>
    <who name="Yves Lafon">ylafon</who>
    <bug_when>2005-08-29 12:41:07 +0000</bug_when>
    <thetext>&gt; are valid info when the submitted rules were
&gt; 
&gt;     p#percentpercentpercent1 { color: rgb(0%, 50%, 0%) }
&gt;     p#percentpercentpercent2 { color: rgb(0%, 49.99%, 0%) }
&gt; 
&gt; Notice that the &quot;%&quot; is lost. Small glitch. Should I open a bug for that?
&gt; 
&gt;
http://qa-dev.w3.org:8001/css-validator/validator?uri=http%3A%2F%2Fwww.hixie.ch%2Ftests%2Fevil%2Fcss%2Fcss21%2Ftests%2Ft040306-syntax-01-f.htm

It should be fixed now.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>