Bug 46658 - The web colours palevioletred and mediumpurple are incorrect
Summary: The web colours palevioletred and mediumpurple are incorrect
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 525.x (Safari 3.2)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Emil A Eklund
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-27 14:59 PDT by Paul Gillard
Modified: 2011-03-10 23:38 PST (History)
4 users (show)

See Also:

Attachments
Patch (14.72 KB, patch)
2011-03-10 15:58 PST, Emil A Eklund 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Gillard 2010-09-27 14:59:30 PDT 
The RGB values returned for the palevioletred and mediumpurple web colours in Safari Version 5.0.2 (6533.18.5) and Chrome 6.0.472.63 are incorrect. Given the colours are incorrect in both browsers I'm assuming this is a WebKit issue. The following code can be used to illustrate the difference when compared to http://www.w3.org/TR/css3-color/#svg-color.

getWebColor = function(colour) {
  i = document.createElement("i")
  i.title = "Example"
  document.body.appendChild(i)
  i.style.color = colour
  document.defaultView.getComputedStyle(i, "").getPropertyValue("color")
}

getWebColor('mediumpurple');
getWebColor('palevioletred');
Comment 1 Rob Buis 2010-10-16 09:46:43 PDT 
I can confirm, and it also documented for instance here:

http://code.google.com/p/doctype/wiki/MediumpurpleCSSColor

However I do not know whether it was done intentional.
Cheers,

Rob.
Comment 2 Emil A Eklund 2011-03-10 15:58:30 PST 
Created attachment 85400 [details]
Patch

Likely a type in the original import, in both cases the fix is to change 8->B.
Comment 3 Emil A Eklund 2011-03-10 16:05:15 PST 
Err, type->typo.
Comment 4 Alexey Proskuryakov 2011-03-10 16:10:56 PST 
Comment on attachment 85400 [details]
Patch

I'll give a tentative r+, but please make sure that this matches other browsers.
Comment 5 Emil A Eklund 2011-03-10 16:27:22 PST 
Of course. I verified on Mozilla (3.6) and IE (9) and they both match the specification.

I could check older versions of IE and Mozilla if you'd like me to.
Comment 6 Alexey Proskuryakov 2011-03-10 16:34:14 PST 
Comment on attachment 85400 [details]
Patch

Probably not worth checking old versions in this case.
Comment 7 WebKit Commit Bot 2011-03-10 23:38:26 PST 
Comment on attachment 85400 [details]
Patch

Clearing flags on attachment: 85400

Committed r80823: <http://trac.webkit.org/changeset/80823>
Comment 8 WebKit Commit Bot 2011-03-10 23:38:29 PST 
All reviewed patches have been landed.  Closing bug.