Last modified: 2014-02-12 23:54:11 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T43517, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 41517 - Windows 8/RT Internet Explorer 10 given non-JS version of mobile site
Windows 8/RT Internet Explorer 10 given non-JS version of mobile site
Status: RESOLVED FIXED
Product: MobileFrontend
Classification: Unclassified
stable (Other open bugs)
unspecified
All Windows 8
: Unprioritized normal
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-10-29 23:23 UTC by Brion Vibber
Modified: 2014-02-12 23:54 UTC (History)
9 users (show)

See Also:
Web browser: Internet Explorer
Mobile Platform: Windows 8 / Windows RT
Assignee Huggle Beta Tester: ---


Attachments

Description Brion Vibber 2012-10-29 23:23:21 UTC
If requesting the mobile site in IE 10 on my Surface RT, I get the non-JavaScript version of the mobile site:
* 'go' button in the search bar
* no typeahead search
* no collapsed sections

Windows 8/rt gets the desktop site by default, but some folks may prefer the touch site so we shouldn't explode on it. :)
Comment 1 Jon 2012-10-30 17:53:02 UTC
This makes sense based on the current regexs:
https://gerrit.wikimedia.org/r/gitweb?p=mediawiki/extensions/MobileFrontend.git;a=blob;f=includes/DeviceDetection.php;h=7a020373e0a39af9859c691b9145ac6dd0371d29;hb=HEAD#l319

Should be made more generic

There are several other device detection issues that could be picked up at the same time (see https://mingle.corp.wikimedia.org/projects/mobile/cards/214)
Comment 2 Jon 2012-11-02 23:05:42 UTC
https://gerrit.wikimedia.org/r/31581 highlights the bug by making a failing test.
Test needs to be fixed in core and then in equivalent code for varnish
Comment 3 Brion Vibber 2012-11-05 06:05:54 UTC
Note this seems to hit IE 10 on Windows Phone 8 as well (I don't have my Win8 machine with me to run the SDK emulator on so can't confirm a fix right now, but can test once back from India.)
Comment 5 Arthur Richards 2012-11-27 18:32:00 UTC
Changes merged to master

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links