Closed Bug 201835 Opened 21 years ago Closed 6 years ago

Use lighter color for whitespace nodes

Categories

(Other Applications :: DOM Inspector, enhancement)

x86
Windows XP
enhancement
Not set
normal

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.
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.
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
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
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.