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

The calendar image should display after the date selectors, not after the enabled checkbox (if it is visible)

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5.1
    • Component/s: Forms Library
    • Labels:
    • Testing Instructions:
      Hide
      1. Create a new quiz activity.
      2. On the editing page click to expand the 'Timing' section.
      3. Ensure the calendar icon is displayed after the date selectors and not the enabled checkbox.
      1. Create a new database activity.
      2. On the editing page click to expand the 'Availability' section.
      3. Ensure the calendar icon is displayed after the date selectors and not the enabled checkbox.
      Show
      Create a new quiz activity. On the editing page click to expand the 'Timing' section. Ensure the calendar icon is displayed after the date selectors and not the enabled checkbox. Create a new database activity. On the editing page click to expand the 'Availability' section. Ensure the calendar icon is displayed after the date selectors and not the enabled checkbox.
    • Affected Branches:
      MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull Master Branch:
      MDL-40008_master

      Description

      1. Create a new database activity.
      2. On the editing page click to expand the 'Availability' section.
      3. Notice the calendar icon is displayed after the enabled checkbox.

        Gliffy Diagrams

          Activity

          Hide
          poltawski Dan Poltawski added a comment -

          The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

          TIA and ciao

          Show
          poltawski Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
          Hide
          poltawski Dan Poltawski added a comment -

          Hi Mark,

          Should this be backported to 2.5?

          Show
          poltawski Dan Poltawski added a comment - Hi Mark, Should this be backported to 2.5?
          Hide
          markn Mark Nelson added a comment -

          Hi Dan, yes it should. Could you cherry-pick the commit for me please? Thanks.

          Show
          markn Mark Nelson added a comment - Hi Dan, yes it should. Could you cherry-pick the commit for me please? Thanks.
          Hide
          poltawski Dan Poltawski added a comment -

          Integrated to master and 25, thanks.

          Show
          poltawski Dan Poltawski added a comment - Integrated to master and 25, thanks.
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Thanks Mark - tested and passed.

          Show
          samhemelryk Sam Hemelryk added a comment - Thanks Mark - tested and passed.
          Hide
          marina Marina Glancy added a comment -

          Thanks for your awesome work! This has now become a part of Moodle.

          Closing as fixed!

          Show
          marina Marina Glancy added a comment - Thanks for your awesome work! This has now become a part of Moodle. Closing as fixed!

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                8/Jul/13