Closed Bug 1775208 Opened 3 years ago Closed 3 years ago

Pasting link copied from "Copy Summary" button in bug pages into composer inserting only URL as plain text

Categories

(Thunderbird :: Message Compose Window, defect)

Thunderbird 102
x86_64
All
defect

Tracking

(thunderbird_esr102 fixed, thunderbird102+ affected)

RESOLVED FIXED
103 Branch
Tracking Status
thunderbird_esr102 --- fixed
thunderbird102 + affected

People

(Reporter: masayuki, Assigned: mkmelin)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression)

Attachments

(1 file)

STR

  1. Open the mail composer in HTML mode
  2. Click "Copy Summary" button in a bug page in Firefox
  3. Paste it into the composer

Expected Result

A link should be inserted, its text is the bug number and bug summary, and its link target is the bug's URL.

Actual Result

The URL is pasted as plain text.

I can reproduce this in the trouble shooting mode too.

With mozregression, I got a regression range, but it does not make sense...

I'm not sure whether is a bug of the editor module (pasting side), the code copying to the clipboard or the composer window.

I don't reproduce this bug if I copy link in Firefox's web content.

The regression is probably bug 1572648's https://hg.mozilla.org/comm-central/rev/e487599c0b4f0bfcaf12513800eed54d142a391e since that modifies how we handle stuff with an html data transfer when we're not offline. Going offline fixes the behavior, since we then don't use the "plain link" code path.

Regressed by: 1572648
Version: unspecified → Thunderbird 102
Blocks: tb102found
Severity: -- → S3
OS: Windows 10 → All
Assignee: nobody → mkmelin+mozilla
Status: NEW → ASSIGNED
Target Milestone: --- → 103 Branch

Pushed by alessandro@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/abee77603a0b
Fully disable link preview handling if not enabled, and don't use it if HTML is already available to paste. r=henry

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

Comment on attachment 9282332 [details]
Bug 1775208 - Fully disable link preview handling if not enabled, and don't use it if HTML is already available to paste. r=henry

[Approval Request Comment]
Regression caused by (bug #):
User impact if declined:
Testing completed (on c-c, etc.): On 103.0b1
Risk to taking this patch (and alternatives if risky):

Attachment #9282332 - Flags: approval-comm-esr102?

Comment on attachment 9282332 [details]
Bug 1775208 - Fully disable link preview handling if not enabled, and don't use it if HTML is already available to paste. r=henry

[Triage Comment]
Approved for esr102

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

Attachment

General

Created:
Updated:
Size: