1917485 - Using the context menu in the message list on a non-selected messages scrolls the list to the selected message after completion of the command

Status: RESOLVED FIXED

(Thunderbird :: Folder and Message Lists, defect)

Description

[Francesco]

Steps to reproduce:
Have a folder with more messages than fit onto the screen.
Select the first one.
Scroll to a message further down the list so the selected first message scrolls out of view.
Do an operation on a message without selecting it via the context menu, for example, mark it (un)read, add/remove a tag, etc.

Actual behaviour since TB 115: After completion of the action, the message list scrolls to the selected message.

Expected behaviour as in TB 102: List position is maintained so the user can apply another action to the same message or a message close by in the list.

Comment 1

[Hartmut Welpmann [:welpy-cw]]

Attachment: Bug 1917485 - Do not scroll the message list when releasing a temporary selection. r=#thunderbird-reviewers

Comment 2

[Francesco]

As comment 0 said, this went broken with Supernova.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/e19bcc617fe0
Do not scroll the message list when releasing a temporary selection. r=tobyp

Comment 4

[Hartmut Welpmann [:welpy-cw]]

Comment on attachment 9423866 [details]
Bug 1917485 - Do not scroll the message list when releasing a temporary selection. r=#thunderbird-reviewers

[Approval Request Comment]
Regression caused by (bug #): (Supernova)
User impact if declined: There may be unexpected scrolling after using the context menu on an unselected message
Testing completed (on c-c, etc.): c-c and beta
Risk to taking this patch (and alternatives if risky): very low

Comment 5

[Hartmut Welpmann [:welpy-cw]]

Uplifting bug 1903910 as well would make it possible to apply this to comm-esr128 without rebasing,

Comment 6

[Corey Bryant]

Comment on attachment 9423866 [details]
Bug 1917485 - Do not scroll the message list when releasing a temporary selection. r=#thunderbird-reviewers

[Triage Comment]
Approved for esr128
(but needs bug 1903910)

Comment 7

[Rob Lemley [:rjl]]

This doesn't merge into comm-esr128 due to changes in 1903910.

Comment 8

[Rob Lemley [:rjl]]

Corey, ok to take bug 1903910?

Comment 9

[Corey Bryant]

(In reply to Rob Lemley [:rjl] from comment #8)

Corey, ok to take bug 1903910?

Let's hold off for now. I think there were some concerns with uplifting 1903910.

Comment 10

[Hartmut Welpmann [:welpy-cw]]

Attachment: D221641.esr128.diff

This is the rebased patch for ESR128.

Comment 11

[Rob Lemley [:rjl]]

Thunderbird 128.3.3esr:
https://hg.mozilla.org/releases/comm-esr128/rev/f1f400a02422