Closed (won't fix)
Project:
Quick Tabs
Version:
6.x-2.0-rc4
Component:
Miscellaneous
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
8 Sep 2010 at 13:51 UTC
Updated:
27 May 2015 at 19:08 UTC
Jump to comment: Most recent
Comments
Comment #1
adrianmak commentedeven though I created another tabstyles folder under the module, quicktabs cannot recognize the new tabstyle
Comment #2
pasqualleyou need a css file also..
the new tabstyle should go into the theme, but it does not work..
only solution for now is to put it into QT module tabstyles directory, or implement the hook_quicktabs_tabstyles() in a custom module..
Comment #3
RichardLynch commentedAs a guy who has to manage 25 multisites in an Enterprise environment with source code control, can I just say for the record that this is a BIG problem?
You've got theming stuff in sites/all/modules/quicktabs which I need to wholesale replace to be certain that all suspect code from a security vulnerability is gone, but I can't, because that's going to nuke the theming of the quicktabs and...
Put it in the DB, or put it in sites/example.com/themes/quicktab/ or something, but NOT mingled in with your module code.
I'm not bumping this up to major as that would be rude, but it's sure a major problem to me.
Comment #4
pasqualle@RichardLynch:
if you do not want to create a module, then you can still use the basic style and override it in the theme as you like..
for the D7 version the hook should be an alter hook then themes could add additional styles.. I do not see a solution how a theme could add additional styles in D6. Do you have examples of other D6 modules which allow similar thing?
Comment #5
RichardLynch commentedIt makes perfect sense, but...
I'm not the guy who creates the theme/css etc.
I'm just the guy who has to apply Security Patches, and can't.
We can have a corporate policy of "don't do that" but you can guess how well that pans out with a revolving door of consultants creating Drupal themes for a lot of sites...
My point is that it's just bad architecture to put theming into the code.
Drupal is a semi-MVC type architecture, and this violates that separation of code and presentation.
Comment #6
netw3rker commentedComment #7
amaisano commented+1 for this request - our site broke because we weren't told someone had put custom quicktab styles in the quicktabs module folder, and we had to update the module "wholesale" as @RichardLynch put it.
Is there no other way to nicely package the custom themes outside the module folder? You mention this can be done with a hook? Will that make the style show up as selectable in the Quicktabs admin UI?
Is this handled in 6.x-3.x?