Download SVG/PNG not working on mobile - Githubissues

appreciated / apexcharts-flow

ApexCharts.js wrapper for the Vaadin Platform

Apache License 2.0

76 stars 46 forks source link

Download SVG/PNG not working on mobile #16

Closed ugurdoksan closed 4 years ago

ugurdoksan commented 5 years ago

Download SVG/PNG

works fine in web browsers 👍 does not work in mobile browsers 👎

I am using the latest version of Chrome browser...

appreciated commented 5 years ago

Is it just pie chart and donut chart for you aswell? Since this seems to be default in this library.

With the other charts it works on my Android device

ugurdoksan commented 5 years ago

For example Line Chart download button is working on js, but not working on Java in my Android device (Samsung S9+)

appreciated commented 5 years ago

Was somehow fixed by upgrading the apex-charts dependency.

ugurdoksan commented 5 years ago

@appreciated is not working on my device "Samsung S9+" again

appreciated commented 5 years ago

This is a "Tomcat" or (maybe Spring Boot) specific problem. It works when I use the addon demo which is running on jetty (and without springboot) . But it does not work when I use the directory demo 🤔 quite weird. Are you also using Spring Boot by any chance?

ugurdoksan commented 5 years ago

I do not use Spring Boot with my Vaadin Project @appreciated.

appreciated commented 5 years ago

I was able to reproduce the issue. The download does not work on mobile devices when adding the charts f.e. to a form-layout. Adding the charts to a div works just fine. Using a VerticalLayout as in the directory demo doesn't work either.

appreciated commented 5 years ago

So to reproduce the issue with a Div simply add display:flex to it and the download won't work on mobile devices anymore.

appreciated commented 5 years ago

I assume that this has something to do with the fact that I wrapped apexcharts.js inside of a webcomponent.

So I see a few options here:

either this is going to be fixed in the offical repository for which a extensive issue report with an non-working example is necessary.
Or the webcomponent wrapper needs to be removed and this addon needs to make usage of JS scripts instead.

Either way this is going to take time, which I currently do not have.

appreciated commented 5 years ago

@ugurdoksan This was possibly fixed by the Polymer 3 port, I've got to check back on this before releaseing the 2.0.0.beta1

appreciated commented 4 years ago

This issue is fixed in the new version