share css should not have !important on line 4 of code, does not let a theme to override default style image.
If you have a site with a black background, the share logo has ugly white corners, and the best way to get around this is by theming a custom image optimized for black corners (or to have transparency for the rounded corners of the image). Currently to override the default share image you have to hack the module (never good for upgradability). We can solve this issue by changing this:

a.share, a.share_node, a.share_link {
	background: transparent url('share-icon-16x16.gif') no-repeat scroll 1px 0pt !important;
	padding: 1px 0pt 3px 22px;
}

to this

a.share, a.share_node, a.share_link {
	background: transparent url('share-icon-16x16.gif') no-repeat scroll 1px 0pt;
	padding: 1px 0pt 3px 22px;
}

There is never a good reason to include !important in css if you can theme around it.
Sorry, I wont be able to provide a patch for this one (tight chedule), but it's an easy enough fix. Just delete !important from the end of line 4.

Comments

greenskin’s picture

greenskin commented

I will remove that bit of code, but I would like to point out that the css file is wrapped in a theme function so that you can include your own css file. For a start you can copy the included css and make changes to it, then put the override theme function to include your custom css file.

westwesterson’s picture

westwesterson commented

Thank you!
while this is true we have used a single styles.css to minimize the number of files needing to load via http requests this enables a simple override in the theme style.css :)

greenskin’s picture

greenskin commented
Status: Active » Fixed

You can also aggregate the css files via /admin/settings/performance. This also makes all the css files condensed into one.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.