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

New 'Add module' dialog doesn't prevent multiple clicks

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3
    • Component/s: Course
    • Labels:
    • Testing Instructions:
      Hide
      • Open the course page
      • Turn editing on
      • Open the Activity Chooser
      • Select an activity and ensure that the help text is shown
      • Select another activity and ensure that it's help text is shown
      • Double-click on another activity
      • Ensure that before the page loads:
        • The 'Add' button is disabled and greyed out
        • The options are disabled
      • Ensure that the module editing page is for the correct module
      • Navigate back to the previous page
      • Open the Activity Chooser
      • Select an activity and ensure that the help text is shown
      • Select another activity and ensure that it's help text is shown
      • Click the add button
      • Ensure that before the page loads:
        • The 'Add' button is disabled and greyed out
        • The options are disabled
      • Ensure that the module editing page is for the correct module
      • Navigate back to the previous page
      • Open the Activity Chooser
      • Select an activity and ensure that the help text is shown
      • Select another activity and ensure that it's help text is shown
      • Click the add button
      • Quickly hit escape - this should close the chooser and cancel the page navigation
      • Open the Activity Chooser again
      • Select a fourth (different) activity
      • Click the add button again
      • Ensure that before the page loads:
        • The 'Add' button is disabled and greyed out
        • The options are disabled
      • Ensure that the module editing page is for the correct module
      Show
      Open the course page Turn editing on Open the Activity Chooser Select an activity and ensure that the help text is shown Select another activity and ensure that it's help text is shown Double-click on another activity Ensure that before the page loads: The 'Add' button is disabled and greyed out The options are disabled Ensure that the module editing page is for the correct module Navigate back to the previous page Open the Activity Chooser Select an activity and ensure that the help text is shown Select another activity and ensure that it's help text is shown Click the add button Ensure that before the page loads: The 'Add' button is disabled and greyed out The options are disabled Ensure that the module editing page is for the correct module Navigate back to the previous page Open the Activity Chooser Select an activity and ensure that the help text is shown Select another activity and ensure that it's help text is shown Click the add button Quickly hit escape - this should close the chooser and cancel the page navigation Open the Activity Chooser again Select a fourth (different) activity Click the add button again Ensure that before the page loads: The 'Add' button is disabled and greyed out The options are disabled Ensure that the module editing page is for the correct module
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-33135-master-3

      Description

      When using Firefox, the 'Add module' dialog lets you click the 'Add' button lots of times, which basically just makes it slower to eventually load an add page.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            quen Sam Marshall added a comment -

            +1 from me - approach seems complex but probably necessary, and I was wondering if you'd also fixed it for double-click. Earlier I also thought double-click didn't work, because I clicked it lots of times but nothing happened, until I eventually stopped clicking it and it worked

            Minor issues:

            // Enusre that the options are shown

            Typo. Plus code style trivia:

            1) A few instances of missing full stop at end of comment.
            2) Missing blank line above/below function.

            Show
            quen Sam Marshall added a comment - +1 from me - approach seems complex but probably necessary, and I was wondering if you'd also fixed it for double-click. Earlier I also thought double-click didn't work, because I clicked it lots of times but nothing happened, until I eventually stopped clicking it and it worked Minor issues: // Enusre that the options are shown Typo. Plus code style trivia: 1) A few instances of missing full stop at end of comment. 2) Missing blank line above/below function.
            Hide
            poltawski Dan Poltawski added a comment -

            Integrated, thanks.

            (Also seems complex to me, so I hope all the complexity there is worth it )

            Show
            poltawski Dan Poltawski added a comment - Integrated, thanks. (Also seems complex to me, so I hope all the complexity there is worth it )
            Hide
            poltawski Dan Poltawski added a comment -

            All works for me. Neat.

            However, i'd love to know why that page takes SO LONG to load!

            Show
            poltawski Dan Poltawski added a comment - All works for me. Neat. However, i'd love to know why that page takes SO LONG to load!
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads).

            Thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - This issue has been integrated upstream and is now available both via git and cvs (and in some hours, via mirrors and downloads). Thanks!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  25/Jun/12