PATCH: override padding for help icon [#278878]

Hi there,
my theme sets padding for the a element, which means the help icon has to big dimensions and the part of the next sprite can be seen. This patch adds padding: 0px; to the advanced-help-link class. The other one is for the views module's configuration sprites (same issue).

Comment	File	Size	Author
	views-override-padding-for-icon-sprites.patch	293 bytes	eMPee584
	advanced_help-override-padding-for-help-icon.patch	263 bytes	eMPee584

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

merlinofchaos CreditAttribution: merlinofchaos commented 7 July 2008 at 20:35

Since there are patches for different modules, should the theme not just provide this as its own override? I'm not entirely sure that's right, but it's difficult for modules to be completely defensive about their CSS, and themes which are doing a lot of work sometimes have to make sure they don't blow up modules.

Comment #2

eMPee584 CreditAttribution: eMPee584 commented 7 July 2008 at 21:00

well this is an important thing sprites have to care about: a 0 padding! And many themes might set very general rules that would mess this up, while those required for these sprites are very specific... the padding for the help icon could also have gone into the views css as that extends the .advanced-help class, but this module's css is the origin of it so i thought it's better suited here. And i only attached the views patch here aswell because i now you are swinging your wand on both modules ;)