Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

www.vw.ca - site is not usable #78042

Open
karlcow opened this issue Jun 23, 2021 · 9 comments
Open

www.vw.ca - site is not usable #78042

karlcow opened this issue Jun 23, 2021 · 9 comments
Labels
browser-firefox-ios os-ios Issues only happening on iOS. type-uaoverride Require a UA override for working type-unsupported Doesn't support one or more browser
Milestone

Comments

@karlcow
Copy link
Member

karlcow commented Jun 23, 2021

URL: https://www.vw.ca/fr.html

Browser / Version: Firefox iOS 34.1
Operating System: iOS 14.6
Tested Another Browser: Yes Safari

Problem type: Site is not usable
Description: Browser unsupported
Steps to Reproduce:
Going to https://www.vw.ca/

Expected
Access to the site

Actual
Message saying Firefox is not supported

View the screenshot Screenshot
Browser Configuration
  • None

From webcompat.com with ❤️

@webcompat-bot webcompat-bot added this to the needstriage milestone Jun 23, 2021
@webcompat-bot webcompat-bot added browser-firefox-ios os-ios Issues only happening on iOS. labels Jun 23, 2021
@karlcow karlcow modified the milestones: needstriage, needsdiagnosis Jun 23, 2021
@karlcow karlcow added the type-unsupported Doesn't support one or more browser label Jun 23, 2021
@karlcow
Copy link
Member Author

karlcow commented Jun 23, 2021

The message sent is

<div
  id="vwaUnsupportedBrowserNotification"
  data-heading="Votre navigateur n'est pas supporté. Afin d'optimiser votre expérience sur le site ainsi que la rapidité et la sécurité, prenez le temps de mettre à jour votre navigateur.&lt;br&gt;
"
  data-text="&lt;p&gt;S'il vous plaît, pensez à mettre à jour votre navigateur.&lt;/p&gt;
"
>
  <div id="vwaUBNHitarea"></div>
  <div class="vwaUBNModal">
    <div class="vwaUBNModalInner"></div>
  </div>
  <div id="vwaUBNCloseButton"></div>
</div>

They are using a script
https://www.vw.ca/idhub/etc/clientlibs/vwa-ngw18/ngw18-frontend/clientlibs-unsupportedBrowserNotification-proxy.f4f91e69a1b68fc20a832c62c3c35232.js

the support browser matrix is

        // browser matrix as defined in https://sso.volkswagen.de/bctwiki/display/FEDEVRO/NGW+D6+Browser+Matrix
        var browserMatrix = {
            safari: '11',
            chrome: '68',
            firefox: '61',
            msie: '17', // all IE are outdated
            msedge: '17',
            opera: '55',
            samsungBrowser: '7.2'
        };

        var isUnsupportedBrowser = bowser.isUnsupportedBrowser(
            browserMatrix,
            true
        );

The script is using the bowser library

/*!
 * Bowser - a browser detector
 * https://github.com/ded/bowser
 * MIT License | (c) Dustin Diaz 2015
 */

which correctly identifies Firefox on iOS

firefox: true
ios: true
iphone: true
name: "Firefox"
osname: "iOS"
osversion: "14.4.2"
version: "34.0"
webkit: true

What is happening is that Firefox on iOS has currently the version number 34, because Firefox on iOS can NOT use Gecko. All browsers are forced to use WebKitWebView for rendering websites.

So the site correctly identifies Firefox but fails to identifies that this is Firefox on iOS. They think it's Firefox on Android. They do not use all the information that is sent by Bowser.

The user agent string has the following shape.

Mozilla/5.0 (iPhone; CPU iPhone OS 14_4_2 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) FxiOS/34.0  Mobile/15E148 Safari/605.1.15

This is an easy fix. And we can fix it with a site intervention too.

@karlcow karlcow modified the milestones: needsdiagnosis, needscontact Jun 23, 2021
@webcompat-bot
Copy link

@karlcow karlcow assigned karlcow and unassigned denschub Jun 23, 2021
@karlcow karlcow added action-needssitepatch This web bug needs a GoFaster site patch. type-uaoverride Require a UA override for working labels Jun 23, 2021
@karlcow
Copy link
Member Author

karlcow commented Jun 23, 2021

This might be a long stretch, but maybe @elgohr can help us find the right person at Volkswagen Canada, so they can fix their user agent detection matrix to take into account Firefox on iOS as described above.

@karlcow karlcow modified the milestones: needscontact, sitewait Jun 23, 2021
@karlcow
Copy link
Member Author

karlcow commented Jun 23, 2021

See the thread on twitter too.
https://twitter.com/bslobodin/status/1407134859315892227

@elgohr
Copy link

elgohr commented Jun 23, 2021

@alexey-irkhin

@karlcow karlcow removed their assignment Aug 17, 2021
@softvision-raul-bucata
Copy link

The issue has been fixed and it is no longer reproducible:

Screenshot_6

Tested with:

Browser/Version: Firefox Daylight 100.1 (9384)
Operating System: iOS 15.4

Closing this as FIXED

[inv_20/2022]

@softvision-oana-arbuzov
Copy link
Member

softvision-oana-arbuzov commented Jun 27, 2022

I was able to reproduce the issue on my iPad.
image

If I request "Mobile Site", the message is not displayed. Behavior not encountered on Chrome.
image

Note: Could be related to mozilla-mobile/firefox-ios#9150

Tested with:
Browser / Version: Firefox Daylight 101.1 (10346)
Operating System: iPad Mini 4 iOS 15.4.1 (1536 x 2048 pixels (~324 ppi density))

Reopening the issue for further investigation.

[inv_26/2022]

@softvision-oana-arbuzov softvision-oana-arbuzov removed this from the fixed milestone Jun 27, 2022
@softvision-oana-arbuzov softvision-oana-arbuzov added this to the needscontact milestone Jun 27, 2022
@webcompat-bot
Copy link

@softvision-raul-bucata
Copy link

Not reproducible on an iPhone device:

A26D1560-FEF4-4031-82FB-E2FEFD6ECE92
297F08B2-539E-4E04-9A81-F4352826D3E4

Tested with:

Browser/Version: Firefox Daylight 101.1 (10346)
Operating System: iOS 15.5

[inv_26/2022]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser-firefox-ios os-ios Issues only happening on iOS. type-uaoverride Require a UA override for working type-unsupported Doesn't support one or more browser
Projects
None yet
Development

No branches or pull requests

6 participants