JQM Navigation : remove page min-height
authorJinhyuk Jun <jinhyuk.jun@samsung.com>
Fri, 29 Mar 2013 07:27:04 +0000 (16:27 +0900)
committerYoumin Ha <youmin.ha@samsung.com>
Sat, 30 Mar 2013 12:49:10 +0000 (21:49 +0900)
remove JQM's page min-height calculation, tizen-web-ui using content's min-height calculation to get min scrollview height

Change-Id: I2184867d5a157c3abccfc136c90ef7231394a7e4

libs/patch/0039-JQM-remove-useless-page-min-height-calculate.patch [new file with mode: 0644]
src/themes/tizen/common/jquery.mobile.core.less

diff --git a/libs/patch/0039-JQM-remove-useless-page-min-height-calculate.patch b/libs/patch/0039-JQM-remove-useless-page-min-height-calculate.patch
new file mode 100644 (file)
index 0000000..941d840
--- /dev/null
@@ -0,0 +1,46 @@
+From 79081b53a6708d4391edac5ed6b3981c7ae3616c Mon Sep 17 00:00:00 2001
+From: Jinhyuk Jun <jinhyuk.jun@samsung.com>
+Date: Fri, 29 Mar 2013 16:20:48 +0900
+Subject: [PATCH] JQM remove useless page min-height calculate
+
+Change-Id: I6598b5fafea8463c6714f12a62ad9575d019d4c9
+---
+ .../js/jquery.mobile.navigation.js                 |   15 ---------------
+ 1 file changed, 15 deletions(-)
+
+diff --git a/libs/js/jquery-mobile-1.2.0/js/jquery.mobile.navigation.js b/libs/js/jquery-mobile-1.2.0/js/jquery.mobile.navigation.js
+index 0a727b4..0be7861 100644
+--- a/libs/js/jquery-mobile-1.2.0/js/jquery.mobile.navigation.js
++++ b/libs/js/jquery-mobile-1.2.0/js/jquery.mobile.navigation.js
+@@ -579,17 +579,6 @@ define( [
+               return promise;
+       }
+-      //simply set the active page's minimum height to screen height, depending on orientation
+-      function resetActivePageHeight() {
+-              var aPage = $( "." + $.mobile.activePageClass ),
+-                      aPagePadT = parseFloat( aPage.css( "padding-top" ) ),
+-                      aPagePadB = parseFloat( aPage.css( "padding-bottom" ) ),
+-                      aPageBorderT = parseFloat( aPage.css( "border-top-width" ) ),
+-                      aPageBorderB = parseFloat( aPage.css( "border-bottom-width" ) );
+-
+-              aPage.css( "min-height", getScreenHeight() - aPagePadT - aPagePadB - aPageBorderT - aPageBorderB );
+-      }
+-
+       //shared page enhancements
+       function enhancePage( $page, role ) {
+               // If a role was specified, make sure the data-role attribute
+@@ -1552,10 +1541,6 @@ define( [
+                       $.mobile._handleHashChange( path.parseLocation().hash );
+               });
+-              //set page min-heights to be device specific
+-              $.mobile.$document.bind( "pageshow", resetActivePageHeight );
+-              $.mobile.$window.bind( "throttledresize", resetActivePageHeight );
+-
+       });//navreadyDeferred done callback
+ })( jQuery );
+-- 
+1.7.9.5
+
index 31d9663..41048ba 100644 (file)
@@ -18,7 +18,7 @@
 .ui-mobile-viewport {  margin: 0; overflow-x: hidden; -webkit-tap-highlight-color: rgba(0, 0, 0, 0); }
 
 /* "page" containers - full-screen views, one should always be in view post-pageload */
-.ui-mobile [data-role=page], .ui-mobile [data-role=dialog], .ui-page { top: 0; left: 0; width: 100%; min-height: 100%; position: absolute; display: none; border: 0; } 
+.ui-mobile [data-role=page], .ui-mobile [data-role=dialog], .ui-page { top: 0; left: 0; width: 100%; position: absolute; display: none; border: 0; }
 .ui-mobile .ui-page-active { display: block; overflow: visible; }
 
 /* on ios4, setting focus on the page element causes flashing during transitions when there is an outline, so we turn off outlines */