1674937 - Drag and drop of email address(es) from contacts side bar to custom header (mail.compose.other.header) does nothing, should drop csv addresses

Status: NEW

(Thunderbird :: Message Compose Window, defect)

Description

[Thomas D. (:thomas8)]

+++ This bug was initially created as a clone of Bug #1673579 +++
Thanks to orwel01 who alerted us to this issue in Bug #1673579. Re-filed here for a clean start.

STR

1. Open a new mail compose window
2. Create a custom header (e.g. "qacontact") via mail.compose.other.header pref
3. Open contacts side bar
4. drag one or more email addresses into the custom header field ("qacontact")

Actual results:

• nothing

Expected results:

• drop addresses as comma-separated plaintext (we don't create pills because custom headers are free-form, but that shouldn't stop us from dropping text at least).

This may be relevant for Addons like Send Later per Bug 1673579 Comment 2, which provides fields for Return-Receipt-To and Dispostion-Notification-To, assuming the addon creates custom headers for that. Edit: As orwel01 correctly pointed out, this bug does not involve addons, but only using the mail.compose.other.header pref.

Comment 2

[Thomas D. (:thomas8)]

Orwel01, I have re-filed the drag-issue of your bug 1673579 here, given you credit for alerting us to this issue, and corrected my comment 0 here to clarify that this problem does not involve addons as you correctly pointed out.

Comment 3

[Thomas D. (:thomas8)]

Nothing in error console. I guess we're checking if the target row allows pills, and custom header rows do not. Let's be more flexible and provide plain-text drop flavor for custom header fields! Failing that, it becomes quite a hassle to add addresses from contacts side bar.

Bug 271917 may improve the situation (if we add a dropdown selector to contacts side bar to allow adding contacts to any header field), but it won't solve the drag issue.

Comment 4

[Thomas D. (:thomas8)]

This is a regression over TB 68.

Comment 5

[orwel01]

Hi Thomas D.,
I am sorry for all the inconvenience I had effected. Maybe it is better to set the regression to version 52.9.1 as I had took a look to old versions and about TB 68 I am not pretty sure, but about 52.9.1 I am as I have the install file marked with "OK" in my archive.
On my defense:

• I have never focus such a situation, I did not know, how to deal with a closed issue (which was not solved in my opinion), I still do not know, how open a "needinfo" request on a closed issue (as there is no "needinfo" check/value, only reopening the issue),
• I did not mention the mail.compose.other.header pref because on the very beginning I did not know, that I had configured it years ago and as I am not a TB programmer, i forgot that this is not a common header - i had figured this out only after your second closing of the issue by mentioning that everything works fine... so i tried to figure out where could be the problem and found a comment about adding this header by an option change...
• On the very first post i did not know, that there are 2 issue - for me as a user there was only one issue - i could not put addresses to headers neither by double click nor by drag and drop (my fault is, i did not test all the headers, only the Return-receipt and Bcc, as i thought all headers work in the same way). I had described the full behavior to grant you a complex information... and as we were talking about the bug 1673579, after few posts I only recognized that the behavior means probably 2 issues. But as we were on discussion, i felt it is more effective to continue with you on the bug report as creating a fully new bug and maybe explain it again to someone else:) If I were told "create a new bug report", i would definitely create it.
  I really appreciate your effort and creating this bug!
  Thank you one more time and feel free to change the regression to 52.9.1 as mentioned above.

Comment 6

[orwel01]

Or maybe remove the regression total... I used to put the address to Return-Receipt-To header always by a double click, which worked perfectly till 52.9.1. and honestly I never used drag and drop. The previous version of TB was mentioned in bug 1673579 where I had post also the "double click" problem.

Comment 7

[Alfred Peters [:infofrommozilla]]

(In reply to Thomas D. (:thomas8) from comment #3)

Nothing in error console. I guess we're checking if the target row allows pills, and custom header rows do not. Let's be more flexible and provide plain-text drop flavor for custom header fields! Failing that, it becomes quite a hassle to add addresses from contacts side bar.

I see it the same way.
It isn't limited to mail.compose.other.header. I also can't drop to the subject header.
But I can drop an address to the body. There I get: "Alfred Peters <[e-mail]>"