Make removing entries in Data Manager trees select the next entry

RESOLVED FIXED in seamonkey2.1b2

Status

SeaMonkey
Passwords & Permissions
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Robert Kaiser, Assigned: Robert Kaiser)

Tracking

(Blocks: 1 bug)

Trunk
seamonkey2.1b2
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
From bug 569341 (Ian's comment, my reply, his reply):
> > When you use the delete key to remove cookies, the next cookie in the list is
> > not selected and you can not use the delete key until you select another
> > cookie.
> 
> Hmm, interesting idea, might really be helpful. Can I pass that into a
> followup, please?
I guess so, but less obvious candidate than first one.



When done, this should be done for all tree views in the Data Manager.
(Assignee)

Updated

7 years ago
Depends on: 569341
(Assignee)

Updated

7 years ago
Version: SeaMonkey 2.0 Branch → Trunk

Updated

7 years ago
Blocks: 599097
(Assignee)

Comment 1

7 years ago
Created attachment 489626 [details] [diff] [review]
update selection after deleting

Here's a patch that updates the selection after delete on all the trees we have in Data Manager and also updates the test to not fail and additionally check for the correct results from this patch.
Assignee: nobody → kairo
Status: NEW → ASSIGNED
Attachment #489626 - Flags: review?(iann_bugzilla)

Comment 2

7 years ago
Comment on attachment 489626 [details] [diff] [review]
update selection after deleting

Lots of similar code, but as discussed on IRC not quite worth a helper function (yet) r=me
Attachment #489626 - Flags: review?(iann_bugzilla) → review+
(Assignee)

Comment 3

7 years ago
Landed as http://hg.mozilla.org/comm-central/rev/4c4f1bcda382
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b2
You need to log in before you can comment on or make changes to this bug.