Details

    • Testing Instructions:
      Hide
      1. Open the course, enable editing
      2. Take the section before the last one and try moving it down far below the last one (the ghost section will be displayed as the last one)
      3. Release the mouse button and observe that the section order has been updated (drop has been processed as if you dropped normally right over the target section)
        (see video in description for example)
      Show
      Open the course, enable editing Take the section before the last one and try moving it down far below the last one (the ghost section will be displayed as the last one) Release the mouse button and observe that the section order has been updated (drop has been processed as if you dropped normally right over the target section) (see video in description for example)
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-32752-master-1

      Description

      If the dragged element is moved a bit further toward the area that has no drop target and dropped, the drop_hit will not be fired, the layout will be corrupted and after page refresh the page will look as initially.

      Bug demo: http://www.youtube.com/watch?v=z6h5MVlmRts&feature=youtu.be

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Michael de Raadt added a comment -

            That happens in the Men's room around here.

            Show
            Michael de Raadt added a comment - That happens in the Men's room around here.
            Show
            Ruslan Kabalin added a comment - Demo added: http://www.youtube.com/watch?v=z6h5MVlmRts&feature=youtu.be
            Hide
            Ruslan Kabalin added a comment -

            Andrew, will you have chance to review it?

            Show
            Ruslan Kabalin added a comment - Andrew, will you have chance to review it?
            Hide
            Dan Poltawski added a comment -

            Looks good to me, thanks Ruslan!

            Show
            Dan Poltawski added a comment - Looks good to me, thanks Ruslan!
            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
            Ruslan Kabalin added a comment -

            Made the integrator life easier (this week only).

            Show
            Ruslan Kabalin added a comment - Made the integrator life easier (this week only).
            Hide
            Sam Hemelryk added a comment -

            Thanks guys has been integrated now

            Show
            Sam Hemelryk added a comment - Thanks guys has been integrated now
            Hide
            Sam Hemelryk added a comment -

            Tested during integration review and passed

            Show
            Sam Hemelryk added a comment - Tested during integration review and passed
            Hide
            Dan Poltawski added a comment -

            Congratulations!

            Your work has made into the latest Moodle release!

            You are only authorised to celebrate after testing 15 Moodle 2.3 QA tests, thanks!

            Show
            Dan Poltawski added a comment - Congratulations! Your work has made into the latest Moodle release! You are only authorised to celebrate after testing 15 Moodle 2.3 QA tests, thanks!
            Hide
            Marina Glancy added a comment -

            It seems that this issue is the cause of MDL-34209 with 106 votes

            Show
            Marina Glancy added a comment - It seems that this issue is the cause of MDL-34209 with 106 votes

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: