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

Bubble map is not working #1576

Closed
1 of 3 tasks
nnixaa opened this issue Feb 28, 2018 · 4 comments · Fixed by #1655
Closed
1 of 3 tasks

Bubble map is not working #1576

nnixaa opened this issue Feb 28, 2018 · 4 comments · Fixed by #1655
Milestone

Comments

@nnixaa
Copy link
Collaborator

nnixaa commented Feb 28, 2018

Issue type

I'm submitting a ... (check one with "x")

  • bug report
  • feature request
  • question about the decisions made in the repository

Issue description

Current behavior:

http://akveo.com/ngx-admin/#/pages/maps/bubble
Expected behavior:

@MikeCullymore
Copy link

Was this fixed? I have downloaded the latest version and it broke my bubble map?

@SUlbrichA
Copy link

I managed to rid the error message on the console by adding

"scripts": [
"../node_modules/echarts/dist/echarts.min.js",
"../node_modules/echarts/dist/extension/bmap.min.js",
...
],

following this response

The map still does not show though. I also tried updating to the newest version of ngx-echarts.

@SUlbrichA
Copy link

I managed to get it running by looking closer at the ngx-echarts exmples

The problems appears to be that the world map is not registered. I copied the file world.json from node_modules into the assets directory and set the options in a callback:

this.themeSubscription = this.theme.getJsTheme().subscribe(config => {
    // .. set variables  
    this.http.get('assets/world.json').subscribe(geoJson => {
      this.es.registerMap('world', geoJson);
      this.options = {...}
   }
}

If that is the right solution, I'd be happy to submit a pull request

@nnixaa nnixaa added this to the v2.1.1 milestone Apr 26, 2018
@nnixaa
Copy link
Collaborator Author

nnixaa commented Apr 26, 2018

Hey @SUlbrichA, many thanks for your help!

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.

3 participants