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
    • Rank:
      39744

      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

        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: