In my (humble) opinion, there is little need to add the call to drupal_add_css in this module - simply adding the relevant class to the inserted links is enough. Adding instructions to show the user how to override this in CSS is far more flexible than having another function call and yet another css file included (more time wasted on page load).

Clearing up this matter is as simple as:

  • delete the .png file
  • delete the .css file
  • remove the drupal_add_css call from alinks.module (just delete the line)
  • instruct user to override the css class "alinks-link" and put any relevant bg images in their theme directory

Comments

tic2000’s picture

tic2000 commented

I have to disagree. Not all users knows how to change the css. The best I can do is to add an option to enable or not the css.
As a side note, I wrote the module, I think I know what I have to do to remove the css.

niklp’s picture

niklp commented

Well, the point of putting that up there was, if you disagreed with me, I've already told people how to get round it if they want to... not to tell you how you wrote the module :)

tic2000’s picture

tic2000 commented
Status: Active » Closed (fixed)