Skip to content
This repository has been archived by the owner on Oct 8, 2021. It is now read-only.

enhanceWithin not respecting keepNative in page linked to with data-rel=dialog #3120

Closed
odedbd opened this issue Nov 20, 2011 · 3 comments
Closed

Comments

@odedbd
Copy link

odedbd commented Nov 20, 2011

Hi,

The title says it all. I had to modify enhanceWithin to the following in order to get it to work (note the added jqmData(role='dialog') selector to .closest).

        enhanceWithin: function( target ) {
            // TODO remove dependency on the page widget for the keepNative.
            // Currently the keepNative value is defined on the page prototype so
            // the method is as well

            //OBD - if this is a dialog through a data-rel=dialog link, the role will be dialog when we get here...
            var page = $(target).closest(":jqmData(role='page'),:jqmData(role='dialog')").data( "page" ),
            keepNative = (page && page.keepNativeSelector()) || "";

            $( this.options.initSelector, target ).not( keepNative )[ this.widgetName ]();
        }
@odedbd
Copy link
Author

odedbd commented Nov 21, 2011

looks like eromba@a8c0485 has implemented this fix, and much more (added to tests etc.)

@odedbd odedbd closed this as completed Nov 21, 2011
@odedbd odedbd reopened this Dec 3, 2011
@odedbd
Copy link
Author

odedbd commented Dec 3, 2011

Oops, seems like the aforementioned commit fixed a similar issue, but in a different place in the code. So this is still open afaik.

@johnbender
Copy link
Contributor

Closed here: 978762d

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants