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

Problems with Safari and the new "overwrite files" dialog

    Details

    • Database:
      Any
    • Testing Instructions:
      Hide

      To test, please use safari browse
      1. create forum post
      2. upload a file
      3. upload the same file with same name, a pop up dialog should appear on top

      Show
      To test, please use safari browse 1. create forum post 2. upload a file 3. upload the same file with same name, a pop up dialog should appear on top
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Pull from Repository:
      git@github.com:dongsheng/moodle.git
    • Pull Master Branch:
      s9_MDL-27381_overwrite_dialog_safari_master

      Description

      This is one followup of MDL-26388.

      In the testing phase it has been detected that the new "overwrite files" dialog, allowing to replace/rename/cancel is shown under any other dialog (file picker, add image...) when using the Safari browser (see attached image).

      It would be great to have this fixed ASAP for quick integration. Ciao

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Triaging and assigning straight to Dongsheng.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Triaging and assigning straight to Dongsheng.
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Hi DS,

            This looks like its going to be as simple as setting a higher z-index on the overlay.
            If it is that simple could you please get a patch up and I'll integrate it today in time for the 2.0.3 release.

            Cheers
            Sam

            Show
            samhemelryk Sam Hemelryk added a comment - Hi DS, This looks like its going to be as simple as setting a higher z-index on the overlay. If it is that simple could you please get a patch up and I'll integrate it today in time for the 2.0.3 release. Cheers Sam
            Hide
            dongsheng Dongsheng Cai added a comment -

            For undknown reason, safari completely ignore zindex property, I set zindex to 99999992, in safari, it uses zindex:11 no matter what I put in js

            Show
            dongsheng Dongsheng Cai added a comment - For undknown reason, safari completely ignore zindex property, I set zindex to 99999992, in safari, it uses zindex:11 no matter what I put in js
            Hide
            dongsheng Dongsheng Cai added a comment -

            Found the solution, seems YUI failed to make modal dialog really 'modal', I removed modal option, use zIndex property solve the problem

            Show
            dongsheng Dongsheng Cai added a comment - Found the solution, seems YUI failed to make modal dialog really 'modal', I removed modal option, use zIndex property solve the problem
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks DS this has been integrated now.

            Cheers
            Sam

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks DS this has been integrated now. Cheers Sam
            Hide
            nebgor Aparup Banerjee added a comment -

            tested on safari for windows.
            the dialog appears fine for me in the proper Z order.

            (note: the file choosing dialog needed one more click as if focus was still elsewhere, but that may have been effects of my terminal client connecting to windows.)

            Show
            nebgor Aparup Banerjee added a comment - tested on safari for windows. the dialog appears fine for me in the proper Z order. (note: the file choosing dialog needed one more click as if focus was still elsewhere, but that may have been effects of my terminal client connecting to windows.)
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Yay, as a Safari-er, many thanks. Closing, ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Yay, as a Safari-er, many thanks. Closing, ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  5/May/11