883485 - Allow deletion of news posts by default (with a prompt!)

Status: RESOLVED FIXED

(MailNews Core :: Database, enhancement)

Description

[Jim Porter (:squib)]

Attachment: Patch

Thanks to jcranmer, we've had the ability to delete the local copies of news posts for some time. However, it's been locked behind a hidden pref. Now that we have a confirmation prompt when shift-deleting messages, we can expand that logic a bit to account for deleting news posts as well.

This means that, with the exception of "Junk", the buttons in the message header for news actually match those for mail!

Comment 1

[Jim Porter (:squib)]

Attachment: Proper patch

Whoops, I accidentally posted an interdiff instead of the proper diff!

Comment 2

[Jim Porter (:squib)]

Just a note: this doesn't actually *require* any l10n changes (in fact, I'm not sure we even want the string I added), so we could theoretically land this during aurora.

Comment 3

[Joshua Cranmer [:jcranmer]]

Comment on attachment 763064 [details] [diff] [review]
Proper patch

Review of attachment 763064 [details] [diff] [review]:
-----------------------------------------------------------------

I don't think there's much of a difference to having the separate string for the "post" versus "message", considering that we don't use a lot of "post" terminology in mailnews.

Also, next time, please try to remember to use -p when making patches. :-)

Comment 4

[Jim Porter (:squib)]

Attachment: Remove new string

This removes the new string (but renames the old shift-delete one since the context changed). I'm posting this here instead of just pushing because the Seamonkey tree is closed.

Comment 5

[Jim Porter (:squib)]

Landed: https://hg.mozilla.org/comm-central/rev/0eca3fec431a