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

weather.com - see bug description #2517

Closed
dhdurgee opened this issue May 4, 2016 · 5 comments
Closed

weather.com - see bug description #2517

dhdurgee opened this issue May 4, 2016 · 5 comments

Comments

@dhdurgee
Copy link

dhdurgee commented May 4, 2016

URL: https://weather.com/weather/hourbyhour/l/20877:4:US
Browser / Version: Firefox 43.0
Operating System: Ubuntu
Problem type: Something else - I'll add details below

Steps to Reproduce

  1. Navigate to: https://weather.com/weather/hourbyhour/l/20877:4:US

Expected Behavior: should display current hour-by-hour forecast

Actual Behavior: "upgrade now" message overlays display and cannot be closed

Screenshot of the site issue

From webcompat.com with ❤️

@dhdurgee
Copy link
Author

dhdurgee commented May 4, 2016

Minor clarification, actual browser is SeaMonkey 2.40 here. I am using PrefBar to spoof the UA as FireFox 43, which generally avoids issues such as the above.

@adamopenweb
Copy link
Collaborator

Thanks for the report @dhdurgee! As mentioned the site is using user agent sniffing to try to detect the browser and displays an overlay when the detection fails. The overlay does not want to close in Gecko and when we check the console we see:
TypeError: event.srcElement is undefined

event.srcElement is non-standard and unsupported in Gecko.
https://developer.mozilla.org/en-US/docs/Web/API/Event/srcElement

It appears a few times on the site, but this looks to be the offender here:
https://weather.com/sites/all/modules/custom/angularmods/assets/older_browser_overlay/older_browser_overlay.js

@adamopenweb
Copy link
Collaborator

event.target can be used instead of event.srcElement
or
(event.target || event.srcElement)
if they're worried about supporting older versions of IE.

Switching to needscontact.

@adamopenweb
Copy link
Collaborator

adamopenweb commented May 4, 2016

Maybe @twc-rhedin or @vadlamak can help out with this one?

@adamopenweb
Copy link
Collaborator

I can't get this to reproduce on my end anymore, I think it's fixed. @dhdurgee can you confirm?

@karlcow karlcow added this to the fixed milestone Oct 30, 2017
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