960826 - gnavi.co.jp sends desktop site to Firefox OS

Status: RESOLVED WONTFIX

(Web Compatibility :: Site Reports, defect, P5)

Description

[Hallvord R. M. Steen [:hallvors]]

Attachment: 067.png

http://x.gnst.jp/md/md.js

function md(t,q){var u=navigator.userAgent;if(t=='pc'){if(((/android/i.test(u) && /mobile/i.test(u)) || (!/ipad/i.test(u) && /ip(hone|od)/i.test(u))) && !(/smp_pcview/.test(document.cookie))){location.replace(q || 'http://mobile.gnavi.co.jp/iphone/top/');}}else if(t == 's'){if(!(/android/i.test(u) && /mobile/i.test(u)) && ( /ipad/i.test(u) || !/ip(hone|od)/i.test(u))) {location.replace(q || 'http://www.gnavi.co.jp/');}}}

Comment 1

[Karl Dubost💡 :karlcow]

They could improve Web compatibility and simplify at the same time:

function md(t, q) {
  var u = navigator.userAgent.toLowerCase();
  if (t == 'pc') {
    if ((
    // testing for mobile devices
    (/mobile/i.test(u) ||
    /android/i.test(u) ||
    /blackberry/i.test(u))
    &&
    // removing tablets
    !(/ipad/i.test(u) ||
    /tablet/i.test(u))
    ) && !(/smp_pcview/.test(document.cookie))) {
      location.replace(q || 'http://mobile.gnavi.co.jp/iphone/top/');
    }
  } else if (t == 's') {
    if (/ipad/i.test(u) || /tablet/i.test(u)) {
      location.replace(q || 'http://www.gnavi.co.jp/');
    }
  }
}


It's not fool proof but it's better for the redirection part.

There will be other issues which affects Firefox Android and Firefox OS. I will open a separate bug.

Comment 2

[Sylvestre Ledru [:Sylvestre]]

Closing as we are not working on Firefox OS anymore.