Jdlrobson has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/374405 )

Change subject: Drop Android2 check for search
......................................................................

Drop Android2 check for search

Android2 is a grade C browser so this code will never be run in
MediaWiki.

Change-Id: I184df4fbea3743d54b24e3782c4d1debf7ff9095
---
M resources/skins.minerva.scripts/search.js
1 file changed, 8 insertions(+), 13 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/skins/MinervaNeue 
refs/changes/05/374405/1

diff --git a/resources/skins.minerva.scripts/search.js 
b/resources/skins.minerva.scripts/search.js
index 10d73ee..6bbdf9c 100644
--- a/resources/skins.minerva.scripts/search.js
+++ b/resources/skins.minerva.scripts/search.js
@@ -39,18 +39,13 @@
                return;
        }
 
-       // See https://phabricator.wikimedia.org/T76882 for why we disable 
search on Android 2
-       if ( browser.isAndroid2() ) {
-               $( 'body' ).addClass( 'client-use-basic-search' );
-       } else {
-               // don't use focus event 
(https://bugzilla.wikimedia.org/show_bug.cgi?id=47499)
-               //
-               // focus() (see SearchOverlay#show) opens virtual keyboard only 
if triggered
-               // from user context event, so using it in route callback won't 
work
-               // 
http://stackoverflow.com/questions/6837543/show-virtual-keyboard-on-mobile-phones-in-javascript
-               $( '#searchInput, #searchIcon' ).on( 'click', openSearchOverlay 
)
-                       // Apparently needed for main menu to work correctly.
-                       .prop( 'readonly', true );
-       }
+       // don't use focus event 
(https://bugzilla.wikimedia.org/show_bug.cgi?id=47499)
+       //
+       // focus() (see SearchOverlay#show) opens virtual keyboard only if 
triggered
+       // from user context event, so using it in route callback won't work
+       // 
http://stackoverflow.com/questions/6837543/show-virtual-keyboard-on-mobile-phones-in-javascript
+       $( '#searchInput, #searchIcon' ).on( 'click', openSearchOverlay )
+               // Apparently needed for main menu to work correctly.
+               .prop( 'readonly', true );
 
 }( mw.mobileFrontend, jQuery ) );

-- 
To view, visit https://gerrit.wikimedia.org/r/374405
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I184df4fbea3743d54b24e3782c4d1debf7ff9095
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/skins/MinervaNeue
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <jrob...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to