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

More theme should not force colours to be set

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.7
    • Fix Version/s: 2.7
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide
      1. Go the theme_more config
      2. Set a value on each colour field, and save
      3. Reload the settings page
      4. Set all the colours to an empty value
      5. Save and make sure the empty value was saved
      1. Install theme_magazine: https://github.com/moodlehq/moodle-theme_magazine
      2. Go to its settings page, fill in every colour field and save
      3. Once back on the page, remove all the colours from the fields, and save
      4. Make sure that the colours with a Default value are populated with that value
      Show
      Go the theme_more config Set a value on each colour field, and save Reload the settings page Set all the colours to an empty value Save and make sure the empty value was saved Install theme_magazine: https://github.com/moodlehq/moodle-theme_magazine Go to its settings page, fill in every colour field and save Once back on the page, remove all the colours from the fields, and save Make sure that the colours with a Default value are populated with that value
    • Affected Branches:
      MOODLE_27_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-44912-master-core
    • Sprint:
      FRONTEND Sprint 11

      Description

      The colour picker does not allow the colours to be reset to nothing if there is a default. This should be allowed as it allows the admin to fallback on the settings from Bootstrapbase.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              fred Frédéric Massart
              Reporter:
              fred Frédéric Massart
              Peer reviewer:
              Damyon Wiese
              Integrator:
              Sam Hemelryk
              Tester:
              Jetha Chan
              Participants:
              Component watchers:
              Bas Brands
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                12/May/14