Fix Safari Download - #5609

Thanks @rlreamy - it's interesting that this works now, but it clearly didn't in past versions of Safari (and some docs haven't caught up yet, like https://www.w3schools.com/jsref/prop_anchor_download.asp still says "Not Supported")

Can we instead move the special handling after the if(canUseSaveLink) block? That way older versions that fail this test will still at least have the previous behavior.

It would be really nice to get this PR to the finish line for the upcoming 2.1.0 release.
@rlreamy could you possibly address @alexcjohnson's comment above?

It is possible, but I cannot promise anything. This change was done as part of a work project, and I have not yet had time to revisit.
I can tell you that this change doesn't quite fix the issue, however. It does improve the situation, in that you will always get the correct filename now. However, there is a strange behavior that the first time you download the png, you get the axes, but not plot (for the scatter plots that we are generating with thousands of points). But, the second time on, it downloads everything as it should.
Another developer on my team will be revisiting this work in the next 2 weeks, and I will make sure to make a note of this request there.

Thanks very much @rlreamy for the information.
It looks something we could fix in a patch release.
So for now I dropped it from v2.1.0 milestone.
I am looking forward to the follow up whenever you had time.

💃 Thanks for the PR.
Merging into safari-download branch.