Posted by curtis on May 14, 2008 at 8:37am
Jump to:
| Project: | Digg this |
| Version: | master |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | yaph |
| Status: | closed (fixed) |
Issue Summary
The CSS snippet included with this module defines a width of 150px and a margin of 5 px. The width is way too wide, and there's really no need to define either of these items, especially since they can't be overridden by theme styles. Please remove the CSS file from the distribution, and allow users to define their own styles using the class names you provide.
Comments
#1
I agree that the CSS file could well be omitted but you can override the rule in your theme simply by choosing a CSS selector that is more specific, e.g.
#content div.diggthis_button { width:...}#2
Yeah, I don't know why I was thinking it couldn't be overridden. Sorry, this can be closed.
#3
done