User "Patrick Nagel" posted a comment on MediaWiki.r89637.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/89637#c20995
Commit summary:

Initial commit of new extension Notificator

Comment:

Basically diff-in-mail.css just contains skins/common/diff.css minus comments 
and minus 'background-color: white'.

The first is due to the fact that the CSS is sent with every notification 
e-mail, and it would just be a waste of bandwidth to send the 
developer-targeted comments along. The latter because it looked weird when the 
mail client window's background colour was grey. I don't like the duplication 
either, but I think mangling 'skins/common/diff.css' in some way isn't elegant, 
either.

What do you think?

_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to