Bug 1695521 Comment 8 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

This should fix everything.

- I removed the dragenter and dragexit from the overlay child elements.
- I'm handling the highlight of the drop bucket (attach/inline) from the parent's dragover event.
- I'm using a timeout to avoid the jarring on/off effect of the overlay when dragging over child elements.

I was planning to add a test to cover this and be sure the overlay is always hidden, but I'm not sure mochitest supports what I need to do (dragging a file from outside Thundebird).
Is this comment still relevant? https://searchfox.org/comm-central/rev/57a26c960bcb91a28575c96c44833b78b8393e20/mail/test/browser/composition/browser_attachment.js#926-927

I'm asking a ui-review to Richard since I changed some styling and to be sure the usage of `pointer-events: none;` doesn't create any issue.
This should fix everything.

- I removed the dragenter and dragexit from the overlay child elements.
- I'm handling the highlight of the drop bucket (attach/inline) from the parent's dragover event.
- I'm using a timeout to avoid the jarring on/off effect of the overlay when dragging over child elements (is this okay or there's a better method?).

I was planning to add a test to cover this and be sure the overlay is always hidden, but I'm not sure mochitest supports what I need to do (dragging a file from outside Thundebird).
Is this comment still relevant? https://searchfox.org/comm-central/rev/57a26c960bcb91a28575c96c44833b78b8393e20/mail/test/browser/composition/browser_attachment.js#926-927

I'm asking a ui-review to Richard since I changed some styling and to be sure the usage of `pointer-events: none;` doesn't create any issue.

Back to Bug 1695521 Comment 8