Skip to main content

Search and Top Navigation

#8984 closed bug (fixed)

Opened January 18, 2013 08:25PM UTC

Closed January 26, 2013 04:36PM UTC

Last modified June 04, 2013 04:12PM UTC

Dialog: Modal dialogs display behind overlay when using appendTo option

Reported by: limitstudios Owned by:
Priority: major Milestone: 1.10.1
Component: ui.dialog Version: 1.10.0
Keywords: regression Cc:
Blocked by: Blocking:
Description

Using the code provided at jsfiddle or url below and clicking the button causes the overlay to position above the dialog, making the dialog unusable.

http://jsfiddle.net/rFAkt/

http://www.limit-studios.com/jquery/jquery-dialog.php

Attachments (0)
Change History (14)

Changed January 18, 2013 08:30PM UTC by limitstudios comment:1

The reason it is attached to a form is the dialog contains form fields which are then submitted via the dialog

Changed January 18, 2013 08:33PM UTC by scottgonzalez comment:2

keywords: → regression
milestone: none1.10.1
priority: minormajor
status: newopen
summary: jQueryUI 1.10 Overlay is above dialogDialog: Modal dialogs display behind overlay when using appendTo option

A properly reduced test case: http://jsfiddle.net/rFAkt/2/

Changed January 19, 2013 03:42PM UTC by jzaefferer comment:3

Gonna look into this, along with the open dialog PR.

Changed January 19, 2013 05:17PM UTC by mikesherov comment:4

owner: → joern.zaefferer
status: openassigned

Changed January 25, 2013 02:23PM UTC by jzaefferer comment:5

owner: joern.zaefferer
status: assignedopen

Since David is working on other dialog bugs and I'm not going to tackle this in the next few days, unassigning myself...

Changed January 25, 2013 04:01PM UTC by C6AC165B-7B1E-445F-804B-A1BFED06162F comment:6

_comment0: "Modal dialogs display behind overlay when using appendTo option" \ \ And they do not display at all when used in a fully scoped css theme. I have to make hacks to work around this and the issue of the ticket. Such as: \ \ {{{ \ var dz = $(".ui-front").css( "z-index" ) \ $(".ui-widget-overlay").css({ "z-index": dz-1 }); \ $(".ui-widget-overlay").appendTo("#my-scoping-id"); \ }}} \ \ I have a long list of bugs that occur when using scoped css themes. Will post in time. I only just now managed to register here, I tried all kinds of random strings as a username last night (there is no way someone has laready registered as "vdfuh5ngsby7atbva3dhpoltdysb", yet I was told that name was already in use...), then it occurred to me to register as a GUID, that worked. So here I am, and once I can be bothered to compile and document the long list of bugs that I have observed, I will post them.1359129758386175
_comment1: "Modal dialogs display behind overlay when using appendTo option" \ \ And they do not display at all when used in a fully scoped css theme. I have to make hacks to work around this and the issue of the ticket. Such as: \ \ {{{ \ var dz = $(".ui-front").css( "z-index" ) \ $(".ui-widget-overlay").css({ "z-index": dz-1 }); \ $(".ui-widget-overlay").appendTo("#my-scoping-id"); \ }}} \ \ I have a long list of bugs that occur when using scoped css themes. Will post in time. I only just now managed to register here, I tried all kinds of random strings as a username last night (there is no way someone has already registered as "vdfuh5ngsby7atbva3dhpoltdysb", yet I was told that name was already in use...), then it occurred to me to register as a GUID, that worked. So here I am, and once I can be bothered to compile and document the long list of bugs that I have observed, I will post them.1359130652452958
_comment2: "Modal dialogs display behind overlay when using appendTo option" \ \ And overlays do not display at all when used in a fully scoped css theme. I have to make hacks to work around this and the issue of the ticket. Such as: \ \ {{{ \ var dz = $(".ui-front").css( "z-index" ) \ $(".ui-widget-overlay").css({ "z-index": dz-1 }); \ $(".ui-widget-overlay").appendTo("#my-scoping-id"); \ }}} \ \ I have a long list of bugs that occur when using scoped css themes. Will post in time. I only just now managed to register here, I tried all kinds of random strings as a username last night (there is no way someone has already registered as "vdfuh5ngsby7atbva3dhpoltdysb", yet I was told that name was already in use...), then it occurred to me to register as a GUID, that worked. So here I am, and once I can be bothered to compile and document the long list of bugs that I have observed, I will post them.1359130729990874

"Modal dialogs display behind overlay when using appendTo option"

And overlays do not display at all when used in a fully scoped css theme. I have to use hacks to work around this and the issue of the ticket. Such as:

var dz = $(".ui-front").css( "z-index" )
$(".ui-widget-overlay").css({ "z-index": dz-1 });
$(".ui-widget-overlay").appendTo("#my-scoping-id");

I have a long list of bugs that occur when using scoped css themes. Will post in time. I only just now managed to register here, I tried all kinds of random strings as a username last night (there is no way someone has already registered as "vdfuh5ngsby7atbva3dhpoltdysb", yet I was told that name was already in use...), then it occurred to me to register as a GUID, that worked. So here I am, and once I can be bothered to compile and document the long list of bugs that I have observed, I will post them.

Changed January 26, 2013 11:00AM UTC by djQuery comment:7

Submitted pull request

https://github.com/jquery/jquery-ui/pull/894

Let me know if there is any more work that needs to be done on this.

Changed January 26, 2013 04:36PM UTC by David Sullivan comment:8

resolution: → fixed
status: openclosed

Dialog: modified _createOverlay to _appendTo instead of document.body. Fixed #8984 - Modal dialogs display behind overlay when using appendTo option

Changeset: 55360eeb7eae5c560d51b09178e64d83c59223a6

Changed January 31, 2013 08:09PM UTC by limitstudios comment:9

I added the commited code to my jqueryui file and it still didn't fix my test case

Changed January 31, 2013 08:11PM UTC by scottgonzalez comment:10

@limitstudios We'd need a reduced test case showing the problem.

Changed January 31, 2013 08:35PM UTC by limitstudios comment:11

yep I'm trying to create a reduce one, but only since upgraded to 1.10.0 has caused this

Changed February 19, 2013 06:35AM UTC by MosheElisha comment:12

The fix in 1.10.1 works for me. Thanks!

Changed June 04, 2013 04:04PM UTC by TUX comment:13

Comenten esta linea de codigo del Jquery y con eso se soluciona:

$(".ui-dialog:visible:last .ui-dialog-content")

.data("ui-dialog")._focusTabbable();

Saludos desde Merida, Yucatan, Mexico...

Changed June 04, 2013 04:12PM UTC by scottgonzalez comment:14

@TUX Please stop posting that comment in various closed tickets.