Closed Bug 1301248 Opened 8 years ago Closed 4 months ago

Preserve login selection after sorting in the password manager

Categories

(Thunderbird :: Preferences, defect)

defect

Tracking

(thunderbird_esr115- wontfix)

RESOLVED FIXED
125 Branch
Tracking Status
thunderbird_esr115 - wontfix

People

(Reporter: MattN, Assigned: welpy-cw)

References

Details

Attachments

(1 file)

In bug 1288558 it was noticed that there is dead code in SortTree to restore the selection after sorting.

This code should be enabled as it seems like the expected UX on Windows and OS X but a test should be written for it.

> if (selectedNumber >= 0 && false) {
See Also: → 1466832
Component: Password Manager → Preferences
Priority: P3 → --
Product: Toolkit → Thunderbird
Whiteboard: [passwords:management]
Severity: normal → S4
Summary: Preserve login selection after sorting in the manager → Preserve login selection after sorting in the password manager

Remember the last selected item when changing view sorting, or showing/hiding passwords in an
unfiltered view.

Depends on D204289

Assignee: nobody → h.w.forms
Status: NEW → ASSIGNED
Target Milestone: --- → 125 Branch

Pushed by daniel@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/a701dd3d1ebe
Refactor selection state preservation in the password manager. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED

Comment on attachment 9391064 [details]
Bug 1301248 - Refactor selection state preservation in the password manager. r=mkmelin

[Approval Request Comment]
See bug 1884661 comment 3

Attachment #9391064 - Flags: approval-comm-esr115?

Comment on attachment 9391064 [details]
Bug 1301248 - Refactor selection state preservation in the password manager. r=mkmelin

[Triage Comment]
Thanks for the patch.
Declining uplift because this isn't a severe problem.

Attachment #9391064 - Flags: approval-comm-esr115? → approval-comm-esr115-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: