could I put under my own theme folder ?

Comments

adrianmak’s picture

even though I created another tabstyles folder under the module, quicktabs cannot recognize the new tabstyle

Pasqualle’s picture

you 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..

RichardLynch’s picture

As 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.

Pasqualle’s picture

@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?

RichardLynch’s picture

It 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.

netw3rker’s picture

Issue summary: View changes
Status: Active » Closed (won't fix)
amaisano’s picture

+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?