Search "show on map" highlight ignoring some POIs #2070

Closed
opened 2022-02-10 08:02:58 +00:00 by arnaudvergnet · 4 comments
arnaudvergnet commented 2022-02-10 08:02:58 +00:00 (Migrated from github.com)

When there is a high density of search results, it seems like OM is unable to correctly highlight everything.

As you can see in the following recordings, some restaurants and bars are shown on the map but not highlighted.

https://user-images.githubusercontent.com/80701113/153360688-3684cde1-04df-4ce4-b31f-53aae626f9af.mp4

https://user-images.githubusercontent.com/80701113/153363288-dedf3f6a-6a04-46b6-8a9d-2cfc52b7709c.mp4

I don't know the implementation details, but it seems like search results highlights are simply shown as an overlay on the map. As they are bigger, they do not use the same density checks so it can cause issues in high density areas.

One way to fix this would be to change the actual POI style instead of adding an icon overlay, with new density rules as the icon is bigger and display priority.

When there is a high density of search results, it seems like OM is unable to correctly highlight everything. As you can see in the following recordings, some restaurants and bars are shown on the map but not highlighted. https://user-images.githubusercontent.com/80701113/153360688-3684cde1-04df-4ce4-b31f-53aae626f9af.mp4 https://user-images.githubusercontent.com/80701113/153363288-dedf3f6a-6a04-46b6-8a9d-2cfc52b7709c.mp4 I don't know the implementation details, but it seems like search results highlights are simply shown as an overlay on the map. As they are bigger, they do not use the same density checks so it can cause issues in high density areas. One way to fix this would be to change the actual POI style instead of adding an icon overlay, with new density rules as the icon is bigger and display priority.
biodranik commented 2022-02-10 10:20:03 +00:00 (Migrated from github.com)

Also see #1003, #1147 and #1560

Also see #1003, #1147 and #1560
vng commented 2022-06-12 12:27:24 +00:00 (Migrated from github.com)

mentioned in issue #2717

mentioned in issue #2717
vng commented 2022-07-29 08:42:18 +00:00 (Migrated from github.com)

assigned to @vng

assigned to `@vng`
vng commented 2022-07-29 08:43:06 +00:00 (Migrated from github.com)

mentioned in merge request !3035

mentioned in merge request !3035
vng (Migrated from github.com) closed this issue 2022-08-03 14:38:31 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: organicmaps/organicmaps#2070
No description provided.