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

register.vvc.edu - Site does not fully load #47362

Closed
webcompat-bot opened this issue Dec 28, 2019 · 3 comments
Closed

register.vvc.edu - Site does not fully load #47362

webcompat-bot opened this issue Dec 28, 2019 · 3 comments
Labels
browser-focus-geckoview engine-gecko The browser uses the Gecko rendering engine ml-needsdiagnosis-false ML - no need for diagnosis priority-normal severity-critical The site or core functionality is unusable, or you would probably open another browser to use it. type-uaoverride Require a UA override for working wcrt-invalid Invalid issues for the new webcompat reporting tool
Milestone

Comments

@webcompat-bot
Copy link

URL: https://register.vvc.edu/?S=c7b69653-7bb9-48c0-abb5-dc1342260465

Browser / Version: Firefox Mobile 71.0
Operating System: Android 7.1.1
Tested Another Browser: No

Problem type: Site is not usable
Description: wont oad entiresitel
Steps to Reproduce:

Browser Configuration
  • None

From webcompat.com with ❤️

@webcompat-bot webcompat-bot added this to the needstriage milestone Dec 28, 2019
@webcompat-bot webcompat-bot added browser-focus-geckoview engine-gecko The browser uses the Gecko rendering engine ml-needsdiagnosis-false ML - no need for diagnosis labels Dec 28, 2019
@softvision-oana-arbuzov softvision-oana-arbuzov added severity-critical The site or core functionality is unusable, or you would probably open another browser to use it. priority-normal labels Dec 30, 2019
@softvision-oana-arbuzov softvision-oana-arbuzov changed the title register.vvc.edu - site is not usable register.vvc.edu - Site does not fully load Dec 30, 2019
@softvision-oana-arbuzov
Copy link
Member

Thanks for the report, I was able to reproduce the issue.
image

Note:

  1. The issue is not reproducible on Chrome.
  2. The issue is reproducible on Firefox regardless of Tracking Protection is enabled or not.

Tested with:
Browser / Version: Firefox Focus 8.0.24 (🦎 71.0a1-20191003093956), Firefox Nightly 68.4a1 (2019-12-26), Firefox Preview Nightly 191230 (🦎: 73.0a1-20191227094418)
Operating System: Huawei P20 Lite (Android 8.0.0) - 1080 x 2280 pixels, 19:9 ratio (~432 ppi density), Samsung Galaxy S7 Edge (Android 8.0.0) - Resolution 1440 x 2560 pixels (~534 ppi pixel density)

Moving to Needsdiagnosis for further investigation.

@ksy36
Copy link
Contributor

ksy36 commented Dec 30, 2019

When I spoof as Chrome the site loads. There is a check for OS in this function:

    function H(a) {
        var b = w[a]()
          , c = v[a];
        if (b in c) {
            return b
        }
        var d = [];
        for (var e in c) {
            d[c[e]] = e
        }
        if (A) {
            A(a, d, b)
        }
        throw null
   }

where w[a]() is:

      , ec = 'linux'
      , cc = 'mac os x'
      , bc = 'macintosh'
      , gc = 'win32'
      , fc = 'windows'
       _b = 'unknown'

    w[ac] = function() {
        var a = m.navigator.userAgent.toLowerCase();
        if (a.indexOf(bc) != -1 || a.indexOf(cc) != -1) {
            return dc
        }
        if (a.indexOf(ec) != -1) {
            return ec
        }
        if (a.indexOf(fc) != -1 || a.indexOf(gc) != -1) {
            return fc
        }
        return _b
    }

The difference between Firefox and Chrome is that since android is not in the list for conditions to check, OS in case of Firefox on mobile is returned as 'unknown'. Chrome has "Linux" in the UA string Mozilla/5.0 (Linux; Android 6.0; ..., so the OS in case of Chrome mobile is returned as 'linux'.

So in Firefox we reach this chunk of code and A is not defined, so an error (null) is being thrown:

 if (A) {
      A(a, d, b)
   }
   throw null

To fix the issue the site could perhaps adjust the checks for OS in w[ac] and accommodate Android in a similar way they do it for Linux. I'll move this to needscontact

@ksy36 ksy36 added the type-uaoverride Require a UA override for working label Dec 30, 2019
@ksy36 ksy36 modified the milestones: needsdiagnosis, needscontact Dec 30, 2019
@softvision-raul-bucata
Copy link

softvision-raul-bucata commented Aug 3, 2021

Unfortunately, the link provided is not a valid link anymore and can not be reached using other browsers and devices:

Screenshot_55

Notes:
Various VPN locations were tested

Tested with:
Browser: Firefox Nightly 92.0a1 (2015825739 -🦎92.0a1-20210730093652🦎)
Operating System: Samsung A51 (Android 11) -1080 × 2400 pixels 20:9 aspect ratio (~405 ppi density)

Closing the issue.

@ksy36 ksy36 added the wcrt-invalid Invalid issues for the new webcompat reporting tool label Oct 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser-focus-geckoview engine-gecko The browser uses the Gecko rendering engine ml-needsdiagnosis-false ML - no need for diagnosis priority-normal severity-critical The site or core functionality is unusable, or you would probably open another browser to use it. type-uaoverride Require a UA override for working wcrt-invalid Invalid issues for the new webcompat reporting tool
Projects
None yet
Development

No branches or pull requests

4 participants