I have finished the first part of the corrections to the themeswitcher module.
Results are posted here:
I found the original array handling too complex, so I simplified it greatly. But that really reflects my limits as a programmer rather than the value of the original code.
May need to put in a check to ensure that count($themes) > 1, otherwise the theme browser block is useless.
Do we still need a themes overview page that shows all available themes and ties them to themeswitcher?
- Ken Rickard agentrickard
I cleaned up the count($themes) check and went ahead and added an overview page.
This new patch replaces the first one I sent today.
I did not use theme_table() and theme_pager() for the output on the themeswitcher_page() function. I am not using database calls, so I didn't use those functions. If someone wnts to make those work, please do.
Patch: http://drupal.org/files/issues/themeswitcher_page.patch
Module: http://drupal.org/files/issues/themeswitcher_0.module
Screenshot: http://drupal.org/files/issues/ts.png
- Ken Rickard agentrickard
-
Ken Rickard