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

mobile.aireuropa.com - The "MENU" button is not working #34760

Closed
eyalyoli opened this issue Jul 7, 2019 · 10 comments
Closed

mobile.aireuropa.com - The "MENU" button is not working #34760

eyalyoli opened this issue Jul 7, 2019 · 10 comments
Labels
browser-firefox-mobile engine-gecko The browser uses the Gecko rendering engine priority-normal severity-critical The site or core functionality is unusable, or you would probably open another browser to use it. status-first-contact
Milestone

Comments

@eyalyoli
Copy link

eyalyoli commented Jul 7, 2019

URL: https://mobile.aireuropa.com/h5/mobile/en/register

Browser / Version: Firefox Mobile 68.0
Operating System: Android 8.0.0
Tested Another Browser: Yes

Problem type: Site is not usable
Description: Side menu button doesn't work & form submit button doesn't work
Steps to Reproduce:
When you click on menu button on upper left, it should open a side menu. When you fill the form on the page it should move on with the subscription but instead it jumps up and doesn't say anything.
Screenshot Description

Browser Configuration
  • None

From webcompat.com with ❤️

@webcompat-bot webcompat-bot added this to the needstriage milestone Jul 7, 2019
@webcompat-bot webcompat-bot added browser-firefox-mobile engine-gecko The browser uses the Gecko rendering engine labels Jul 7, 2019
@mozilianulb
Copy link

Thanks for the report @eyalyoli , I am able to reproduce this issue. When tapping on "Menu" button nothing happens.

Browser / Version: Firefox Nightly 68.1a1 (2019-07-09)
Operating System: OnePlus 2 (Android 6.0.1) - 1080 x 1920 pixels, 16:9 ratio (~401 ppi density)

image

@cipriansv cipriansv changed the title mobile.aireuropa.com - site is not usable mobile.aireuropa.com - The "MENU" button is not working Jul 10, 2019
@cipriansv cipriansv added priority-normal severity-critical The site or core functionality is unusable, or you would probably open another browser to use it. labels Jul 10, 2019
@cipriansv cipriansv modified the milestones: needstriage, needsdiagnosis Jul 10, 2019
@ksy36 ksy36 self-assigned this Jul 16, 2019
@ksy36
Copy link
Contributor

ksy36 commented Jul 29, 2019

Regarding the form submission, it works for me and I was able to subscribe.

As for the menu, this event listener is attached to it:

       /**
       * Setup live binds for show/hide actions on marked elements
       */
            $(document).on('click', '.translate3d.hasFixed .menuize_show-left, .translate3d.hasFixed .menuize_show-right', function(e) {
                e.preventDefault();

                if (!status.current) {
                    var that = $(this);
                    var panelId = that.hasClass('menuize_show-left') ? 'left' : 'right';

                    show(panelId);
                } else {
                    hidePanel();
                }

            });

So it looks for an element with a class .translate3d.hasFixed .menuize_show-left.
In case of Firefox for Android hasFixed is missing because of the UA detection.
It starts here:

hasFixed: device.hasPositionFixed || device.isIE11 || device.isDesktop,

Then the actual detection is happening here:

hasPositionFixed: window.Worker && f || m || v || w || x || k || D,
, a = navigator.userAgent
      , s = /Web[kK]it[\/]{0,1}([\d.]+)/.test(a)
      , c = /(iPhone\sOS)\s([\d_]+)/.test(a)
      , l = /(iPod)(.*OS\s([\d_]+))?/.test(a)
      , u = /(iPad).*OS\s([\d_]+)/.test(a)
      , f = c || l || u
      , d = !!a.match(/\(Macintosh\; Intel /)
      , h = /(Android);?[\s\/]+([\d.]+)?/.test(a)
      , p = /android 2./gi.test(a)
      , m = /android 3./gi.test(a)
      , v = /android 4./gi.test(a)
      , g = /Android /.test(r) && !/Chrome\/\d/.test(r)
      , y = /(BlackBerry).*Version\/([\d.]+)/.test(a)
      , b = y && /version\/6/gi.test(a)
      , w = y && /version\/7/gi.test(a)
      , x = /(BB10).*Version\/([\d.]+)/.test(a)
      , T = /MSIE\s([\d.]+)/.test(a) || /Trident\/[\d](?=[^\?]+).*rv:([0-9.].)/.test(a)
      , E = /MSIE 9/gi.test(a)
      , k = /MSIE 10/gi.test(a)
      , S = /Trident\/[\d](?=[^\?]+).*rv:([0-9.].)/.test(a)
      , D = /Chrome\/([\d.]+)/.test(a) || /CriOS\/([\d.]+)/.test(a)
      , P = /Firefox\/([\d.]+)/.test(a)
      , C = /\((?:Mobile|Tablet); rv:([\d.]+)\).*Firefox\/[\d.]+/.test(a)

So User Agent detection should be changed in order to for the menu to work (it accounts only for android version 4).

@ksy36
Copy link
Contributor

ksy36 commented Jul 29, 2019

Considering the above code, when I spoof as Chrome, the menu opens up. The menu icon is shifted to the left though, and when I click on it, the elements inside the flyout panel look different. I'll investigate this further.

@ksy36
Copy link
Contributor

ksy36 commented Jul 29, 2019

So the menu is shifted to the left:

Screen Shot 2019-07-29 at 3 23 49 PM

is caused by this

.main-header-inner .main-menu-button{
     width: 60px;
      ....
}

.main-header-inner a {

    display: table-cell;
    height: 50px;
    width: 42px;
    max-width: 42px;
    text-decoration: none;
    text-align: center;
    vertical-align: middle;
    color: #FFFFFF;
    background-position: center;
    background-repeat: no-repeat;
}

Adding max-width: 60px; to the first rule fixes the issue for me.

@ksy36
Copy link
Contributor

ksy36 commented Jul 29, 2019

The flyout panel looks like this:

Screen Shot 2019-07-29 at 3 23 58 PM

And the code:

.links-list.with-icons li > a {
	display: table;
	min-height: 50px;
	padding: 0px;
}

Adding height: 50px; fixes the issue:

Screen Shot 2019-07-29 at 3 34 51 PM

I guess it's related to https://bugzilla.mozilla.org/show_bug.cgi?id=307866 , but lets contact the site to see if they could fix this

@adamopenweb
Copy link
Collaborator

Contacted via LinkedIn.

@ksy36
Copy link
Contributor

ksy36 commented Jul 29, 2019

Reached out via contact form as well and got identification number WEB051623

@ksy36 ksy36 removed their assignment Jul 29, 2019
@ksy36 ksy36 added the action-needssitepatch This web bug needs a GoFaster site patch. label Jul 29, 2019
@ksy36
Copy link
Contributor

ksy36 commented Aug 2, 2019

@cipriansv cipriansv modified the milestones: sitewait, fixed Aug 30, 2019
@cipriansv
Copy link

After retesting the issue I confirm that the issue has been fixed. Tapping on the "MENU" button displayed the flyout panel containing the menu items.

image

Tested with:
Browser / Version: Firefox Nightly 68.1a1 (2019-08-27), Firefox Preview 1.3.1 (Build #12341950)
Operating System: Huawei P10 (Android 8.0) - 1080 x 1920 pixels (~432 ppi pixel density)

@lock
Copy link

lock bot commented Sep 6, 2019

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue at https://webcompat.com/issues/new if you are experiencing a similar problem.

@lock lock bot locked as resolved and limited conversation to collaborators Sep 6, 2019
@ksy36 ksy36 removed the action-needssitepatch This web bug needs a GoFaster site patch. label Sep 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
browser-firefox-mobile engine-gecko The browser uses the Gecko rendering engine priority-normal severity-critical The site or core functionality is unusable, or you would probably open another browser to use it. status-first-contact
Projects
None yet
Development

No branches or pull requests

6 participants