Closed Bug 1549690 Opened 6 years ago Closed 6 years ago

fix incorrect id "status-notification" and flexing in calendar-summary-dialog.xul

Categories

(Calendar :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mkmelin, Assigned: mkmelin)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug # 1536935 +++

The notification id is wrong in calendar-summary-dialog.xul.
This causes the notifications in it not to show (and other problems too i think).

And like bug 1548277, the flex="1" should go. Otherwise on resize, the notification is huge.

Summary: fix incorrect id "status-notification" in calendar-summary-dialog.xul → fix incorrect id "status-notification" and flexing in calendar-summary-dialog.xul

To test, open an event you were invited to.

Attachment #9063175 - Flags: review?(alessandro)
Comment on attachment 9063175 [details] [diff] [review] bug1549690_calsummary_dialog_notification.patch Review of attachment 9063175 [details] [diff] [review]: ----------------------------------------------------------------- Ah, thanks for fixing this. Silly me and my typos.
Attachment #9063175 - Flags: review?(alessandro) → review+
Keywords: checkin-needed

Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/36c6df962b9f
fix incorrect id "status-notification" and flex="1" in calendar-summary-dialog.xul r=aleca DONTBUILD

Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Product: Thunderbird → Calendar
Target Milestone: --- → 7.0
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: