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

menshealth.com - Touch interactions are not working on mobile #1242

Closed
webcompat-bot opened this issue Jun 13, 2015 · 8 comments
Closed

menshealth.com - Touch interactions are not working on mobile #1242

webcompat-bot opened this issue Jun 13, 2015 · 8 comments

Comments

@webcompat-bot
Copy link

URL: http://www.menshealth.com/
Browser / Version: Firefox Mobile 41.0
Operating System: Android
Problem type: Something else - I'll add details below

Steps to Reproduce

  1. Navigate to: http://www.menshealth.com/

Expected Behavior:

Actual Behavior: menu button doesn't work. No menu overflow.

@karlcow
Copy link
Member

karlcow commented Jun 15, 2015

  • Top right menu is not working on tap
  • Carousel is not working
  • search button is not working on tap
  • share button in articles not working
  • bottom page with promoted stories is too large

@karlcow
Copy link
Member

karlcow commented Jun 15, 2015

top right menu

The markup for the top right menu is the following 😢

<div id="mobify-menu"><span id="mobify-menu-toggle">Menu</span>
    <div>
        <a title="Automobile" href="/">Home</a>
        <a title="Bizarre" href="/fitness/">Fitness</a>
        <a title="Business" href="/sex-women/">Sex &amp; Women</a>
        <a title="Critic" href="/health/">Health</a>
        <a title="Design" href="/nutrition/">Nutrition</a>
        <a title="Environment" href="/weight-loss/">Weight Loss</a>
        <a title="Style" href="/style/">Style</a>
        <a title="Lifestyle" href="/grooming/">Grooming</a>
        <a title="Video" href="/video/">Video</a>
        <a href="https://secure.rodale.com/webapp/wcs/stores/servlet/OaeEntryPage?storeId=10057&amp;mktOfferId=HLH62519&amp;keycode=I4EZ0A00&amp;cm_mmc=Menshealth.com-_-Mobile-_-MH%20Mobile%20Simple%20Nav-_-OfferId_HLH62519_keycode_I4EZ0A00">Subscribe</a>
        <a href="#" class="last" onclick="Mobify.desktop()">Full Site</a>
    </div>
</div>

ok Many resources are not loaded on Firefox Android.

14:29:14.118 ReferenceError: Mobify is not defined www.menshealth.com:65:31
14:29:14.334 ReferenceError: jQuery is not defined www.menshealth.com:90:5
14:29:14.834 ReferenceError: Mobify is not defined www.menshealth.com:170:1
14:29:14.875 ReferenceError: Mobify is not defined www.menshealth.com:469:0
14:29:14.909 TypeError: $ is not a function www.menshealth.com:475:1
14:29:14.938 ReferenceError: Mobify is not defined

@karlcow karlcow changed the title wyciwyg: - see bug description menshealth.com - Touch interactions are not working on mobile Jun 15, 2015
@sagrid
Copy link

sagrid commented Jun 15, 2015

The latest nightly doesn't have such a problem. However being so buggy on my device (videos doesn't exit from full screen, zoom level overwrite some clickable links etc) it was reinstalled and I must say that user experience is a little bit different.

@sagrid
Copy link

sagrid commented Jun 15, 2015

For "zoom level" read: doesn't have a proper click delay in relation with touch adjustment feature. That's not the case here but note that site compat is not under discussion here.

@karlcow
Copy link
Member

karlcow commented Jun 16, 2015

@sagrid I tested with Firefox Nightly on Android too. I confirm issues I put in the previous comment.
When you tap on the menu does it expand for you?

Could you share your UA when testing?

@sagrid
Copy link

sagrid commented Jun 16, 2015

Browser
Headers Mozilla/5.0 (Android; Tablet; rv:41.0) Gecko/41.0 Firefox/41.0
Match
JavaScript UA Mozilla/5.0 (Android; Tablet; rv:41.0) Gecko/41.0 Firefox/41.0 Menu, slide, search button, everything looks fine with latest build or 2015-06-02 running on Nexus 7 gen 1 Android 4.3.

@karlcow
Copy link
Member

karlcow commented Jun 16, 2015

Ah! Tablet. Let see if we have a different behavior. With the Tablet in the UA, you are receiving the desktop site not the mobile site. So indeed different behavior.

menshealth

The issues described above are for the Mobile site.

@hallvors
Copy link

Seems to work fine on Firefox Android and Firefox OS. I did also test on a Firefox OS tablet, seemed OK. I could not find any videos on the site.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants