The new mail alert looks a little bit outdated with his thick blue 3D border. This bug is to modernize this alert and make it equal to the toolkit notification alerts.
Created attachment 731662 [details] [diff] [review] proposed fix This patch implements the styles from toolkit's alert.css.
Is there any way we could use the toolkit's CSS directly, instead of copying it?
(In reply to Blake Winton (:bwinton) from comment #4) > Is there any way we could use the toolkit's CSS directly, instead of copying > it? This could be possible with @import. But almost all items in alarm are classes which are in newmailalert IDs. I could change this but is it worth to do it? It's only a small CSS file.
Perhaps not. I wonder more about what we'ld do when toolkit changes their style next time. In the end, I'll be happy with whatever you and Mike decide is best. :)
Comment on attachment 731662 [details] [diff] [review] proposed fix First off, it's really cool that we're cleaning up this little popup. Now that I'm seeing it in isolation, I think my major beef with this is that I find the alignment a little screwy. The "Bluewin received 1 new message" is not lined up with the separator line below it, and the lines below that. Can that be fixed here?
Created attachment 735071 [details] [diff] [review] proposed fix v2 Now with aligned content. I've also removed a hard coded color (.folderSummary-previewText) from 'color: grey' to 'opacity: 0.6' to work better on every theme.
The dark border around the alerts are my backgrounds.
Comment on attachment 735071 [details] [diff] [review] proposed fix v2 I like it! Thanks Richard.