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

Links to failure lists don't work in Firefox #181

Closed
bzbarsky opened this issue Sep 22, 2017 · 4 comments · Fixed by #184
Closed

Links to failure lists don't work in Firefox #181

bzbarsky opened this issue Sep 22, 2017 · 4 comments · Fixed by #184

Comments

@bzbarsky
Copy link

STEPS TO REPRODUCE (in Firefox):

  1. Load https://web-confluence.appspot.com/#!/confluence
  2. Click "FAILURE TO SHIP".
  3. Select any data point (the first data point at the "550" level for Safari, aka Safari 8.0, will do).
  4. Click the little box-with-arrow after the number in the resulting popup.

EXPECTED RESULTS: Opens the actual list of things failing to ship in a new tab. That's what it does in Chrome and Safari. I can't test what it does with Edge, because in Edge I don't see the graph at all.

ACTUAL RESULTS: Nothing happens.

It's entirely possible that this is a Firefox issue, of course. I'm happy to do some debugging, but it would be fairly helpful to know what the site is actually trying to do here, without having to reverse-engineer it from the minified source.

@mdittmer
Copy link
Collaborator

This is caused by using a synthetic <a>.click(). "Download CSV" is probably also affected. I'm half-done a fix, but probably can't land it until early next week.

@bzbarsky
Copy link
Author

This is caused by using a synthetic <a>.click()

Is the anchor not in a document? Looks like until 8 hours ago whatwg/html#2615 was open with discussion of what the actual behavior should be....

mdittmer added a commit that referenced this issue Sep 25, 2017
@bzbarsky
Copy link
Author

Thank you!

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

Successfully merging a pull request may close this issue.

2 participants