Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-40314 Clean up whitespace in Formfactor
  3. MDL-40455

Move slash comments to be before code (not after) and add missing layouts in FormFactor config.php

    XMLWordPrintable

    Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5.2
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide

      Nothing to test. Just need to check theme/formfactor/config.php

      Set the theme to formfactor - check a few different page types like frontpage, course, embedded and popup. Make sure it doesn't look odd...

      Show
      Nothing to test. Just need to check theme/formfactor/config.php Set the theme to formfactor - check a few different page types like frontpage, course, embedded and popup. Make sure it doesn't look odd...
    • Affected Branches:
      MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull Master Branch:
      MDL-40455_M26

      Description

      Confusing comments in some themes config.php, so by moving these to above the code rather than below, as they are at present, will make them easier to read and understand. Also in the process check to see if there are missing layouts, like report or secure, etcetera.

        Attachments

          Activity

            People

            • Assignee:
              lazydaisy Mary Evans
              Reporter:
              lazydaisy Mary Evans
              Integrator:
              Damyon Wiese
              Tester:
              Andrew Davis
              Participants:
              Component watchers:
              Amaia Anabitarte, Bas Brands, Carlos Escobedo, Sara Arjona (@sarjona), Víctor Déniz Falcón
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                9/Sep/13