Code review tool no longer needs to work around position:fixed handling on iPad
authorabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 2 Apr 2012 03:51:50 +0000 (03:51 +0000)
committerabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 2 Apr 2012 03:51:50 +0000 (03:51 +0000)
https://bugs.webkit.org/show_bug.cgi?id=82850

Reviewed by Eric Seidel.

We no longer need to avoid position:fixed on iPad because Mobile Safari
now does something reasonable with position:fixed elements.

* code-review.js:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@112827 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Websites/bugs.webkit.org/ChangeLog
Websites/bugs.webkit.org/code-review.js

index f99e980..e995b83 100644 (file)
@@ -1,3 +1,15 @@
+2012-04-01  Adam Barth  <abarth@webkit.org>
+
+        Code review tool no longer needs to work around position:fixed handling on iPad
+        https://bugs.webkit.org/show_bug.cgi?id=82850
+
+        Reviewed by Eric Seidel.
+
+        We no longer need to avoid position:fixed on iPad because Mobile Safari
+        now does something reasonable with position:fixed elements.
+
+        * code-review.js:
+
 2012-03-09  Ashod Nakashian  <ashodnakashian@yahoo.com>
 
         Bash scripts should support LF endings only
index 824c150..d133484 100644 (file)
@@ -1063,11 +1063,6 @@ var CODE_REVIEW_UNITTEST;
   }
 
   function updateToolbarAnchorState() {
-    // For iPad, we always leave the toolbar at the bottom of the document
-    // because of the iPad's handling of position:fixed and scrolling.
-    if (navigator.platform.indexOf("iPad") != -1)
-      return;
-
     var toolbar = $('#toolbar');
     // Unanchor the toolbar and then see if it's bottom is below the body's bottom.
     toolbar.toggleClass('anchored', false);