Details

    • Testing Instructions:
      Hide

      Check that the activity chooser still works.
      In Firefox, Chrome and IE browsers:

      There should be no actual changes other than to fix incorrect syntax which may improve some browser compatibility.

      Also, run jshint (available from Node.js) over lib/yui/chooserdialogue/chooserdialogue.js for each branch

      Show
      Check that the activity chooser still works. In Firefox, Chrome and IE browsers: There should be no actual changes other than to fix incorrect syntax which may improve some browser compatibility. Also, run jshint (available from Node.js) over lib/yui/chooserdialogue/chooserdialogue.js for each branch
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull Master Branch:
    • Rank:
      47039

      Description

      I'm planning some work on the chooser dialogue at the moment, and it would be great if it validated correctly first...

        Issue Links

          Activity

          Hide
          Damyon Wiese added a comment -

          Thanks Andrew,

          This patch looks good to me.

          Sending for integration review.

          Show
          Damyon Wiese added a comment - Thanks Andrew, This patch looks good to me. Sending for integration review.
          Hide
          Damyon Wiese added a comment -

          Just adding a subset of browsers to the testing instructions (this is a low risk change).

          Show
          Damyon Wiese added a comment - Just adding a subset of browsers to the testing instructions (this is a low risk change).
          Hide
          Damyon Wiese added a comment -

          Also - this is an improvement and should not be backported.

          Show
          Damyon Wiese added a comment - Also - this is an improvement and should not be backported.
          Hide
          Andrew Nicols added a comment -

          Part of me would argue that this should be backported as some of the changes made are to fix issues with browser compatibility (e.g. IE sucks at trailing commas). That said, none of the issues I've addressed so far have been for an extra ,

          Show
          Andrew Nicols added a comment - Part of me would argue that this should be backported as some of the changes made are to fix issues with browser compatibility (e.g. IE sucks at trailing commas). That said, none of the issues I've addressed so far have been for an extra ,
          Hide
          Eloy Lafuente (stronk7) 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
          Eloy Lafuente (stronk7) 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
          Dan Poltawski added a comment -

          I'm on the fence about backporting it (epsecially as its hard for us to prove that there was a problem before in normal use). If there were JS warnings I would be more convinced.

          Show
          Dan Poltawski added a comment - I'm on the fence about backporting it (epsecially as its hard for us to prove that there was a problem before in normal use). If there were JS warnings I would be more convinced.
          Hide
          Dan Poltawski added a comment -

          I missed Damyons comment, thats enough confirmation for me. This will go into master only.

          Show
          Dan Poltawski added a comment - I missed Damyons comment, thats enough confirmation for me. This will go into master only.
          Hide
          Dan Poltawski added a comment -

          Integrated to master only, thanks Andrew.

          Show
          Dan Poltawski added a comment - Integrated to master only, thanks Andrew.
          Hide
          Michael de Raadt added a comment -

          Test result: Success!

          Tested in Master only.

          JSHint revealed no warnings.

          Testing in IE10, FF18, Chrome23 and Safari5 showed no JS errors.

          Show
          Michael de Raadt added a comment - Test result: Success! Tested in Master only. JSHint revealed no warnings. Testing in IE10, FF18, Chrome23 and Safari5 showed no JS errors.
          Hide
          Dan Poltawski added a comment -

          Hurray! We did it! Thanks to all the reporters, testers, user and watchers for a bumper week of Moodling!

          Show
          Dan Poltawski added a comment - Hurray! We did it! Thanks to all the reporters, testers, user and watchers for a bumper week of Moodling!

            People

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

              Dates

              • Created:
                Updated:
                Resolved: