Closed (fixed)
Project:
Theme Settings
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
20 Aug 2008 at 16:25 UTC
Updated:
29 Jul 2010 at 15:40 UTC
Will the 'Theme Settings' module be ported to Drupal 6.x? If so, when?
Comments
Comment #1
johnalbinI had no idea anybody was using this module. :-p
It should be straight-forward to port it.
Comment #2
fm commentedDon't you use it? :P
I've found the module quite useful.
Comment #3
dbnettle commentedIs there any work underway to port this to Drupal 6? This is the last module that I would need to upgrade a couple of sites to Drupal 6.
-Brian
Comment #4
johnalbinOk, looks like there's good usage of this module, so I'll port it to Drupal 6.
However, the more I've played with the code, the more I realized that having per-theme settings is difficult to get right.
The per-node-type settings are very useful, but I'm not sure that per-theme settings are that useful. When I first wrote this module, I was wildly optimistic about giving all sorts of flexibility. But it makes the code hard to grok and ugly-looking.
So can I get some opinions about removing the per-theme settings? i.e. The admin/build/themes/settings would have Theme Settings widgets and those settings would affect all themes (and admin/build/themes/settings/THEMENAME wouldn't have any Theme Settings widgets.)
Comment #5
zilla commentedi believe that the "string overrides" module for d6 replicates all of this functionality, as well as "form edit" - between those two you can replace strings, manage the replacements in a central admin screen and also move the elements of forms up and down by weight, an also edit text within forms not touched by strings directly (per the output)
before you spend time porting to d6, you may want to take a look at those two other modules already out on d6 - feels like identical functionality (though form edit is really quite useful for non-stringed output like little text bits below boxes or whatever)
Comment #6
ahkiam commented@JohnAlbin
+1 For Port w/ Per Node Settings but not Per Theme (never used that one anyway)
@zilla
String Override is only adequate if your string changes are site-wide, which doesn't work if, say, I want "read more" replaced on only my podcasts content with "Listen >>". Plus Theme Settings allows for relocating of read more and other elements which is not duplicate functionality.
Comment #7
wallbay1 commentedsubscribing
Comment #8
asb commentedsubscribing :-p
Comment #9
ktonini commentedsub
Comment #10
mikey_p commentedI've rolled this functionality into a patch over at #538376: Logic problem with per-type settings and isset() Assigned to: mikey_p.
Comment #11
dddave commented#538362: Initial port to Drupal 6
Just for the record.....
Are there plans for a proper release?
Comment #12
johnalbin#538376: Logic problem with per-type settings and isset() has been committed.