Closed (won't fix)
Project:
Drigg
Version:
5.x-1.33
Component:
Code
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
1 Jul 2008 at 09:45 UTC
Updated:
8 Jul 2008 at 15:06 UTC
Hello,
First of all I would like to thank you again for this excellent module and I really appreciate the hard work you put into it and maintaining it.
Is it possible to override some of drigg's functions using the template.php?
i want to override drigg_ui_article_menu() without changing the source module file?
This not very important so if it's complicated I can just edit the module file.
Thanks,
Nimi.
Comments
Comment #1
sikjoy commentedHello:
The markup here isn't wrapped in a theme function, but that can be easily fixed. If you can give me a week or two, I'll move the markup to a function that can be overridden.
--sicjoy
Comment #2
mercmobily commentedHi,
some notes:
* There is no real point in changing functions into themes if they are very complicated. This is the case here.
* Drupal 6 deals with this in a ver ydifferent way
* Changing that function will break the theme
* The change should be aimed, in my opinion, at making sure the output of that function is easy to customise with CSS. I wouldn´t touch it.
But, I might well be wrong. What is it that you can't do with CSS *precisely*?
Merc.
Comment #3
nimi commentedsorry for the delay. Had to format my computer...
I wanted to add another tab in the drigg_ui_article_menu(). In the end I've decided to do it though the module file.
Thanks.
Nimi.
Comment #4
mercmobily commentedHi,
This is a theming thing. Adding a tab should be possible by changing the template, rather than the code.
My comments on the need of changing CSS rather than turning 150 line functions into theme_ functions stands _solid_...
Merc.
Comment #5
mercmobily commentedHi,
Closing this issue. Things should be done through CSS. drigg_ui's output is long, and takes a lot of processing. They are not actually theme functions.
Merc.