737115 - Remove deprecated dnd methods from feed subscribe dialog, several enhancements

Status: RESOLVED FIXED

(MailNews Core :: Feed Reader, defect)

Comment 1

[alta88]

Attachment: patch.

1. remove com interfaces and nsDragAndDrop in favor of getting everything from the event.dataTransfer.
2. allow copy of feeds to other accounts.
3. try harder to get valid urls; support dnd from chrome/safari links (ie8/9 dnd doesn't have the info in dt though).  for both folderpane and dialog.

Comment 3

[alta88]

Attachment: patch.

tweak to make a reusable func.

Comment 4

[David :Bienvenu]

Comment on attachment 607995 [details] [diff] [review]
patch.

this seems to work ok. My one question is about imap - I was able to drag an rss folder to an imap account once, and the messages were copied, but then I couldn't actually read the messages. When I tried to recreate the issue, I was unable to copy the rss feed to an imap account because creating the folder failed because of hierarchy delimiter issues (we were using ^ instead of . for some reason). The code w/o your patch failed the same way, so I'm not going to minus the patch for that. But, would you expect feeds to work if they're in imap accounts? Is drag&drop of feeds to imap accounts useful?

Comment 5

[David :Bienvenu]

oh, I should say, the patch has bit-rotted a little bit - feed-subscriptions.xul doesn't quite apply.

Comment 6

[alta88]

Attachment: unbitrotted patch for checkin.

Comment 7

[alta88]

(In reply to David :Bienvenu from comment #4)
> Comment on attachment 607995 [details] [diff] [review]
> patch.
> But, would you expect feeds to work
> if they're in imap accounts? Is drag&drop of feeds to imap accounts useful?

currently a message is a 'feed' only if its folder.server.type == "rss", otherwise it's subject to remote content and the web page won't be loaded.  but, the mbox (summary) should be viewable no differently in imap than in any other server type, ie local folders etc.

'feedness' should be a property of the message not its folder, bug 522645.  now that bug 647699 has been recently fixed, a property can be added to do this, going forward.

Comment 8

[alta88]

btw, the 'copy feeds to other accounts' meant the feed subscription in the Subscribe dialog (not folder/messages in folderpane).

the folderpane is extremely confusing, even to me.  a 'feed' is not the folder.  a folder in a feed account may have 0, 1, or more than 1 feed subscriptions (just a folder property).  and the user can't tell without looking in the Subscribe dialog.

Comment 9

[Ryan VanderMeulen [:RyanVM]]

http://hg.mozilla.org/comm-central/rev/43c6ccec3b1e

BTW, I had a lot of difficulty getting your patch to apply due to bad hunks. Please make sure you're using the settings shown in the article below for the patches you submit.
https://developer.mozilla.org/en/Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F