Closed Bug 1658114 Opened 4 years ago Closed 4 years ago

Order of file attachments cannot be changed by drag and drop

Categories

(Thunderbird :: Message Compose Window, defect)

defect

Tracking

(thunderbird_esr78 fixed, thunderbird80 fixed)

VERIFIED FIXED
81 Branch
Tracking Status
thunderbird_esr78 --- fixed
thunderbird80 --- fixed

People

(Reporter: karenacollins, Assigned: khushil324)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/84.0.4147.105 Safari/537.36

Steps to reproduce:

With multiple file attachments, attempts to reorder the attachments with a mouse drag and drop fail to change the attachment order. The attachment order insert marker moves to the dragged location as expected, but when the mouse is released, the attachment order remains as it was originally.

Actual results:

The attachment order stayed the same.

Expected results:

The attachment order should have changed.

Good find, Karen, thank you!
Confirming on win10, 78.1.1 (64-bit).

Status: UNCONFIRMED → NEW
Component: Untriaged → Message Compose Window
Ever confirmed: true

In the meantime, please use Reorder Attachments from context menu and/or keyboard shortcuts as advertised on Reorder attachments panel (Alt+Up, alt+Down, Alt+Home, Alt+End etc.). Keyboard shortcuts work without the panel, but also while the panel is open, which is a good way of learning them.

Blocks: tb78found
Severity: -- → S3
Summary: File attachment order cannot be changed by drag and drop → Order of file attachments cannot be changed by drag and drop

Strangely, nothing in error console.

Confirming win10, 78.1.1 (64-bit). Thanks for the heads up on the alt+key workarounds.

Alice, would you do us the favour to find the regression. Just add 3+ attachments to a new message and then try to drag the last one first.

Flags: needinfo?(alice0775)

Thanks so much, Alice, let's make it so.

Flags: needinfo?(khushil324)
Regressed by: 1226362

Does this code fault also impact the ability to mouse click in the attachment window and add attachments?

(In reply to Wayne Mery (:wsmwk) from comment #8)

Does this code fault also impact the ability to mouse click in the attachment window and add attachments?

No, still works for empty attachment pane, then unexpectedly fails if you try the same trick with attachments present, that's bug 1656180. Most convenient, big no-brain click target, works perfectly in TB68, not harmful in any way (we can protect ctrl/shift selection misclicks a bit more), not sure why someone would want to break that and create ux-inconsistency...

Assignee: nobody → khushil324
Flags: needinfo?(khushil324)
Attachment #9169392 - Flags: review?(mkmelin+mozilla)
Status: NEW → ASSIGNED
Comment on attachment 9169392 [details] [diff] [review] Bug-1658114-DnD-attachments-pane-0.patch Review of attachment 9169392 [details] [diff] [review]: ----------------------------------------------------------------- LGTM, r=mkmelin
Attachment #9169392 - Flags: review?(mkmelin+mozilla) → review+
Target Milestone: --- → 81 Branch

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/2e4a327e4e26
Fix order of file attachments cannot be changed by drag and drop. r=mkmelin

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

Comment on attachment 9169392 [details] [diff] [review]
Bug-1658114-DnD-attachments-pane-0.patch

[Approval Request Comment]
Regression caused by (bug #): bug 1226362
User impact if declined: can't reorder attachments
Testing completed (on c-c, etc.): yes.
Risk to taking this patch (and alternatives if risky): small patch, small risk

Attachment #9169392 - Flags: approval-comm-esr78?
Attachment #9169392 - Flags: approval-comm-beta?

Comment on attachment 9169392 [details] [diff] [review]
Bug-1658114-DnD-attachments-pane-0.patch

[Triage Comment]
Approved for beta
Approved for esr78

Attachment #9169392 - Flags: approval-comm-esr78?
Attachment #9169392 - Flags: approval-comm-esr78+
Attachment #9169392 - Flags: approval-comm-beta?
Attachment #9169392 - Flags: approval-comm-beta+

I was able to change the order of attachments using drag and drop using TB 80.0b4 on Ubuntu 18.04.4 LTS.

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: