121182 - Can't copy values from computed style

Status: RESOLVED FIXED

(Other Applications :: DOM Inspector, enhancement)

Description

[timeless]

run inspector
inspect navigator
dig down to 
window>toolbox>xul:vbox>#navbar>xul:hbox>#navbar-buttons>toolbarbutton
select view computed style
select -moz-binding
try to edit>copy
try to right-click>copy
try to ctrl-c [copy]

expect the style and value to be copied to the clipboard.

bonus points: copy the pair in a tabular/collumn format for html, and tab 
separated for text/plain.

Comment 1

[Christopher Aillon (sabbatical, not receiving bugmail)]

I think it would be better to copy these values in a format ready for dropping
into a CSS rule.

Eg: following the steps in comment 0, the following would be on the clipboard:

   -moz-binding: url(chrome://foo/bar.xml#binding);

I can see having more practical uses for that, since I can copy a value that I
haven't specified yet, add it to my document's specified stylesheet, make
modifications to its specified value, and see what effect ot has on the original
document.

Comment 2

[Christopher Aillon (sabbatical, not receiving bugmail)]

Mass re-assigning bugs to [e-mail]

Comment 3

[Cedric. S.]

And in addition to copying one rule, be able to copy a selection of multiple
rules (ready de drop into CSS).

One practicel use of such feature would be to facilitate the developpment of
theme-independant stylesheets for applications : develop one the basis of one
theme, copy all the resulting CSS rules : your styles won't change if the theme
changes.

Comment 4

[Alex Vincent [:WeirdAl]]

Show me how to properly utilize the clipboard, and I'll probably be interested
in fixing this.

Comment 5

[Christian :Biesinger (don't email me, ping me on IRC)]

http://lxr.mozilla.org/seamonkey/source/widget/public/nsIClipboardHelper.idl ?

Comment 6

[Jason Barnabe (np)]

Attachment: patch v1

This copies the declarations to the clipboard in the format described in comment 1. It supports multiple selection; each declaration will be seperated by a newline.

Comment 7

[timeless]

Comment on attachment 220482 [details] [diff] [review]
patch v1

You could get fancy and have
function InCSSDecl(prop, val)
{
  this.property=prop;
  this.value=val;
}
InCSSDecl.prototype.toString =
function()
{
  return this.property + ": " + this.value + ";";
}

and then
return new InCSSDecl(view.getCellText(aIndex, {id: "olcStyleName"}), 
                     value: view.getCellText(aIndex, {id: "olcStyleValue"});

and later:
viewer.pane.panelset.setClipboardData(copiedDeclarations, "inspector/css-declarations", 
copiedDeclarations.join("\n"));

Note: some of your lines have trailing whitespace, it'd be nice if that didn't make it into cvs (or future patches ;-).

Comment 8

[Jason Barnabe (np)]

Attachment: patch v2

More fancy, less whitespace.

Comment 9

[[e-mail]]

Comment on attachment 221937 [details] [diff] [review]
patch v2

>+    var rangeCount = this.mTree.view.selection.getRangeCount();

>+      this.mOlBox.view.selection.getRangeAt(i,start,end);

Please consistently use this.mTree.view throughout. sr=me with this fixed.

Comment 10

[Jason Barnabe (np)]

Attachment: patch v3

All uses of mOlBox removed.

Comment 11

[Jason Barnabe (np)]

timeless checked this in
	mozilla/extensions/inspector/resources/content/viewers/computedStyle/computedStyle.js 1.12
mozilla/extensions/inspector/resources/content/viewers/computedStyle/computedStyle.xul 1.13

Comment 12

[[e-mail]]

>>+      this.mOlBox.view.selection.getRangeAt(i,start,end);
[Doh, this should have been (i, start, end); next time, Inspector, next time]

Comment 13

[Ed Hume]

As a themer, I want to thank you for adding this function.
Thank you. Thank you. Thank you.

The multiline block select works: I can select several statements in a row. Also, I can use ctrl-click and select non-contiguous lines, and copy them (!).

For "next time":

When pasting a multiline selection, the line break does not work in Notepad (a Windows accessory) but does work in Microsoft Word and Wordpad.

Although the context-menu copy function works, the ctrl-C keyboard function does not work. That would be a lovely addition.

But this is wonderful as it is.

Comment 14

[Ed Hume]

and this function should be added to Bon Echo when the branch is reopened.

Comment 15

[Jason Barnabe (np)]

(In reply to comment #12)
> >>+      this.mOlBox.view.selection.getRangeAt(i,start,end);
> [Doh, this should have been (i, start, end); next time, Inspector, next time]
I copied that line from somewhere else, so you could say it's the prevailing style :)
(In reply to comment #13)
> Although the context-menu copy function works, the ctrl-C keyboard function
> does not work. That would be a lovely addition.
Works for me.