853488 - Styling tweaks for Mac XUL notification alerts

Status: RESOLVED FIXED

(Toolkit :: Themes, defect)

Description

[Markus Stange [:mstange]]

Attachment: v1

At the moment the fade animation isn't very effective because there's opaque gray window background behind notification box which doesn't fade away.
This patch removes that background so that the animation looks as intended.
Also, I got carried away and tweaked the styling a little.

Comment 1

[Markus Stange [:mstange]]

Attachment: before (screnshot)

Comment 2

[Markus Stange [:mstange]]

Attachment: after (screenshot)

Comment 3

[Justin Dolske [:Dolske]]

Comment on attachment 727719 [details] [diff] [review]
v1

It's nice to see you around again. :D

Comment 4

[Markus Stange [:mstange]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b48c78488fce

Comment 5

[Matthew N. [:MattN]]

(In reply to Markus Stange from comment #0)
> Also, I got carried away and tweaked the styling a little.

Do any of the styling changes also apply to other platforms? I'd rather not have the styles diverge unless necessary for platform integration.

Comment 6

[Markus Stange [:mstange]]

Well, I don't know what you want the panels to look like on the other platforms. The styling changes I applied here make them look more like the native Mac Notification Center alerts.
You might be able to apply some of them to the Windows / Linux ones, too, but I don't know how well we handle transparent windows and their window shadow on those platforms nowadays. Also, non-Mac platforms make more use of customizable platform theme colors, and some of the color changes I've made here might not fit with those.

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/b48c78488fce