yandex-maps-unofficial / vue-yandex-map

Yandex Maps Component for VueJS
MIT License
359 stars 103 forks source link

loadYmap срабатывает рано в в двух разный компонентах #283

Closed Pinegin closed 3 years ago

Pinegin commented 3 years ago

Привет, классный плагин не нарадуюсь. Спасибо.

Сейчас начал пользоваться функцией loadYmap. Однако заметил одну проблему. Есть два компонента корорым нужен экземпляр ymaps. В обоих идёт подключение в моунт loadYmap. И один из компонентов рано начинает работу, до появления ymaps.

https://github.com/PNKBizz/vue-yandex-map/blob/410f3e62e7a1a3b5cbd6475eb3d86152dac657db/src/utils.js#L200

так как первый компонент уже установил script.

возможно что то подобное:

if (window.ymaps) return res();

if (document.getElementById('vue-yandex-maps')) {
   emitter.$on('scriptIsLoaded', res);
   return
}
PNKBizz commented 3 years ago

Привет! Спасибо за отзыв и за issue. Справедливое замечание. Если будет от вас PR - с удовольствием вмержу, либо на след неделе сам добавлю.

Pinegin commented 3 years ago

Нет проблем!

https://github.com/PNKBizz/vue-yandex-map/pull/284

PNKBizz commented 3 years ago

Супер! )

PNKBizz commented 3 years ago

v0.10.10