Closed Bug 1827218 Opened 2 years ago Closed 2 years ago

Navigate in standalone message window using next/previous `B` and `P` shows to incorrect message

Categories

(Thunderbird :: Search, defect)

Thunderbird 112
defect

Tracking

(thunderbird_esr102 unaffected, thunderbird114 fixed)

RESOLVED FIXED
115 Branch
Tracking Status
thunderbird_esr102 --- unaffected
thunderbird114 --- fixed

People

(Reporter: dmccammishjr, Assigned: darktrojan)

References

(Blocks 1 open bug)

Details

(Keywords: regression, Whiteboard: [Supernova3p])

Attachments

(2 files)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:109.0) Gecko/20100101 Firefox/111.0

Steps to reproduce:

Created search of trash with subject and date. Selected last in results list, Opened mail. Used "b" key to view previous message. Selected item from center of list, used "f" key to view next item.

Actual results:

Previous (b) and next (f) Returned non-related mail from trash.

Expected results:

Previous & next should return from search list.

Component: Untriaged → Search
Summary: Thunderbird 112 .0b7 search list next & last not from list → Thunderbird 112.0b7 search list next & last not from list
Blocks: sn-msglist
Keywords: regression
Summary: Thunderbird 112.0b7 search list next & last not from list → Thunderbird 112.0b7 : navigate next/previous jumps to unrelated items
Whiteboard: [Supernova]

Not working in 113.0b1 (Win 11 64 bit )

Whiteboard: [Supernova] → [Supernova3p]

what folder modes do you have enabled?

Flags: needinfo?(dmccammishjr)

I don't know anything about Folder Modes. My standard layout is list of accounts in left pane with folders under each account name. Some accounts are collapsed, some expanded. Message list (currently open folder) in right pane. Sometimes other things in tabbed right pane. All messages open in separate window.

Flags: needinfo?(dmccammishjr)

I agree, order of messages returned is strange. I tested using a random saved search, not on trash folder. The messages being displayed by F and B are from some order other than what is shown in the message list - at least for me. But I don't know what the order is.

I don't use this type of key navigation, so I'm not in a great position to know past/correct behavior.

Does it happen for folders other than Trash?
Does it happen in version 102?
The messages shown are correct when not in the standalone message window?

Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(dmccammishjr)
Summary: Thunderbird 112.0b7 : navigate next/previous jumps to unrelated items → Navigate in standalone message window using next/previous `B` and `P` shows to incorrect message

Created search of trash with subject and date. Selected last in results list, Opened mail. Used "b" key to view previous message. Selected item from center of list, used "f" key to view next item.

I didn't follow this precise STR. I simply opened a message in the middle of the list, and did P and B, the message shown in standalone window was almost never what I expected. I tried message list ordered by date and also by location.

Attached image SearchT1.png (deleted) —
Flags: needinfo?(dmccammishjr)

See screenshot. Searched for Correspondent containing "face" - got list shown. Selected and opened one at random. "b" moved to previous mail in folder by DATE (not previous in search results). "f" moved to next mail in folder by DATE (not next in search results).
TB 113.0b6 on Win 11, 64 bit.

Tried search in a different folder in a different account with same result. Tried b & f in a random account (sorted by date) and they worked as expected (i.e., next & prev by date). Seems to be a problem within the search results.

We haven't been passing on the view wrapper to about:message, so it has been creating its own using the folder of the message displayed. This is wrong if we weren't looking at that folder.

Assignee: nobody → geoff
Status: NEW → ASSIGNED

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/fd56e3de5a77
Use existing view wrapper where possible when displaying a message. r=leftmostcat

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 115 Branch
Attachment #9333562 - Flags: approval-comm-beta?

Comment on attachment 9333562 [details]
Bug 1827218 - Use existing view wrapper where possible when displaying a message. r=#thunderbird-reviewers

[Triage Comment]
Approved for beta

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

Attachment

General

Creator:
Created:
Updated:
Size: