Moodle
  1. Moodle
  2. MDL-33722

Drag and drop message overlapping Activity chooser

    Details

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

      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

        Activity

        Hide
        Andrew Nicols added a comment -

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

        Show
        Andrew Nicols added a comment - Need to set a z-index on the dndupload-status
        Hide
        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
        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
        Barbara Ramiro added a comment -

        Thanks Andrew for taking this one

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

        Hey Davo,

        Thought you might want to peer review this one.

        Show
        Dan Poltawski added a comment - Hey Davo, Thought you might want to peer review this one.
        Hide
        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
        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
        Andrew Nicols added a comment -

        Cheers Davo

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

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

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

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

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

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

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

        Works as described. Passing.

        Show
        Andrew Davis added a comment - Works as described. Passing.
        Hide
        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
        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: