Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-22874

Add 20 theme to the Moodle 2.0 core

    Details

    • Type: Task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: Themes
    • Labels:
      None
    • Environment:
      Moodle 2.0
    • Difficulty:
      Moderate
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      This is a meta task for myself to keep track of all of the themes to go into the core.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              dougiamas Martin Dougiamas added a comment -

              I just fixed some core themes that had a mistakenly-named "general" layout (should be called "standard"). This was causing bugs like blocks not appearing on some pages like calendars etc

              arialist/config.php binarius/config.php boxxie/config.php canvas/config.php formfactor/config.php fusion/config.php leatherbound/config.php nonzero/config.php

              Just wanted to mention it to catch any future themes from inheriting the issue.

              Show
              dougiamas Martin Dougiamas added a comment - I just fixed some core themes that had a mistakenly-named "general" layout (should be called "standard"). This was causing bugs like blocks not appearing on some pages like calendars etc arialist/config.php binarius/config.php boxxie/config.php canvas/config.php formfactor/config.php fusion/config.php leatherbound/config.php nonzero/config.php Just wanted to mention it to catch any future themes from inheriting the issue.
              Hide
              korpelainen Mauno Korpelainen added a comment -

              Patrick & John,

              when you add these new themes could you please doublecheck the format of settings.php :

              defined('MOODLE_INTERNAL') || die;

              if ($ADMIN->fulltree) {

              instead of

              $temp = new admin_settingpage('theme_xxx...
              ...
              and $settings->add($setting); instead of $temp->add($setting);

              or we will get 40 themes to our theme administration menu

              From current core themes Brick, Fusion, Magazine, Overlay, Sky High and Splash are using the "old theme settings.php style" which renders double links in administration menu... like Joseph pointed in http://moodle.org/mod/forum/discuss.php?d=154251

              Show
              korpelainen Mauno Korpelainen added a comment - Patrick & John, when you add these new themes could you please doublecheck the format of settings.php : defined('MOODLE_INTERNAL') || die; if ($ADMIN->fulltree) { instead of $temp = new admin_settingpage('theme_xxx... ... and $settings->add($setting); instead of $temp->add($setting); or we will get 40 themes to our theme administration menu From current core themes Brick, Fusion, Magazine, Overlay, Sky High and Splash are using the "old theme settings.php style" which renders double links in administration menu... like Joseph pointed in http://moodle.org/mod/forum/discuss.php?d=154251
              Hide
              epsd John Stabinger added a comment -

              Thanks for the heads up Mauno. All of the themes you mentioned should now be using the updated code.

              Show
              epsd John Stabinger added a comment - Thanks for the heads up Mauno. All of the themes you mentioned should now be using the updated code.

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    24/Nov/10