search

volga-volga / react-native-yamap

React Native Yandex Maps | Яндекс Карты | Yandex.MapKit implementation for react native | YandexMaps
151 stars 84 forks source link

Новый prop для маркера и улучшения на IOS #105

Closed ch3rn1k closed 2 years ago

ch3rn1k commented 2 years ago

Добавлен новый проп visible для компонента Marker Уменьшено время задержи перерисовки маркеров на IOS (улучшения согласно документации)

ownikss commented 2 years ago

По изменениям все ок. Единственное - насчет задержки на ios - сейчас там стоит комментарий

    // todo: Если вызывать сразу то frame имеет размеры 0. В идеале делать подписку на событие

При использовании dispatch_get_main_queue проблем не с children возникает? Возможно проблема вообще была от старых версий

ownikss commented 2 years ago

Если проблем нет - завтра готов принять реквест и выпустить версию. Спасибо за помощь в развитии библиотеки

ch3rn1k commented 2 years ago

По изменениям все ок. Единственное - насчет задержки на ios - сейчас там стоит комментарий

    // todo: Если вызывать сразу то frame имеет размеры 0. В идеале делать подписку на событие

При использовании dispatch_get_main_queue проблем не с children возникает? Возможно проблема вообще была от старых версий

Если полностью убрать задержки - вылет есть. Но я потестил с обычными маркерами и маркерами с child - без вылетов.

ch3rn1k commented 2 years ago

Переделаю реквест с учетом исправления #102