Re: Patch for CSS Validator bug #761

He Peter,

On Dec 9, 2006, at 04:17 , Peter Zhelezniakov wrote:
> Here's my patch for Validator bug #761. It's short and simple.  
> Could you
> please review it? XMLStyleSheetHandler is the only file changed  
> since i
> want the new logic to apply to XHTML only.

Looks good, I applied and tested it. Looking at
http://www.w3.org/Bugs/Public/show_bug.cgi?id=761
I noticed that it would be better that the parser should not pass the  
style content within the comments to the CSS parser, so I later  
modified your code to do that, see:
http://lists.w3.org/Archives/Public/www-validator-cvs/2006Dec/0042.html

> I tried to enter the patch into Bugzilla but it didn't work for me  
> -- is
> it working at all?

Bugzilla has a bug that becomes unbearable in our setup, indeed.
The solution I had found for it is to
* go to http://www.w3.org/Bugs/Public/
* log out if necessary and go to the login page
* make sure the checkbox for "restrict this session to this IP  
addres" is NOT checked
* log in

You should be rid of the problem.

(The problem comes when you lose authentication between the time you  
go to the attachment page and submit it, and with our setting of  
multiple proxies, it breaks very often)

> Anyway the patch is not complete, it lacks message
> strings for languages other than English (i define a new warning  
> message
> there). How do we go about that?

I copied the english version in all the message properties files, and  
will do a call for translation.
http://esw.w3.org/topic/CssValidator/Localization

Again, thanks!
-- 
olivier

Received on Tuesday, 12 December 2006 06:50:24 UTC