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]
This patch implements the styles from toolkit's alert.css.
Created attachment 731663 [details]
screenshot on Windows
Created attachment 731664 [details]
screenshot on Linux
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
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]
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.
Created attachment 735073 [details]
Patch under Linux
Created attachment 735074 [details]
patch under Windows
The dark border around the alerts are my backgrounds.
Comment on attachment 735071 [details] [diff] [review]
proposed fix v2
I like it! Thanks Richard.