Closed Bug 1577659 Opened 2 years ago Closed 2 years ago

Remove the statusbarpanel CE

Categories

(Thunderbird :: General, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 70.0

People

(Reporter: darktrojan, Assigned: khushil324)

References

Details

Attachments

(1 file)

This is a pretty unnecessary binding, we could just use label, and it causes a layout warning every time it is changed:

"XUL box for statusbarpanel element contained an inline #text child, forcing all its children to be wrapped in a block."

Assignee: nobody → khushil324
Attachment #9089310 - Flags: review?(mkmelin+mozilla)
Status: NEW → ASSIGNED
Comment on attachment 9089310 [details] [diff] [review]
Bug-1577659_remove-statusbarpanel-CE.patch

Review of attachment 9089310 [details] [diff] [review]:
-----------------------------------------------------------------

Looks ok. r=mkmelin
Attachment #9089310 - Flags: review?(mkmelin+mozilla) → review+
Summary: Remove the statusbarpanel binding → Remove the statusbarpanel CE
Keywords: checkin-needed

Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/aaa8e8ef1aa5
remove statusbarpanel custom element. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 70.0
Version: unspecified → Trunk

Thanks Khushil. It's nice when stuff gets fixed without even having to ask.

Regressions: 1578109
Regressions: 1578117
You need to log in before you can comment on or make changes to this bug.