20277 2012-12-06 17:19:59 +0000 Incorrect implementation of CssBackgroundPositionValue.toString() 2012-12-13 08:53:47 +0000 1 1 1 Unclassified CSSValidator Parser CSS Validator All All RESOLVED FIXED P2 major --- 1 hannes+w3cbugs dave.null www-validator-cvs oldest_to_newest 79567 0 hannes+w3cbugs 2012-12-06 17:19:59 +0000 The current code swaps the horizonal and vertical offsets in its toString() output. Here's a patch for the file org/w3c/css/properties/css3/CssBackgroundPosition.java : Index: CssBackgroundPosition.java =================================================================== --- CssBackgroundPosition.java (revision 8290) +++ CssBackgroundPosition.java (working copy) @@ -439,21 +439,22 @@ public String toString() { StringBuilder sb = new StringBuilder(); + if (horizontal != null) { + sb.append(horizontal); + if (horizontal_offset != null) { + sb.append(' ').append(horizontal_offset); + } + if (vertical != null) { + sb.append(' '); + } + } + if (vertical != null) { sb.append(vertical); if (vertical_offset != null) { sb.append(' ').append(vertical_offset); } - if (horizontal != null) { - sb.append(' '); - } } - if (horizontal != null) { - sb.append(horizontal); - if (horizontal_offset != null) { - sb.append(' ').append(horizontal_offset); - } - } return sb.toString(); } } 80069 1 ylafon 2012-12-13 08:53:47 +0000 Fixed by removing the inner class definition (it was not necessary, and even worse buggy as you pointed out).