Closed
Bug 201835
Opened 21 years ago
Closed 6 years ago
Use lighter color for whitespace nodes
Categories
(Other Applications :: DOM Inspector, enhancement)
Tracking
(Not tracked)
RESOLVED
INCOMPLETE
People
(Reporter: jruderman, Unassigned)
Details
When whitespace nodes are displayed, they should be lighter blue than other text nodes, in order to be less in-the-way visually.
Comment 1•21 years ago
|
||
bz: for this bug, it would require hacking inIDOMView to return a different - moz-tree-cell-text() style rule, which the Modern and Classic skins would then need a patch to for the new color code. How would we go about that, if we were to add this feature? jruderman: how would a lighter color be less in-the-way? I don't quite follow that argument.
Reporter | ||
Comment 2•21 years ago
|
||
Scanning the tree, I think it would be easier to ignore whitespace nodes if they were closer to white than other nodes.
Comment 3•21 years ago
|
||
> How would we go about that, if we were to add this feature? See http://lxr.mozilla.org/seamonkey/source/extensions/inspector/base/src/inDOMView.cpp#306 and http://lxr.mozilla.org/seamonkey/source/extensions/inspector/base/src/inDOMView.cpp#105 and http://lxr.mozilla.org/seamonkey/source/extensions/inspector/base/src/inDOMView.cpp#125 Use http://lxr.mozilla.org/seamonkey/source/extensions/inspector/base/src/inDOMView.cpp#1271 -like code to test for whitespaceness (note the construction of mDOMUtils earlier in that function, though the test for mShowWhitespaceNodes should actually be reversed there....
Comment 4•21 years ago
|
||
Mass re-assigning bugs to dom.inspector@extensions.bugs
Assignee: caillon → dom.inspector
Updated•20 years ago
|
Product: Core → Other Applications
Updated•17 years ago
|
Assignee: dom-inspector → nobody
QA Contact: timeless → dom-inspector
Comment 5•6 years ago
|
||
Bulk close. This component is no longer supported or maintained. https://bugzilla.mozilla.org/show_bug.cgi?id=1499023
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INCOMPLETE
Comment 6•6 years ago
|
||
Bulk close. This component is no longer supported or maintained. https://bugzilla.mozilla.org/show_bug.cgi?id=1499023
You need to log in
before you can comment on or make changes to this bug.
Description
•