Skip to content

FIREFLY-162: fixed download feature not working in firefox and safari #839

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

Merged
merged 1 commit into from
Jul 19, 2019

Conversation

loitly
Copy link
Contributor

@loitly loitly commented Jul 19, 2019

https://jira.ipac.caltech.edu/browse/FIREFLY-162
Test link: https://irsawebdev9.ipac.caltech.edu/FIREFLY-162_save_firefox_safari/firefly/

As reported in ticket, downloading is no longer working in Firefox and Safari.
This is due to a recent chance in the code where the <A> tag was not added to the HTML document before triggering a click event. Not testing it in Firefox and Safari was an oversight.
You should definitely test the fix in all browsers. :)

  • also, minor tweaked to File Upload layout

@loitly loitly requested a review from ejoliet July 19, 2019 00:58
@loitly loitly self-assigned this Jul 19, 2019
@lrebull
Copy link
Contributor

lrebull commented Jul 19, 2019

Yes, I confirm that it works for me on Firefox now.

Copy link
Contributor

@ejoliet ejoliet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed! Thanks!
(Yes, we should request to reviewers to test in all browsers everytime, at least one different per reviewer maybe)

@loitly loitly merged commit 0e26acc into dev Jul 19, 2019
@ejoliet ejoliet deleted the FIREFLY-162_save_firefox_safari branch July 19, 2019 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants