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

Click on point entity shows choropleth popup #10

Closed
nesnoj opened this issue Feb 6, 2024 · 5 comments · Fixed by #20
Closed

Click on point entity shows choropleth popup #10

nesnoj opened this issue Feb 6, 2024 · 5 comments · Fixed by #20
Assignees
Labels
bug 🐞 Something isn't working

Comments

@nesnoj
Copy link
Contributor

nesnoj commented Feb 6, 2024

When choropleth is active, clicking on a power unit (or other point entity) always shows the choropleth popup.
Both objects' popups should be accessible, depending on what is clicked on.

image

@nesnoj nesnoj added the bug 🐞 Something isn't working label Feb 6, 2024
@nesnoj
Copy link
Contributor Author

nesnoj commented Feb 20, 2024

Solved in rl-institut/digiplan#393 , I'll integrate here

@nesnoj
Copy link
Contributor Author

nesnoj commented Feb 20, 2024

depends on #12

@henhuy
Copy link
Contributor

henhuy commented Feb 20, 2024

Solved in rl-institut/digiplan#393 , I'll integrate here

This is actually a different bug - showing an empty popup.
-> point popup depends on order of Popups - but luckily I fixed this in #12
So merging it solves two problems

Nevertheless, #393 must still be implemented here

@henhuy
Copy link
Contributor

henhuy commented Feb 26, 2024

Nevertheless, #393 must still be implemented here

In fact, this is also covered by updating mapeninge...

@henhuy
Copy link
Contributor

henhuy commented Mar 1, 2024

In fact, this is also covered by updating mapeninge...

This is not true. Fixed it in newest mapengine version v0.18.3.
Add PR for this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐞 Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants