Re: Various approachs to bug 5890

Miguel Garcia wrote:
> Hello
> 
> There is a remaining css related bug, this is bug 5890 - Carriage
> returns in CSS Styles not correctly handled. Currently the checker is
> processing CSS line by line but is correct to write a carriage return in
> the middle of a style like
> 
> p { font 
> : 
>  
>  
> bold italic 
>  12px Palatino, serif }
> 
> (I haven't checked if the CSS specification really allows this kind of
> syntax but the css validator doesn't trigger any error on the previous
> code).

The CSS1 grammar is a bit hard to read, but it does allow this kind of 
syntax, indeed.

> 
> We have thought 3 possible solutions:
> 
> 1) Process all CSS code in all together instead of line by line. It will
> be quite fast to implement this approach but the drawback is that will
> lose the line number of the error.

[...]
> 
> The second option is more a workaround than a real fix and we are
> lacking enough time to choose the third one. 
> 
> Any other thoughts, or any point we are missing?

I think you've said it all ;)
I haven't checked, though, do we only lose the line of code or do we 
also lose the code excerpt?

The former one is a plus, but not that compulsory I suppose. The latter 
one is more of a problem, IMO.

Francois.

Received on Monday, 28 July 2008 15:43:05 UTC