Closed Bug 1703501 Opened 4 years ago Closed 3 years ago

Can only move an account one step at a time by keyboard

Categories

(MailNews Core :: Account Manager, defect)

Thunderbird 88
defect

Tracking

(thunderbird_esr91+ verified)

VERIFIED FIXED
92 Branch
Tracking Status
thunderbird_esr91 + verified

People

(Reporter: freaktechnik, Assigned: freaktechnik)

References

Details

Attachments

(1 file)

Currently, the account manager gets reloaded every time the accounts are reordered. This leads to the focus being lost and thus only one move is possibly with the keyboard shortcut.

Magnus pointed out that it also leads to the account list getting scrolled to the top, which can also be confusing in the drag'n'drop case.

The function that reloads the account manager is https://searchfox.org/comm-central/rev/36bc68ce1085a077c3e676153be5c30163a09d9e/mailnews/base/prefs/content/AccountManager.js#878-890

I'm not sure if the reload is done to avoid any state inconsistencies, since currently the function is only called when the order is changed from within the account manager's own tree.

Martin, can you check if we can improve this?

Assignee: nobody → martin
Severity: -- → S3
Version: unspecified → Thunderbird 88

Moving accounts by keyboard does not work correctly after adding items, even though the code tries to do the right thing, the nodes end up in the wrong places.

Status: NEW → ASSIGNED

Pushed by richard.marti@gmail.com:
https://hg.mozilla.org/comm-central/rev/2e2c9f4b952a
Allow moving an account multiple steps by keyboard. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 92 Branch

This has been on beta for a while, so time for the backport? Lots of other bugs already had ESR91 approved.

Flags: needinfo?(martin)
Flags: needinfo?(mkmelin+mozilla)

Leaving for patch author to request uplift.

Flags: needinfo?(mkmelin+mozilla)

Comment on attachment 9231451 [details]
Bug 1703501 - Allow moving an account multiple steps by keyboard. r=mkmelin

[Approval Request Comment]
Regression caused by (bug #):
User impact if declined: Can only move an account one step by keyboard before having to reacquire focus of it.
Testing completed (on c-c, etc.): automated test, nightly, beta
Risk to taking this patch (and alternatives if risky): low, worst case the wrong account is selected or an account manager tree might be out of sync of the account order.

Flags: needinfo?(martin)
Attachment #9231451 - Flags: approval-comm-esr91?

Comment on attachment 9231451 [details]
Bug 1703501 - Allow moving an account multiple steps by keyboard. r=mkmelin

[Triage Comment]
Approved for esr91

Attachment #9231451 - Flags: approval-comm-esr91? → approval-comm-esr91+

Verified testing the 91.1.0 release candidate on Windows 10.

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: