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

Commit

Permalink
check the data role to see if it's a header or footer, not the class …
Browse files Browse the repository at this point in the history
…from the page sections plugin. This allows for a looser order of calling the page sections and fixedtoolbar plugins on an element, which is a small improvement, and fixes #3875 in the process.
  • Loading branch information
scottjehl committed Apr 3, 2012
1 parent 203e00b commit 94c0aeb
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions js/jquery.mobile.fixedToolbar.js
Expand Up @@ -72,7 +72,7 @@ define( [ "jquery", "./jquery.mobile.widget", "./jquery.mobile.core", "./jquery.
var self = this,
o = self.options,
$el = self.element,
tbtype = $el.is( ".ui-header" ) ? "header" : "footer",
tbtype = $el.is( ":jqmData(role='header')" ) ? "header" : "footer",
$page = $el.closest(".ui-page");

// Feature detecting support for
Expand Down Expand Up @@ -189,7 +189,7 @@ define( [ "jquery", "./jquery.mobile.widget", "./jquery.mobile.core", "./jquery.
elHeight = $el.height(),
pHeight = $el.closest( ".ui-page" ).height(),
viewportHeight = $.mobile.getScreenHeight(),
tbtype = $el.is( ".ui-header" ) ? "header" : "footer";
tbtype = $el.is( ":jqmData(role='header')" ) ? "header" : "footer";

return !notransition &&
( this.options.transition && this.options.transition !== "none" &&
Expand Down

0 comments on commit 94c0aeb

Please sign in to comment.