Use lighter color for whitespace nodes

NEW
Unassigned

Status

Other Applications
DOM Inspector
--
enhancement
15 years ago
10 years ago

People

(Reporter: Jesse Ruderman, Unassigned)

Tracking

Trunk
x86
Windows XP

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

15 years ago
When whitespace nodes are displayed, they should be lighter blue than other text
nodes, in order to be less in-the-way visually.
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

15 years ago
Scanning the tree, I think it would be easier to ignore whitespace nodes if they
were closer to white than other nodes.
> 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....
Mass re-assigning bugs to dom.inspector@extensions.bugs
Assignee: caillon → dom.inspector
Product: Core → Other Applications
Assignee: dom-inspector → nobody
QA Contact: timeless → dom-inspector
You need to log in before you can comment on or make changes to this bug.