Moodle
  1. Moodle
  2. MDL-33721

Filepicker button 'Select this file' should prevent double clicks

    Details

    • Testing Instructions:
      Hide
      1. Go to your private files and click to add the file
      2. Choose a file from any repository (for example, Dropbox)
      3. Click 'Select this file'
      4. Make sure you can not click the button second time (loading icon is displayed)
      Show
      Go to your private files and click to add the file Choose a file from any repository (for example, Dropbox) Click 'Select this file' Make sure you can not click the button second time (loading icon is displayed)
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull Master Branch:
      wip-MDL-33721-master

      Description

      When browsing a repository (Dropbox in my case) and selecting a file, there is no confirmation or loading screen and the button 'Select this file' is still active. Which means that each click on it will retrieve one more copy from the repository.

      Ideally a loading icon should pop and the button should be disabled.

      1. Set up your Dropbox repository
      2. Go to your private files and click add
      3. Navigate to a file in your Dropbox repository and select it
      4. Click 'Select this file'

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Michael de Raadt added a comment -

            Double-clicking in general is a big problem, and not just in the file picker.

            Show
            Michael de Raadt added a comment - Double-clicking in general is a big problem, and not just in the file picker.
            Hide
            Frédéric Massart added a comment -

            Yes, but in this case, you can click once and wait two seconds and click again because the UI does not show any sign of 'loading'.
            Especially when dealing with remote repositories such as Dropbox, Wikimedia, Google Docs, etc...
            I should have set the title to 'should prevent multiple clicks and show loading icon to be consistent with overall filepicker'

            Show
            Frédéric Massart added a comment - Yes, but in this case, you can click once and wait two seconds and click again because the UI does not show any sign of 'loading'. Especially when dealing with remote repositories such as Dropbox, Wikimedia, Google Docs, etc... I should have set the title to 'should prevent multiple clicks and show loading icon to be consistent with overall filepicker'
            Hide
            Marina Glancy added a comment -

            TO INTEGRATORS: please cherry-pick to 23_STABLE

            Show
            Marina Glancy added a comment - TO INTEGRATORS: please cherry-pick to 23_STABLE
            Hide
            Dan Poltawski added a comment -

            Integrated, thanks!

            Show
            Dan Poltawski added a comment - Integrated, thanks!
            Hide
            Ankit Agarwal added a comment -

            Works as described
            Passing!
            Thanks

            Show
            Ankit Agarwal added a comment - Works as described Passing! Thanks
            Hide
            Sam Hemelryk added a comment -

            Congratulations your code is upstream - gold star for you!

            This issue + 79 others made it in in time for the minor releases.
            Thank you everyone involved for your exuberant efforts.

            Show
            Sam Hemelryk added a comment - Congratulations your code is upstream - gold star for you! This issue + 79 others made it in in time for the minor releases. Thank you everyone involved for your exuberant efforts.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: