Closed Bug 805446 Opened 12 years ago Closed 12 years ago

<Delete> key in Feed Subscriptions window does not work as expected

Categories

(MailNews Core :: Feed Reader, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 19.0

People

(Reporter: trinsnet, Assigned: alta88)

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20100101 Firefox/16.0
Build ID: 20121010144125

Steps to reproduce:

Go to Account settings > Account Actions > Add Other Account > choose Blogs and News Feeds > Next, Next, Finish with defaults > choose to Manage Subscriptions in Blogs and News Feeds account > enter a valid feed URL
(e.g. http://rss.slashdot.org/Slashdot/slashdot )
You are now in the Feed Subscriptions window:
Attempt to edit the Title field by clicking on it and pressing <Delete> to clear the title.


Actual results:

You are prompted if you would like to unsubscribe from the feed. This dialog should not appear in this situation.
If you say 'no' to the dialog box, the title field is then cleared as it should be.


Expected results:

The title field should be cleared and you should be able to type a new title for the feed.

You can work around this by closing the Feed Subscription window and renaming the subscription by right clicking on it.

Either the title field in Feed Subscription window should not be editable or the <delete> key should not attempt to delete the feed when editing the title field.
Status: UNCONFIRMED → NEW
Component: Untriaged → Feed Reader
Ever confirmed: true
OS: Linux → All
Product: Thunderbird → MailNews Core
Hardware: x86_64 → All
Attached patch patchSplinter Review
yeah, the del key is meant only if the feed is selected *and* focused in the tree..

patch also includes ensuring valid feed urls yet with 0 items cases are handled.
Assignee: nobody → alta88
Attachment #675217 - Flags: review?(mconley)
Comment on attachment 675217 [details] [diff] [review]
patch

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

Looks good - thanks alta88!
Attachment #675217 - Flags: review?(mconley) → review+
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/a5bf6bfcce18
Status: NEW → RESOLVED
Closed: 12 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 19.0
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: