Closed
Bug 7046
Opened 26 years ago
Closed 25 years ago
[PDT-]style.color returns rgb(r,g,b) rather than #rrggbb
Categories
(Core :: DOM: Core & HTML, defect, P3)
Tracking
()
VERIFIED
INVALID
M14
People
(Reporter: reboughner, Assigned: pollmann)
Details
(Keywords: testcase, Whiteboard: [TESTCASE] The button should change color onMouseOver)
Attachments
(3 files)
The style of a button for changing page layout doesn't happen. This is done
through Javascript and something is obviously going on since the button is
changing as the cursor moves over it.
Reporter | ||
Comment 1•26 years ago
|
||
Reporter | ||
Comment 2•26 years ago
|
||
Was tested using the 5/17/99 build of NGLayout for a Windows platform
hi, reboughner. nglayout bugs usually should get filed under the "DOM level 1"
or "DOM level 0" component. thanks!!!
Updated•26 years ago
|
Assignee: norris → vidur
Component: JavaScript → DOM Level 1
Updated•26 years ago
|
Assignee: vidur → pollmann
Comment 4•26 years ago
|
||
Eric, I believe the style change notification is going through. The change
doesn't seem to be propogated to the widget, though.
Comment 5•26 years ago
|
||
Using the July 26 build for Linux, this seems to be working. I think this should
be WORKSFORME, but can anyone else confirm? This bug report isn't too clear. I
simplified the attachment and got rid of the extra bits to try and test this a
bit better, and asides from a minor button rendering bug, it works for me.
Updated•26 years ago
|
Whiteboard: [TESTCASE] Seems to work (July 26 Linux build)
Comment 6•26 years ago
|
||
Reporter | ||
Comment 7•26 years ago
|
||
I've retried this example with the M8 build and it still doesn't work.
Obviously, something is going on because I see the button quivver but its style
or the lettering on it doesn't change. It works fine in IE 5.0
Updated•26 years ago
|
Whiteboard: [TESTCASE] Seems to work (July 26 Linux build) → [TESTCASE] The button should change color onMouseOver
Comment 8•26 years ago
|
||
Oops, my mistake. Everything works as of the July 30th build on windows 98
except that the button does not change color. It should toggle to yellow from
black onMouseOver.
Updated•25 years ago
|
Target Milestone: M13 → M16
Comment 9•25 years ago
|
||
Moving to M16. Result is aesthetically incorrect but nonfatal and pollmann's got
lots of worse bugs to fix for now.
Comment 10•25 years ago
|
||
Bulk moving [testcase] code to new testcase keyword. Sorry for the spam!
Keywords: testcase
Comment 12•25 years ago
|
||
PDT wants to know more details on why this is a beta1 stopper
Assignee | ||
Comment 14•25 years ago
|
||
This already almost works. The problem is that you get btn.style.color which
was set to "#000000" and we return "rgb(0,0,0)". If you compare that string
format rather than "#nnnnnn", it works. (Of course for black and yellow you
could just use the color name.) Attaching a test case.
Assignee | ||
Comment 15•25 years ago
|
||
Reporter | ||
Comment 16•25 years ago
|
||
I am glad that the example works with the proper coding. Just two comments:
1. I rarely use the rgb(x,x,x) form for the color since I prefer the hex format.
I just didn't think to try it.
2. For your information, IE does return the hex format and that is the reason I
used that form.
Assignee | ||
Comment 17•25 years ago
|
||
Since rgb(0,0,0) is a valid, canonical format for returning the color attribute,
I'm going to close this issue as invalid.
It is part of the DOM spec that attributes may be returned in a format different
than that specified. For more information on this see the following URL:
http://www.w3.org/TR/DOM-Level-2/css.html#CSS-CSS2Properties
Thanks!
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → INVALID
Summary: Style change specified by Javascript doesn't occur → style.color returns rgb(r,g,b) rather than #rrggbb
Comment 18•25 years ago
|
||
Per pollmann's comments, marking Verified. PDT-
Status: RESOLVED → VERIFIED
Summary: style.color returns rgb(r,g,b) rather than #rrggbb → [PDT-]style.color returns rgb(r,g,b) rather than #rrggbb
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•