Closed Bug 960432 Opened 10 years ago Closed 9 years ago

nifty.com / cocolog-nifty.com sends desktop site to Firefox OS and Firefox on Android

Categories

(Web Compatibility :: Site Reports, defect)

All
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: hsteen, Assigned: karlcow)

References

()

Details

(Whiteboard: [clientsniff] [country-jp] [sitewait])

Attachments

(2 files)

Attached image 035.png
This is the browser sniffing that redirects some browsers to the mobile site (which seems to run well in Gecko, BTW):

<script type="text/javascript">
//<![CDATA[
var checkBrowserType=function(){var A=navigator.userAgent;this.IE=(A.indexOf("MSIE")!==-1);this.Gecko=(A.indexOf("Gecko")!==-1);this.Firefox=(A.indexOf("Firefox")!==-1);this.IE9=(A.indexOf("Trident/5.0")!==-1);this.IE8=(A.indexOf("Trident/4.0")!==-1);this.IE7=(A.indexOf("MSIE 7.")!==-1)&&(A.indexOf("Trident/4.0")===-1)&&(A.indexOf("Trident/5.0")===-1);this.IE6=(A.indexOf("MSIE 6.")!==-1);this.IE5=(A.indexOf("MSIE 5.")!==-1);this.IE50=(A.indexOf("MSIE 5.0")!==-1);this.IE4=(A.indexOf("MSIE 4.")!==-1);this.IE3=(A.indexOf("MSIE 3.")!==-1);this.Opera=(A.indexOf("Opera")!==-1);this.Chrome=(A.indexOf("Chrome")!==-1);this.Safari=(A.indexOf("Safari")!==-1)&&(A.indexOf("Chrome")===-1)&&(A.indexOf("Silk")===-1)&&(A.indexOf("Kindle")===-1);this.Safari2=(A.indexOf("Safari")!==-1)&&(A.indexOf("Chrome")===-1)&&(A.indexOf("Version")===-1)&&(A.indexOf("Silk")===-1);this.NS=(A.indexOf("Netscape")!==-1);this.iPhone=(A.indexOf("iPhone")!==-1);this.iPod=(A.indexOf("iPod")!==-1);this.Android=(A.indexOf("Android")!==-1);this.Android3=(A.indexOf("Android 3")!==-1);this.Android1=(A.indexOf("Android 1")!==-1);this.Android2=(A.indexOf("Android 2")!==-1);this.AndroidTab=(A.indexOf("Android")!==-1)&&(A.indexOf("Mobile")===-1);this.iPad=(A.indexOf("iPad")!==-1);this.Kindle=(A.indexOf("Silk")!==-1)||(A.indexOf("Kindle")!==-1);this.MSafari=(A.indexOf("Mobile")!==-1)&&(A.indexOf("Safari")!==-1)};(function(){var B=new checkBrowserType();if(B.iPhone||B.iPod||B.Android1||B.Android2||(B.Android&&B.MSafari)){if(location.search!="?pc"){var p=location.search.substr(1)||"top";location.href="http://www.nifty.com/smp/?"+p}}else{if(B.AndroidTab||B.iPad||B.Kindle){if(location.search!="?pc"){location.href="http://www.nifty.com/tablet.html"+location.search}}else{if(B.IE5||B.IE50||B.IE4||B.IE3||B.Safari2||B.NS){location.href="http://www.nifty.com/text/"}}}})();
Attached image 061.png
www.cocolog-nifty.com should be fixed too, another screenshot for that
Summary: nifty.com sends desktop site to Firefox OS and Firefox on Android → nifty.com / cocolog-nifty.com sends desktop site to Firefox OS and Firefox on Android
They have basically in that script posted by Hallvord, 3 options (prettified for readibility):

(function () {
    var B = new checkBrowserType();
    if (B.iPhone || B.iPod || B.Android1 || B.Android2 || (B.Android && B.MSafari)) {
        if (location.search != '?pc') {
            var p = location.search.substr(1) || 'top';
            location.href = 'http://www.nifty.com/smp/?' + p
        }
    } else {
        if (B.AndroidTab || B.iPad || B.Kindle) {
            if (location.search != '?pc') {
                location.href = 'http://www.nifty.com/tablet.html' + location.search
            }
        } else {
            if (B.IE5 || B.IE50 || B.IE4 || B.IE3 || B.Safari2 || B.NS) {
                location.href = 'http://www.nifty.com/text/'
            }
        }
    }
}) ();


To note that the checkBrowserType() is also used a few times in another script for the desktop site.


The smp site has another layer of UA sniffing 
http://www.nifty.com/smp/

(function () {
    var u = navigator.userAgent;
    var p = '';
    if (u.indexOf('iPhone') != - 1) {
        p = '&SP-DET=iphone';
    } else if (u.indexOf('iPad') != - 1) {
        p = '&SP-DET=ipad';
    } else if (u.indexOf('Android') != - 1) {
        if (u.indexOf('Docomo') != - 1) {
            p = '&SP-DET=android&robot=false';
        } else {
            p = '&SP-DET=android';
        }
    }
    CM8Profile = (window.CM8Profile || '') + p;
}) ();


also calling this script might create some issues on Firefox Android vs WebKit interactions.
http://www.nifty.com/smp/js/jquery.flicksimple.min.js

 * jQuery.flickSimple v1.1.1
 *
 * Copyright (c) 2011 Makog. http://d.hatena.ne.jp/makog/
Trying a human contact
http://www.linkedin.com/in/takashinakaizumi
http://twitter.com/MozWebCompat/status/469634163331457024
Assignee: nobody → kdubost
Status: NEW → ASSIGNED
Whiteboard: [clientsniff] [country-jp] [contactready] → [clientsniff] [country-jp] [sitewait]
No luck with this contact. He is not very active on twitter.
Maybe there are better contacts among Japanese team. Added in Cc:
Trying another person on twitter
http://twitter.com/MozWebCompat/status/554501858031464448
Trying to find another way to contact them.
I found https://github.com/higebu
And sent an email 

I wonder if Japanese team has contacts with
Flags: needinfo?(kudo)
We are sorry for the too late reply. I'll tell the right person this issue soon.
Thanks a lot Yuya. Appreciated.
Flags: needinfo?(kudo)
We fixed this issue. Could you please confirm it?
Yuya, Fabulous! 

On Firefox OS and Firefox Android
http://www.nifty.com/ gets redirected to
http://www.nifty.com/smp/?top

Excellent.

Let's close it as FIXED.


I'm not sure it's part of the same team, but in case.
Firefox OS 
http://www.cocolog-nifty.com/ gets http://www.cocolog-nifty.com/
Firefox Android 
http://www.cocolog-nifty.com/ gets http://www.cocolog-nifty.com/smart/

Thanks.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Product: Tech Evangelism → Web Compatibility
Component: Mobile → Site Reports
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: