Menu buttons in summary dialog are displayed as simple buttons

RESOLVED FIXED in 6.5

Status

defect
RESOLVED FIXED
10 months ago
10 months ago

People

(Reporter: MakeMyDay, Assigned: Paenglab)

Tracking

(Blocks 1 bug, {regression})

Lightning 6.6
Dependency tree / graph

Details

Attachments

(1 attachment)

When displaying an event in the summary dialog after accepting an inviation for it, the toolbar buttons to accept, tentatively accept or decline the participation are missing the dropmarker and thus don't allow the user to decide whether or not to send a reply anymore when making his choice. Maybe related, for toolbarbutton -moz-box-align is applied with stretched instead of center.

Works in ESR60, not on current tip. Maybe a regression from bug 1487765 or bug 1457218, so eventually already an issue in 6.5.

Richard, do you have any idea offhand what's we're missing here?
Blocks: 1480039
This fixes it for me. TB 60 isn't affected, but please check to be sure.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #9009466 - Flags: review?(makemyday)
Comment on attachment 9009466 [details] [diff] [review]
bug1491689.patch

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

Thanks. So this is a regression from bug 1457218, right? In that case we need to uplift this to beta - esr worked without that.

::: calendar/base/content/dialogs/calendar-summary-dialog.xul
@@ +4,5 @@
>     - file, You can obtain one at http://mozilla.org/MPL/2.0/.
>  -->
>  
>  <?xml-stylesheet type="text/css" href="chrome://global/skin/global.css"?>
> +<?xml-stylesheet type="text/css" href="chrome://messenger/content/bindings.css"?>

can you move that down after the several calendar includes?
Attachment #9009466 - Flags: review?(makemyday) → review+
I'd like to leave it where it is. xul.css is also loaded as one of the first stylesheet and bindings.css has the rules which are removed from xul.css. Like this, following rules can override the styles easier.
Flags: needinfo?(makemyday)
Keywords: checkin-needed
Please request beta uplift from Philipp asap.
Attachment #9009466 - Flags: approval-calendar-beta?(philipp)
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/316140574f95
Fix menu buttons in summary dialog by adding bindings.css to calendar-summary-dialog.xul. r=MakeMyDay DONTBUILD
Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 6.6
Attachment #9009466 - Flags: approval-calendar-beta?(philipp) → approval-calendar-beta+
Flags: needinfo?(makemyday)
You need to log in before you can comment on or make changes to this bug.