W3C home > Mailing lists > Public > www-style@w3.org > September 2008

[css3-color] XSL FO SG comments on "CSS Color Module Level 3" WD

From: Tony Graham <Tony.Graham@MenteithConsulting.com>
Date: Mon, 01 Sep 2008 19:04:51 +0100
To: www-style@w3.org
Cc: w3c-xsl-fo-sg@w3.org
Message-ID: <8763pfpwsc.fsf@menteith.com>

The XSL FO Subgroup reviewed the CSS Color Module Level 3 WD
(http://www.w3.org/TR/2008/WD-css3-color-20080721).

The following comments are editorial only.

Regards,

Tony Graham                         Tony.Graham@MenteithConsulting.com
Director                                  W3C XSL FO SG Invited Expert
Menteith Consulting Ltd
XML, XSL and XSLT consulting, programming and training
Registered Office: 13 Kelly's Bay Beach, Skerries, Co. Dublin, Ireland
Registered in Ireland - No. 428599   http://www.menteithconsulting.com
  --  --  --  --  --  --  --  --  --  --  --  --  --  --  --  --  --
xmlroff XSL Formatter                               http://xmlroff.org
xslide Emacs mode                  http://www.menteith.com/wiki/xslide
Unicode: A Primer                               urn:isbn:0-7645-4625-2



 - The capitalisation of the titles of sections 4.2.4.1, 4.5, 4.5.1, 8,
   and 9 is inconsistent with the majority of the titles.


Abstract
========

 - "color related properties" should be "color-related properties".


Status of This Document
=======================

 - In the 'interoperable' definition, "must one or more" should be "must
   be one or more".


1. Introduction
===============

 - "color related properties" should be "color-related properties".


2. Dependencies
===============

 - The links to other CSS3 modules don't work when the spec is printed.


3.1 Foreground color: the ‘color’ property
==========================================

 - The two uses of "color:inherit" (the other is in section 4.4) are the
   only declarations that do not have a space after 'color:'.  While
   allowed by CSS syntax, this is inconsistent with the other examples.

 - There is no Example I or Example II.


3.1.1 Gamma correction
======================

 - How to set gamma correction on different operating systems should be
   in a note or in an informative appendix.

 - How to populate a lookup table should be in a note.


3.2 Transparency: the ‘opacity’ property
========================================

 - "thought of conceptually" is a tautology.


4.1 HTML4 color keywords
========================

 - Should the document be consistent and use either only uppercase or
   only lowercase for A-F (or a-f) in numerical color specifications?


4.2.2 RGBA color values
=======================

 - Using 'new' in "specify new effects that are now possible with the
   new rgba() notation" will be incorrect when CSS3 is no longer new.
   (Also in section 4.2.4.1.)

 - There should be a reference to where to find the 'CSS forward
   compatibility parsing rules'.


4.2.3 ‘transparent’ color keyword
=================================

 - There should be a comma after "rgba(0,0,0,0)".


4.2.4 HSL color values
======================

 - What is the "something similar" in "internalizing how to translate
   hue, saturation and lightness, or something similar"?

 - "[0,360)" indicates a range exclusive of 360.  The notation is not
   commonplace and should be explained.  Also, the notations for this
   range and for the "0..1" range used later in the section (and the
   "0..255" ranges in section 4.2.1) are not consistent.

 - There is a parenthesis missing in "((x mod 360) + 360) mod 360)".

 - How to populate a lookup table should be in a note.

 - It is not clear why an inverted exclamation mark is being used to
   represent degrees.  (It was also not clear that they were inverted
   exclamation marks until I looked at the page source.)


4.4 'currentColor' color keyword
================================

 - "‘flood-color’, ‘lighting-color’" should be "‘flood-color’, and
   ‘lighting-color’" (or "‘flood-color’ and ‘lighting-color’", depending
   on how you feel about commas).


4.5 CSS System Colors
=====================

 - The only content is in section 4.5.1.  This title should be removed,
   and section 4.5.1 should become section 4.5.


4.5.1 CSS2 System Colors
========================

 - "pre-defined" should be "predefined".

 - It is not clear (unless it's for historical reasons) why the
   definition of the computed value of a System Color keyword ("The
   computed value of a System Color keyword value is the keyword
   itself.") is a note but similar statements for 'transparent' and
   'currentColor' keywords are not notes.
Received on Monday, 1 September 2008 20:47:43 GMT

This archive was generated by hypermail 2.2.0+W3C-0.50 : Monday, 27 April 2009 13:55:12 GMT