mendixlabs / SimpleCharts

The Mendix SimpleCharts widget.
https://appstore.home.mendix.com/link/app/139/Mendix/Simple-Chart
Apache License 2.0
0 stars 5 forks source link

Widget not loading when javascript bundled #9

Open simo101 opened 8 years ago

simo101 commented 8 years ago

There seems to be an issue when bundling the javascript. I have experienced this in 6.3.1. This causes issues on the mobile and fails to load the application. This is probably down to loading of minified libaries in the code.

ghartwich commented 8 years ago

Hi. I'm experiencing the same problem in MX6.3.1. When I run the application locally on my pc the application works correctly, but when I run the same application in the sandbox the application doesn't run: I only get a blank page. The console gives no errors, just these warnings:

mxui.js?635947941302653910:21 The XMLHttpRequest progress event property 'position' is deprecated and will be removed in M50, around April 2016. Please use 'loaded' instead. See https://www.chromestatus.com/features/5044837464145920 for more details.

mxui.js?635947941302653910:21 Synchronous XMLHttpRequest on the main thread is deprecated because of its detrimental effects to the end user's experience. For more help, check https://xhr.spec.whatwg.org/.

Jelte, do you have any suggestion what I can do to remedy this?

Cheers, Gonzo

JelteMX commented 8 years ago

Those two warnings actually have nothing to do with the widget, these can be ignored (have seen them around as well).

As for bundling Javascript, this is a common problem. I'll look into it when I can. SimpleCharts is actually a very old widget that has Community Support (as in, we're not actively supporting this widget). For a project we needed to release a MX6 compatible version, but as you can see at the releases (two pre-releases), these are not stable.

If I have time, I'll see if we can come up with a more stable version. We advice people to use ChartJS instead of SimpleCharts for now

rvanosnabrugge commented 8 years ago

I've just created a pull request where I've fixed the widget bundling bug.