Closed Bug 1982586 Opened 2 months ago Closed 2 months ago

perma comm/mail/test/browser/composition/browser_attachmentDragDrop.js failure starting august 12

Categories

(Thunderbird :: Upstream Synchronization, defect, P5)

Tracking

(thunderbird_esr140 unaffected, thunderbird142 unaffected)

RESOLVED FIXED
143 Branch
Tracking Status
thunderbird_esr140 --- unaffected
thunderbird142 --- unaffected

People

(Reporter: intermittent-bug-filer, Assigned: aleca)

References

(Regression)

Details

(Keywords: intermittent-failure, intermittent-testcase, regression, Whiteboard: [collect_confirm_failure])

Attachments

(1 file)

See Also: → 1982590
Keywords: regression
Regressed by: 1941406

Gentle ping to get some help here.
Our usage of <details> inside the message compose is broken after the changed in bug 1941406.
https://searchfox.org/comm-central/rev/9f80cdef126dd529ca33ce30b062f1f3d2939974/mail/components/compose/content/messengercompose.xhtml#2494
That element doesn't seem to play well with CSS grid anymore.
Any help?

Flags: needinfo?(emilio)
Flags: needinfo?(dholbert)
Assignee: nobody → alessandro
Status: NEW → ASSIGNED

I see you've got a patch up, but if you need other ideas -- as a somewhat-inelegant hackaround, you can suppress the new box and restore the old behavior by adding this CSS for the content in question:

::details-content {
    display: contents;
}

(as an example, I did this in https://hg-edge.mozilla.org/mozilla-central/rev/fa6903bc79db for an a11y test that needs a bit more investigation from a11y folks.)

Flags: needinfo?(emilio)
Flags: needinfo?(dholbert)

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/4ada14179924
Adapt the <details> element to work with layout.details.force-block-layout. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 143 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: