The placeholder margins left & top are not included in the revert position calcing causing a gap if margin(s) are defined.

Here's a fix ( works fine FF & Safari ) :

Line 288 - Function _mouseStop :

if(this.options.revert) { var self = this; var cur = self.placeholder.offset(); self.reverting = true;

var margin_left=parseInt(self.placeholder.css('marginLeft'),10);
var margin_top=parseInt(self.placeholder.css('marginTop'),10);

$(this.helper).animate({

left: cur.left - margin_left .......... ,
top: cur.top - margin_top ...........

}, parseInt(this.options.revert, 10)

500, function() { self._clear(event); });

Hi guyz!

I have a code like this:

<div id="container-1>
  <ul>
  <li><a href="#fragment-1"><span><span>
                            <img src="Images/pdf_16.png" height="12" alt="" style="border: 0px solid #000000;" /></span>
                          </span></a></li>
  <li><a href="#fragment-2"><span><span>
                            <img src="Images/pdf_16.png" height="12" alt="" style="border: 0px solid #000000;" /></span>
                          </span></a></li>
  </ul>
<div id="fragment-1">Pippo</div>
<div id="fragment-2">
        <!--File Documenti-->
        <table width="100%">
            <tr>
                <td>
                    <div>
                        <iframe width="100%" height="500px" style="border:1px solid #888888;"></iframe>
                    </div>
                </td>
            </tr>
        </table>
</div>
</div>

... programmaticallyI change the iframe.src with the location of a page that stream a PDF file with the "response.binarywrite / content type" method.

Result: all works fine in IE, FF, Chrome and Opera, but with the Windows version of Safari when I change tab from "fragment-2" to "fragment-1" the iframe with the streamed document still appears all over the tab control!

I though it's a z-index issue but changing it doesn't affect the result.

Thx and c u!

Zwonimir

Hi,

Talking about the following commit:

​https://github.com/jquery/jquery-ui/commit/3829a37ca122e923c3a08b964c4b1a946a2a1456

The new moveToTop() function breaks iframe usage because of this line of code:

var moved = this.uiDialog.nextAll( ":visible" ).insertBefore( this.uiDialog );

The insertBefore() usage makes the iframe to be detached and reattached from the DOM; you can have occasional javascript errors or entire iframe refresh, depending on which browser you use; i think it is because of this:

​http://bugs.jquery.com/ticket/8533

What do u think?

I know it's not a jQuery ui problem, but how can we avoid this ?

Thank you for your work.