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

Remove the Confirm AMD modal type

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.4
    • 3.4
    • JavaScript, Usability
    • MOODLE_34_STABLE
    • MOODLE_34_STABLE
    • MDL-59759-master-2
    • Hide
      Course participants
      1. Login as a teacher to a course.
      2. Go to the course participants page.
      3. Click the trash icon on one of the students.
      4. Check the unernol modal.
        • Confirm that the buttons show "Unenrol" and "Cancel".
      5. Click on "Unenrol"
        • Confirm that the selected student gets unenrolled.
      Calendar
      1. Go to the calendar page and create a user event.
      2. Click on the user event to open the event modal.
      3. Click "Delete"
      4. Check the event deletion confirmation modal.
        • Confirm that the buttons show "Delete event" and "Cancel".
      5. Click on "Delete event"
        • Confirm that the event gets deleted.
      Messaging
      1. Log in as a student s1 and send one or more messages to another student s2.
      2. Log in as s2 and check your messages from s1.
      3. Click on "Edit" at the top right corner of the messaging window.
      4. Click on "Delete all"
      5. Check the message deletion confirmation modal.
        • Confirm that the buttons show "Delete" and "Cancel".
      6. Click on "Delete"
        • Confirm that all of the messages are deleted.
      Behat
      1. Initialise Behat to run on all themes: 

        php admin/tool/behat/cli/init.php -a

        (For a faster run time, use -j=n, where n is the number of parallel runs.)

      1. Run the following Behat test and confirm that everything passes: 

        php admin/tool/behat/cli/run.php --tags="@core_user,@core_calendar,@message"

       Backwards-compatibility test
      1. Undo the commit https://github.com/junpataleta/moodle/commit/fb7fa04e3b173e12fd152435472f3ef5a915d0df
      2. Go to the calendar page and create a user event.
      3. Click on the user event to open the event modal.
      4. Click "Delete"
      5. Check the event deletion confirmation modal. The buttons will show "Save changes" and "Cancel".
      6. Click on "Save changes"
        • Confirm that the event gets deleted.
      Show
      Course participants Login as a teacher to a course. Go to the course participants page. Click the trash icon on one of the students. Check the unernol modal. Confirm that the buttons show "Unenrol" and "Cancel". Click on "Unenrol" Confirm that the selected student gets unenrolled. Calendar Go to the calendar page and create a user event. Click on the user event to open the event modal. Click "Delete" Check the event deletion confirmation modal. Confirm that the buttons show "Delete event" and "Cancel". Click on "Delete event" Confirm that the event gets deleted. Messaging Log in as a student s1 and send one or more messages to another student s2. Log in as s2 and check your messages from s1. Click on " Edit " at the top right corner of the messaging window. Click on " Delete all " Check the message deletion confirmation modal. Confirm that the buttons show "Delete" and "Cancel". Click on "Delete" Confirm that all of the messages are deleted. Behat Initialise Behat to run on all themes: php admin/tool/behat/cli/init.php -a (For a faster run time, use -j= n , where n is the number of parallel runs.) Run the following Behat test and confirm that everything passes: php admin/tool/behat/cli/run.php --tags="@core_user,@core_calendar,@message"  Backwards-compatibility test Undo the commit https://github.com/junpataleta/moodle/commit/fb7fa04e3b173e12fd152435472f3ef5a915d0df Go to the calendar page and create a user event. Click on the user event to open the event modal. Click "Delete" Check the event deletion confirmation modal. The buttons will show "Save changes" and "Cancel". Click on "Save changes" Confirm that the event gets deleted.

    Description

      Yes/No dialogs are a no-no as it hinders usability. We need to remove it in order to discourage its use. 

      The way to go is to use the Save/Cancel type of AMD modal instead, in which has a customisable save button text.

      Attachments

        Issue Links

          Discovered while testing

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-59757 Delete icon makes a Yes/No dialog.

          • Minor - Minor loss of function where workaround is possible
          • Closed
          has to be done after

          Task - A task that needs to be completed. Tasks usually refer to work that must be done outside the product. MDL-60013 Final deprecation for core/modal_confirm

          • Minor - Minor loss of function where workaround is possible
          • Closed
          will help resolve

          Improvement - An enhancement to an existing Moodle feature. MDL-59937 Confirm delete modal should use "delete" and "cancel"

          • Minor - Minor loss of function where workaround is possible
          • Closed

          Activity

            People

              jpataleta Jun Pataleta
              jpataleta Jun Pataleta
              Andrew Lyons Andrew Lyons
              David Monllaó David Monllaó
              Ryan Wyllie Ryan Wyllie
              Andrew Lyons, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze, Huong Nguyen, Barbara Ramiro, Bas Brands, Mathew May
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                13/Nov/17