- From: Yves Lafon <ylafon@w3.org>
- Date: Tue, 08 Nov 2005 07:45:49 +0000
- To: olivier Thereaux <olivier@tokyoartbeat.com>
- Cc: QA Dev <public-qa-dev@w3.org>
On Mon, 31 Oct 2005, olivier Thereaux wrote: > Hi Yves, all. > > While coding in new functionalities into the perl lib that accesses the CSS > validator's SOAP output, I noticed that the uri context is apparently only > given at the start of the <m:warninglist> or <m:errorlist>. I suggest it > should be given for each message (error, warning), even if it makes the > output more verbose. Well, there may be multiple <m:warninglist> and <m:errorlist>, each list gives the uri context in it, for all the item parts of this list. If URIs are put in each warning or error, the list grouping makes less sense. Both version (the current one and the proposed one) have the same expressiveness, so it's just a matter of style and how people intend to use this output. > > Rationale: when validating a document linking to different stylesheets, the > errors and warnings may come from several sources. Knowing exactly which > errors come from which document/location is therefore very important. > > so instead of : > > <m:warninglist> > <m:uri>http://www.example.org/style.css</m:uri> > <m:warning> > <m:line>24</m:line> > <m:level>1</m:level> > <m:message>You have no background-color with your color</m:message> > <m:context>A:link</m:context> > </m:warning> > <m:warning> > <m:line>30</m:line> > <m:level>1</m:level> > <m:message>You have no background-color with your color</m:message> > <m:context>A:visited</m:context> > </m:warning> > </m:warninglist> > > I suggest > <m:warninglist> > <m:warning> > <m:uri>http://www.example.org/style.css</m:uri> > <m:line>24</m:line> > <m:level>1</m:level> > <m:message>You have no background-color with your color</m:message> > <m:context>A:link</m:context> > </m:warning> > <m:warning> > <m:uri>http://www.example.org/style.css</m:uri> > <m:line>30</m:line> > <m:level>1</m:level> > <m:message>You have no background-color with your color</m:message> > <m:context>A:visited</m:context> > </m:warning> > </m:warninglist> > > If you think this makes sense, here's a patch (attached) which I will commit > unless you think it's a bad idea. > > Thank you. > -- Yves Lafon - W3C "Baroula que barouleras, au tiéu toujou t'entourneras."
Received on Tuesday, 8 November 2005 08:31:02 UTC