Closed Bug 668491 Opened 13 years ago Closed 6 months ago

"Open in conversation" doesn't work when the thread is collapsed, because because Gloda only trusts References: and In-Reply-To headers

Categories

(Thunderbird :: Mail Window Front End, enhancement)

All
Other
enhancement

Tracking

(thunderbird_esr115 fixed)

RESOLVED DUPLICATE of bug 1860539
Tracking Status
thunderbird_esr115 --- fixed

People

(Reporter: simendsjo, Assigned: protz)

References

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/534.30 (KHTML, like Gecko) Chrome/12.0.742.100 Safari/534.30

Steps to reproduce:

Press Ctrl+Shift+O on a collapsed thread (the menu item is disabled too)


Actual results:

Nothing


Expected results:

Should have opened the thread in a new tab as it does when the thread is expanded
Component: General → Mail Window Front End
QA Contact: general → front-end
Status: UNCONFIRMED → NEW
Ever confirmed: true
This would cause issues if you have mail.strict_threading set to false. Consider messages:
- Message A, Subject = Foo
- Message B, Subject = Re: Foo, *no references header*

The thread pane will thread message A and message B together because they share the same subject, but Gloda will fail to figure out these messages are related, because Gloda only trusts References: and In-Reply-To headers. Then, hitting "Show in conversation" on the collapsed thread will take the first message, run a gloda query, and display the results of the query in a new tab, that is, message A only.

This will be slightly confusing, but not much more confusing than what happens already if you do show in conversation when the thread is expanded. Plus, I'm pretty sure we have no UI for these options.

Thoughts?
Well I guess it's a worthwhile addition, and since the prefs that might lead to weird behaviors are hidden, I guess it's worth having.
Assignee: nobody → jonathan.protzenko
Status: NEW → ASSIGNED
Attachment #547469 - Flags: review?(squibblyflabbetydoo)
Note that this doesn't actually show the menu item, it just enables the keyboard shortcut and the Message > Open In Conversation menu entry. I guess I could make sure the menu entry is shown but that probably requires a ui-r and a strings change.
Blake, you might want to voice your opinion on the issue. Enabling the menu item would show a "open message in conversation" menu item while a collapsed thread is shown. This can be confusing, and we might need to turn it into a "open thread in conversation", possibly with a separate menu item. This is much more involved than the patch I submitted, and I'm not sure it's worth it, when you can simply expand the thread and then do "open in conversation"...
Comment on attachment 547469 [details] [diff] [review]
Always enable open in conversation

Review of attachment 547469 [details] [diff] [review]:
-----------------------------------------------------------------

The patch works for a single thread, but behaves strangely when multiple threads are selected. "Open in Conversation" is enabled in that case, yet trying to open all the thread in a conversation only opens the first thread. Bug 550775 discussed some ways to handle it (I think we should open a single conversation tab with all the conversations listed). I think this would entail modifying the gConversationOpener object and possibly the conversation tab implementation to handle multiple conversations at once.

I also think we should show the context menu item for "Open in Conversation" when multiple messages are selected, but that's pretty easy to do.
Attachment #547469 - Flags: review?(squibblyflabbetydoo) → review-
I think "Open in Conversation" should be enabled when a thread is selected, too.  How much more work are we talking about?

(Also, similarly to the way "Open Message" opens one tab per message, I think "Open in Conversation" should open one tab per conversation.  As a side benefit, that should be easier to do.  ;)

Thanks,
Blake.
This should be revisited as 546621 bubbles up
See Also: → 953122
Depends on: 546621
Hi, 

I was about to open a feature request for exactly what this topic address.
What is the situation now, this topic hasn't been updated in two years.

The functionality asked would be greatly appreciated.

Cheers
Type: defect → enhancement
See Also: → 546621
Summary: "Open in conversation" doesn't work when the thread is collapsed → "Open in conversation" doesn't work when the thread is collapsed, because because Gloda only trusts References: and In-Reply-To headers

(In reply to jbonlinea from comment #8)

Hi,

I was about to open a feature request for exactly what this topic address.
What is the situation now, this topic hasn't been updated in two years.

The functionality asked would be greatly appreciated.

Cheers

Hello,

I second this thought. I often find myself wanting to open a collapsed thread directly in a conversation tab.
I know it's possible to expand the thread and then do Ctrl+Shift+O, but being able to use the shortcut immediately just feels more fluent.
It's been 2 years since the last update, so I was wandering if this is still being considered.

Thanks,
Have a nice one

Severity: normal → S3

While this was already working due to a regression in 115 ESR, it is now explicitly fixed by bug 1860539.

Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Duplicate of bug: 1860539
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: