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.

Sign up for GitHub

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

Jump to bottom

onRegionSelect - region is undefined #201

Closed
D1M opened this issue Jan 18, 2016 · 6 comments
Closed

onRegionSelect - region is undefined #201

D1M opened this issue Jan 18, 2016 · 6 comments
Labels
Bug Report

Comments

@D1M
Copy link

D1M commented Jan 18, 2016

onRegionClick: function(element, code, region) - region is defined.
onRegionSelect: function(element, code, region) - region is undefined for all codes.
@eugene-sh
Copy link

@D1M
Write it on the line 468
regionClickEvent = $.Event('regionClick.jqvmap');
After
jQuery(params.container).trigger('regionClick.jqvmap', [code, mapData.pathes[code].name]);

@manifestinteractive
Copy link
Contributor

Looks like something got lost there, I will take a look into why this did not get passed over.

@manifestinteractive
Copy link
Contributor

Can you verify if this issue is resolved in this branch ?

https://github.com/manifestinteractive/jqvmap/tree/v1.5.0_prerelease

@eugene-sh
Copy link

@manifestinteractive
Doesn't work. It's my project my_repository
In the folder 'js/libs/jqvmap'. working 'jquery.vmap.js'

manifestinteractive added a commit that referenced this issue Mar 16, 2016
@manifestinteractive
1. Updated LICENSE path in packaged dist files
a53a51e 
2. Fixed bugs in labels and pins where mouse events were not passing through to clickable region
3. Removed -merc suffix from map files created by map creator ( fixes #204 )
4. Added new example for custom placement of pins `./examples/pins_custom.html`
5. Fixed `onRegionSelect` issue not returning region ( fixes #201 )
6. Added better support to make regions disabled ( see #197 and new `./examples/inactive_regions.html` )
@Thoum
Copy link

Thoum commented Mar 22, 2016

@manifestinteractive : Seems the issue is not solved, i still have the problem with the latest version.
The code is defined, but not the region.

@JeansReal
Copy link

Same problem here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Report
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@manifestinteractive @D1M @JeansReal @Thoum @eugene-sh