Closed Bug 1842482 Opened 2 years ago Closed 2 years ago

Reimplement middle click "open message in a tab" action

Categories

(Thunderbird :: Folder and Message Lists, defect, P3)

Thunderbird 116

Tracking

(thunderbird_esr102 unaffected, thunderbird_esr115? fixed, thunderbird117? affected)

RESOLVED FIXED
118 Branch
Tracking Status
thunderbird_esr102 --- unaffected
thunderbird_esr115 ? fixed
thunderbird117 ? affected

People

(Reporter: dreadnaut, Assigned: solange)

References

(Blocks 1 open bug)

Details

(Keywords: regression, Whiteboard: [Supernova3p])

Attachments

(1 file)

Steps to reproduce:

In Thunderbird 102.x and earlier, clicking with the middle button on a folder or message selects the folder/message, and opens it in a background new tab.

Actual results:

In Thunderbird 116.0b1, clicking with the middle button has no effect: no selection, no new tab.

Expected results:

I would expect Supernova to implement the existing middle-click behaviour: select the folder/message, and open it in a new background tab.

The feature is particularly useful when the "Open messages in:" option is set to "A new message window": double-click will open in a new window, and middle-click in a new tab.

I can confirm this.

Severity: -- → S3
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(alessandro)
Keywords: regression
Whiteboard: [Supernova3p]
Flags: needinfo?(alessandro)
Priority: -- → P3
Assignee: nobody → solange

Just realised this is an almost-duplicate of bug 1836853, which mentions folders, but not messages.

Status: NEW → ASSIGNED

Windows 10
Confirmed issue in 116.0b3

Blocks: 1836853
Attachment #9343198 - Attachment description: WIP: Bug 1842482 - Supernova: Middle click on folder or message should open a tab. → Bug 1842482 - Supernova: Middle click on folder or message should open a tab. r=aleca,elizabeth
Attachment #9343198 - Attachment description: Bug 1842482 - Supernova: Middle click on folder or message should open a tab. r=aleca,elizabeth → Bug 1842482 - Reimplement middle click on message list to open tabs in background. r=aleca,elizabeth
Summary: Supernova: Middle click on folder or message should open a tab → Reimplement middle click "open message in a tab" action
Attachment #9343198 - Attachment description: Bug 1842482 - Reimplement middle click on message list to open tabs in background. r=aleca,elizabeth → WIP: Bug 1842482 - Reimplement middle click on message list to open tabs in background.
Attachment #9343198 - Attachment description: WIP: Bug 1842482 - Reimplement middle click on message list to open tabs in background. → WIP: Bug 1842482 - Reimplement middle click on message list to open tabs.
Attachment #9343198 - Attachment description: WIP: Bug 1842482 - Reimplement middle click on message list to open tabs. → Bug 1842482 - Reimplement middle click on message list to open tabs. r=aleca,elizabeth
Attachment #9343198 - Attachment description: Bug 1842482 - Reimplement middle click on message list to open tabs. r=aleca,elizabeth → WIP: Bug 1842482 - Reimplement middle click on message list to open tabs in background.
Duplicate of this bug: 1847883
Attachment #9343198 - Attachment description: WIP: Bug 1842482 - Reimplement middle click on message list to open tabs in background. → Bug 1842482 - Reimplement middle click on message list to open tabs in background. r=aleca,elizabeth

With the title change from "Middle click on folder or message should open a tab" to "Reimplement middle click on message list to open tabs in background", we have lost the "on folder" part of the issue.

It probably make sense to keep the two separate. Should I open a new bug for the "Reimplement middle click on folder to open tabs in background" ?

(In reply to dreadnaut from comment #6)

With the title change from "Middle click on folder or message should open a tab" to "Reimplement middle click on message list to open tabs in background", we have lost the "on folder" part of the issue.

It probably make sense to keep the two separate. Should I open a new bug for the "Reimplement middle click on folder to open tabs in background" ?

Thanks for the offer! There is already a bug tracking that issue so it shouldn't be needed.

Ah, search failed me — or I need to learn how to use it 😅

Thank you!

Target Milestone: --- → 118 Branch

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/e263d251112e
Reimplement middle click on message list to open tabs in background. r=aleca,elizabeth

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED

Comment on attachment 9343198 [details]
Bug 1842482 - Reimplement middle click on message list to open tabs in background. r=aleca,elizabeth

[Approval Request Comment]
Regression caused by (bug #):
User impact if declined: User would not be able to open messages on new tab using the middle click
Testing completed (on c-c, etc.): extensive manual and automated testing
Risk to taking this patch (and alternatives if risky): low

[Approval Request Comment]
Regression caused by (bug #):
User impact if declined: User would not be able to open messages on new tab using the middle click
Testing completed (on c-c, etc.): extensive manual and automated testing
Risk to taking this patch (and alternatives if risky): low

Attachment #9343198 - Flags: approval-comm-esr115?
Attachment #9343198 - Flags: approval-comm-beta?

Comment on attachment 9343198 [details]
Bug 1842482 - Reimplement middle click on message list to open tabs in background. r=aleca,elizabeth

[Triage Comment]
This will appear in 118 beta, and then uplifted to a point release after 115.2.0

Attachment #9343198 - Flags: approval-comm-beta? → approval-comm-beta-
Duplicate of this bug: 1847391
Duplicate of this bug: 1850546

Comment on attachment 9343198 [details]
Bug 1842482 - Reimplement middle click on message list to open tabs in background. r=aleca,elizabeth

[Triage Comment]
Approved for esr115

Attachment #9343198 - Flags: approval-comm-esr115? → approval-comm-esr115+

I recently updated Thunderbird to 115.2.2, and noticed that middle-click on a message now has an effect. However, it does not open a message in a new tab, but it follows the "Open messages in" option, under "Settings" -> "Reading & Display".

If the options is set to "A new tab", middle-clicking on a message will open it in a new tab, if it's set to "A new message window", a new window will pop up. This makes it no different than double clicking or hitting Enter.

I would expect middle-click to always open a message in a new tab, unaffected by the "Open messages in" setting. From this point of view, the issue is not fixed unfortunately.

Something else I noticed (if this is not expected behaviour, I can open a separate issue)

  • middle-click on a column header or on empty space opens the currently selected message
  • double-click on empty space opens the currently selected message

Using 118.0b4
Requirements:

  • Single left click on selected message to open in 'Message Pane' and have focus
  • Double click on selected message to open in A new message Window and have focus
  • Single middle click on selected email to open message in a new tab in background.

Settings

  • Enable 'Message Pane'
  • Set : 'Settings' > 'General' > 'Reading & Display' - select open messages in 'A new message window'

Actual:

  • Single click opens message in Message Pane and has focus
  • Double click opens message in a new Window with focus
  • Middle click opens message in a new Window with focus

Expected:

  • Single click opens message in Message Pane and has focus
  • Double click opens message in a new Window with focus
  • Middle click open selected email in a new tab in the background.

Confirm single middle-click will open in the area designated in 'Settings' > 'General' > 'Reading & Display'.
Single middle click should open in tab in background regardless of what is set up in the 'Reading & Display' settings.

This bug has not been fixed although it has made middle click operational, just not performing the expected result.

Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Summary: Reimplement middle click "open message in a tab" action → Reimplement middle click "open message in a tab in background" action

Please don't reopen bugs, and don't change the title of closed bugs, especially since patches have been already uplifted.
Report the wrong behaviour in a follow up bug blocked or regressed by the original bug.

Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED
Summary: Reimplement middle click "open message in a tab in background" action → Reimplement middle click "open message in a tab" action

(In reply to Alessandro Castellani [:aleca] from comment #18)

Please don't reopen bugs, and don't change the title of closed bugs, especially since patches have been already uplifted.
Report the wrong behaviour in a follow up bug blocked or regressed by the original bug.

Just to make sure I understand the process used on this forum:
If a bug is set as 'Resolved' but a bad patch was uplifted which did not fix bug then a new bug must be opened purely on the basis that a patch was uplifted on this bug.
You never reopen incorrectly marked as resolved bugs if any patch whether it fixed the bug or as in this case did not fix this bug.

Flags: needinfo?(alessandro)

As requested, I have opened a new bug 1853923

(In reply to Anje from comment #19)

If a bug is set as 'Resolved' but a bad patch was uplifted which did not fix bug then a new bug must be opened purely on the basis that a patch was uplifted on this bug.
You never reopen incorrectly marked as resolved bugs if any patch whether it fixed the bug or as in this case did not fix this bug.

We only reopen bugs if the patch gets backout, since the golden rule is to keep 1 patch per bug and if the patch lands the bug should reflect the correct state of the patch.
For all other cases, opening a follow up or regression is the recommended flow.

Flags: needinfo?(alessandro)

(In reply to Alessandro Castellani [:aleca] from comment #21)

(In reply to Anje from comment #19)

If a bug is set as 'Resolved' but a bad patch was uplifted which did not fix bug then a new bug must be opened purely on the basis that a patch was uplifted on this bug.
You never reopen incorrectly marked as resolved bugs if any patch whether it fixed the bug or as in this case did not fix this bug.

We only reopen bugs if the patch gets backout, since the golden rule is to keep 1 patch per bug and if the patch lands the bug should reflect the correct state of the patch.
For all other cases, opening a follow up or regression is the recommended flow.

Many thanks Alessandro. I understand.

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: