Export to GitHub

fbug - issue #3790

Better DOM representation for element attributes


Posted on Dec 9, 2010 by Massive Wombat

One this I would like to have in the DOM panel (and so, also in the watch panel!) is a better representation of element attributes. Every time I am debugging and inspecting attributes of an element through the watch window, I am suffering from the visual rep of it.

The main info about an attribute is usually its name and value.

And the problem is that this info is not immediately visible. See my attached image "theCurrentState.png"

I am imagining something like: "theNewState.png"

This would require new reps (in reps.js) for NamedNodeMap and Attr types.

What do you think?

Honza

Attachments

Comment #1

Posted on Dec 9, 2010 by Happy Dog

Might be easier to just change the Rep. I do that in Illumination. I only show the first three of prioritized properties. In fact, I had an issue open for that a while back. But this should easier, but will need a Attr { prefix for when it also goes into the console.

Comment #2

Posted on Dec 10, 2010 by Massive Wombat

Patch committed at R8707

I decided to proceed with this one, since I think better solution is needed. It's not for 1.6 so we have enough time to get feedback on it.

FBTest committed at R8708, dom/attributes

Honza

Comment #3

Posted on Dec 10, 2010 by Happy Giraffe

That's definitely better than before.

Comment #4

Posted on Jan 13, 2011 by Massive Wombat

Fix for this issue is included in Firebug 1.7a8

Available here: http://getfirebug.com/releases/firebug/1.7X/firebug-1.7X.0a8.xpi

Please check out this version and let us know if it's working for you now.

Thanks for the help! Honza

Comment #5

Posted on May 4, 2011 by Happy Giraffe

Just changing label Test-case-in-suite to FBTest-available

Comment #6

Posted on Jun 5, 2012 by Helpful Wombat

(No comment was entered for this change.)

Status: Fixed

Labels:
1.6 ui dom Type-Enhancement fixed1.7a8 FBTest-available fixed-1.7-a8