Last Comment Bug 606992 - Add ability to drag feed URL from header pane
: Add ability to drag feed URL from header pane
Status: RESOLVED FIXED
:
Product: SeaMonkey
Classification: Client Software
Component: MailNews: Message Display (show other bugs)
: Trunk
: All All
: -- enhancement (vote)
: seamonkey2.1b2
Assigned To: Jens Hatlak (:InvisibleSmiley)
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-10-25 09:45 PDT by Jens Hatlak (:InvisibleSmiley)
Modified: 2010-11-08 15:14 PST (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
patch (1.56 KB, patch)
2010-11-06 16:21 PDT, Jens Hatlak (:InvisibleSmiley)
neil: superreview+
Details | Diff | Review
patch v1a [Checkin: comment 5] (1.58 KB, patch)
2010-11-08 10:12 PST, Jens Hatlak (:InvisibleSmiley)
mnyromyr: review+
jh: superreview+
Details | Diff | Review

Description Jens Hatlak (:InvisibleSmiley) 2010-10-25 09:45:53 PDT
Currently you can only click the URL of a feed (which appears in the header pane) or use the context menu to copy it. It would be nice if you could also drag and drop it, e.g. to a browser window (new or existing tab), the desktop, the Bookmark Manager or the Bookmark sidebar (just to name some examples).
Comment 1 Jens Hatlak (:InvisibleSmiley) 2010-11-06 16:21:17 PDT
Created attachment 488710 [details] [diff] [review]
patch

Cf. https://developer.mozilla.org/En/DragDrop/Recommended_Drag_Types (Dragging Links):
- text/x-moz-url is needed to allow dragging to the Personal Toolbar, Bookmark Manager and Bookmarks sidebar
- text/uri-list alone allows dragging to a browser window
- text/plain allows dragging to a text editor (tested SciTE).
Comment 2 neil@parkwaycc.co.uk 2010-11-08 04:50:28 PST
Comment on attachment 488710 [details] [diff] [review]
patch

>+                 ondragstart="setDataTransfer(event);"
Nit: prefer this.parentNode.setDataTransfer rather than relying on our weird XUL event scoping rules.
Comment 3 Jens Hatlak (:InvisibleSmiley) 2010-11-08 10:12:03 PST
Created attachment 488899 [details] [diff] [review]
patch v1a [Checkin: comment 5]
Comment 4 Karsten Düsterloh 2010-11-08 14:32:45 PST
Comment on attachment 488899 [details] [diff] [review]
patch v1a [Checkin: comment 5]

Cool!
Comment 5 Jens Hatlak (:InvisibleSmiley) 2010-11-08 15:13:44 PST
Comment on attachment 488899 [details] [diff] [review]
patch v1a [Checkin: comment 5]

http://hg.mozilla.org/comm-central/rev/1634e5c4c0bc

Note You need to log in before you can comment on or make changes to this bug.