Bug 605583 - Don't reopen vkb just because user switches to 'all pages' [r=mfinkle]
authorVivien Nicolas <21@vingtetun.org>
Thu, 04 Nov 2010 19:47:09 +0100
changeset 66961 a437c4da3694fc96619b03a386b4d462c758a3c2
parent 66960 4189d2e7bdb632a641253bff395d07bc78f8c155
child 66962 f322780547ead21377217ecca6b31ab44961e572
push id1
push userroot
push dateTue, 26 Apr 2011 22:38:44 +0000
treeherdermozilla-beta@bfdb6e623a36 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmfinkle
bugs605583
Bug 605583 - Don't reopen vkb just because user switches to 'all pages' [r=mfinkle]
mobile/chrome/content/browser-ui.js
--- a/mobile/chrome/content/browser-ui.js
+++ b/mobile/chrome/content/browser-ui.js
@@ -292,17 +292,17 @@ var BrowserUI = {
       this._edit.detachController();
       this.popDialog();
     }
 
     if (this._activePanel)
       this._activePanel.close();
 
     // The readOnly state of the field enabled/disabled the VKB
-    let isReadOnly = !(aPanel == AllPagesList && Util.isPortrait());
+    let isReadOnly = !(aPanel == AllPagesList && Util.isPortrait() && (willShowPanel || !this._edit.readOnly));
     this._edit.readOnly = isReadOnly;
     if (isReadOnly)
       this._edit.blur();
 
     this._activePanel = aPanel;
     if (willHidePanel || willShowPanel) {
       let event = document.createEvent("UIEvents");
       event.initUIEvent("NavigationPanel" + (willHidePanel ? "Hidden" : "Shown"), true, true, window, false);