Skip to content

Fix map clicked by not doing off() during the event#40

Merged
ZakarFin merged 1 commit intooskariorg:developfrom
ZakarFin:fix-mapclicked
Mar 3, 2025
Merged

Fix map clicked by not doing off() during the event#40
ZakarFin merged 1 commit intooskariorg:developfrom
ZakarFin:fix-mapclicked

Conversation

@ZakarFin
Copy link
Copy Markdown
Member

@ZakarFin ZakarFin commented Mar 1, 2025

Looks like unlistening during event notifications stop propagating the event to other listeners. Fixes the issue where for example marker tool or find by coordinates didn't receive the first map click after the event.

@ZakarFin
Copy link
Copy Markdown
Member Author

ZakarFin commented Mar 1, 2025

Might be worth it to fix the core issue in sandbox instead

@ZakarFin
Copy link
Copy Markdown
Member Author

ZakarFin commented Mar 3, 2025

oskariorg/oskari-frontend#2801 Fixes the core issue.

@ZakarFin ZakarFin added this to the 1.29.0 milestone Mar 3, 2025
@ZakarFin ZakarFin merged commit 89d640b into oskariorg:develop Mar 3, 2025
@ZakarFin ZakarFin deleted the fix-mapclicked branch March 3, 2025 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant