158593 - [RFE] It should be posible to drag items from the media tab and the links tab in page info.

Status: RESOLVED FIXED

(SeaMonkey :: Page Info, enhancement)

Description

[Nicolás Lichtmaier]

It should be posible to start a drag from the media and links tabs to (e.g.)
drop them into an open tab, or create a shortcut to that URL.

Comment 1

[Nicolás Lichtmaier]

Attachment: Patch

Comment 2

[Daniel Brooks [:db48x]]

Comment on attachment 92167 [details] [diff] [review]
Patch

I like it. Just break the function up into smaller chunks with some blank
lines, and it'll be cool. I'll have to test it tommorrow though.

Comment 3

[Nicolás Lichtmaier]

Attachment: Broken in chunks

Comment 4

[Nicolás Lichtmaier]

Note that the XUL part of the patch has a line which depends in other patch, the
one in bug 158245 (which has already been reviewed and super-reviewed and is
waiting for approval =/).

But it's easy to fix, this patch just need to add that ondraggesture to the <tree>.

Comment 5

[Daniel Brooks [:db48x]]

Comment on attachment 92178 [details] [diff] [review]
Broken in chunks

thanks, that's really the only 'style' issue I care about.

Just to make sure, do you have to cal addDataFlavor(), doesn't it add the
flavor automagically if needed when you call setTransferData()?

Comment 6

[Johnny Stenback (:jst)]

Comment on attachment 92178 [details] [diff] [review]
Broken in chunks

+function onBeginLinkDrag(event,urlField,descField)
+{
+  if (event.originalTarget.localName != "treechildren")
+    return;
+  var tree = event.target;
+  if(!("treeBoxObject" in tree))
+    tree=tree.parentNode;

Be consistent with how you use space between "if" and the opening paren, I'd
suggest you always insert the space, but if the majority of the code in this
file doesn't use a space then don't insert the space, but be consistent!

+  if ( !transArray )
+    return;

Also be consistent with how you use spaces inside the parens in if expressions
(and function calls too, of course). I'd suggest you loose the spaces, i.e.:

  if (!transArray)

...
+  if ( !trans )
+    return;

Same thing.

+  trans.addDataFlavor("text/x-moz-url");
...
+  trans.setTransferData ("text/x-moz-url", stringURL, stringURL.data.length *
2 );

Again, be consistent with your usage of space before the opening paren, loose
the space on the above line.

sr=jst if you clean up those nits.

Comment 7

[Johnny Stenback (:jst)]

Oh, and be consistent with space-after-comma too in argument lists.

Comment 8

[Nicolás Lichtmaier]

Attachment: Addes and removed spaces

Comment 9

[Nicolás Lichtmaier]

Attachment: interface names changes (wstring -> string)

Comment 10

[Pete Zha]

Attachment: fix contractID&interface problem for current codebase

Also, I add some return false to avoid the warning of the js engine

Comment 11

[Daniel Brooks [:db48x]]

Comment on attachment 95050 [details] [diff] [review]
fix contractID&interface problem for current codebase

personally I wouldn't have cared about the warning, that one is often
meaningless.

Anyway, r=db48x

Comment 12

[Nicolás Lichtmaier]

This was commited long ago! =)