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

Drag and drop message overlapping Activity chooser

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3
    • Component/s: Course
    • Labels:

      Description

      Replication steps:

      1. Resize browser a bit smaller
      2. go to a course page
      3. refresh browser
      4. click the activity chooser right away

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            dobedobedoh Andrew Nicols added a comment -

            Need to set a z-index on the dndupload-status

            Show
            dobedobedoh Andrew Nicols added a comment - Need to set a z-index on the dndupload-status
            Hide
            dobedobedoh Andrew Nicols added a comment -

            Turns out that it's the statusbar at fault not the chooser. I think the transition causes the status bar to take precedence if no z-index is set.
            Setting a z-index of 0 should restore world order.

            Tested in:

            • Opera 12 - Mac
            • Firefox 13 - Mac
            • Chrome 19 - Mac
            Show
            dobedobedoh Andrew Nicols added a comment - Turns out that it's the statusbar at fault not the chooser. I think the transition causes the status bar to take precedence if no z-index is set. Setting a z-index of 0 should restore world order. Tested in: Opera 12 - Mac Firefox 13 - Mac Chrome 19 - Mac
            Hide
            barbararamiro Barbara Ramiro added a comment -

            Thanks Andrew for taking this one

            Show
            barbararamiro Barbara Ramiro added a comment - Thanks Andrew for taking this one
            Hide
            poltawski Dan Poltawski added a comment -

            Hey Davo,

            Thought you might want to peer review this one.

            Show
            poltawski Dan Poltawski added a comment - Hey Davo, Thought you might want to peer review this one.
            Hide
            davosmith Davo Smith added a comment - - edited

            All looks good to me.

            I can't put forward for integration review - Dan could you do that?

            Show
            davosmith Davo Smith added a comment - - edited All looks good to me. I can't put forward for integration review - Dan could you do that?
            Hide
            dobedobedoh Andrew Nicols added a comment -

            Cheers Davo

            Show
            dobedobedoh Andrew Nicols added a comment - Cheers Davo
            Hide
            poltawski Dan Poltawski added a comment -

            Davo: could you email michaeld@moodle.com asking to be a pull requester.

            Show
            poltawski Dan Poltawski added a comment - Davo: could you email michaeld@moodle.com asking to be a pull requester.
            Hide
            davosmith Davo Smith added a comment -

            I've dropped him a Moodle message a few min ago - I assume that will be sufficient.

            Show
            davosmith Davo Smith added a comment - I've dropped him a Moodle message a few min ago - I assume that will be sufficient.
            Hide
            nebgor Aparup Banerjee added a comment -

            ah i remember seeing htis. thanks, integrated into master and also this workzz for me.

            Show
            nebgor Aparup Banerjee added a comment - ah i remember seeing htis. thanks, integrated into master and also this workzz for me.
            Hide
            andyjdavis Andrew Davis added a comment -

            Works as described. Passing.

            Show
            andyjdavis Andrew Davis added a comment - Works as described. Passing.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            And this has been incorporated to all the weekly builds and also, to Moodle 2.3 Release Candidate 1, yay!

            Many, many thanks for your hard work!

            Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - And this has been incorporated to all the weekly builds and also, to Moodle 2.3 Release Candidate 1, yay! Many, many thanks for your hard work! Ciao

              People

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

                Dates

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