I have finished the first part of the corrections to the themeswitcher module. Results are posted here: http://drupal.org/node/82340 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. http://drupal.org/node/82340 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
participants (1)
-
Ken Rickard