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

Changed static pages to use OSM instead of Google maps #33

Closed
wants to merge 1 commit into from
Closed

Changed static pages to use OSM instead of Google maps #33

wants to merge 1 commit into from

Conversation

dinamic
Copy link

@dinamic dinamic commented Nov 5, 2018

Изглежда като че тези файлове да не се използват вече, но.. fixes #25

Changes:

  • use OSM instead of GMaps
  • minor refactoring; removed trailing spaces

@bozhodimitrov
Copy link

bozhodimitrov commented Nov 5, 2018

Промяната включва OSM embed линк, който не е точен.
А самия OSM все още е доста назад от към информация спрямо Google Maps.
За пример - улицата на лаба дори я нямаше на картата, а на нейно място имаше друга улица без име.
Самия init Lab също не беше добавен като POI.
Добавих малко промени в OSM, за да може локацията на лаба да се намира коректно при търсене/навигиране - OSM changes - add init Lab

Според мен е по-добре да има и двете карти. или поне да има статичен линк към Google Maps (ако не искаме iframe-ове от Google).

@dinamic
Copy link
Author

dinamic commented Nov 6, 2018

Промяната включва OSM embed линк, който не е точен.

Да, включва OSM embed линк, вместо Google maps javascript код. Замененият код сочеше към Интерпред, затова и новият сочи насам.

А самия OSM все още е доста назад от към информация спрямо Google Maps.
За пример - улицата на лаба дори я нямаше на картата, а на нейно място имаше друга улица без име.

Затова е OSM, защото можем да го направим по-добър. Ако нещо липсва - нищо не пречи да бъде добавено, нали? :)

Добавих малко промени в OSM, за да може локацията на лаба да се намира коректно при търсене/навигиране - OSM changes - add init Lab

Браво! Хората търсещи init Lab в OSM ще са ти благодарни за труда! 👍

Според мен е по-добре да има и двете карти. или поне да има статичен линк към Google Maps (ако не искаме iframe-ове от Google).

Може би ни трябва дизайнер или frontend програмист които да го направи да изглежда приятно. Как си го представяш - с табове или имаш нещо друго предвид?

@bozhodimitrov
Copy link

@dinamic по-скоро OSM-а да е по подразбиране, а за хората, които все пак са свикнали с Google Maps - да имат опцията със статичен линк, за да го отворят в нов прозорец (target blank). Не е нужно да викаме дизайнер за подобно нещо.

Относно местоположението - OpenFest отдавна не е в сградата на Интерпред. Новото място (за сега) е сградата на форум "Джон Атанасов" (OSM link)

@bozhodimitrov
Copy link

Btw, ако не греша - тази тема май вече не се ползва или по-скоро промените от prod, не са добавени в това хранилище. На сайта никъде не виждам Google Maps-a.

@krokodilerian
Copy link
Contributor

Ще налея последните промени от живия сайт (въпреки че би трябвало да са тук), и ще видя как ще се получи. Ще имам време тия дни :)

@dinamic
Copy link
Author

dinamic commented Nov 6, 2018

Да, също така OpenFest вече не е в Интерпред. :)

@krokodilerian
Copy link
Contributor

Sry, че толкова се бавя, днес ще го погледна и вероятно ще го вкарам

@krokodilerian
Copy link
Contributor

Добре де, сега имах време да видя pull request-а, и за съжаление не е смислен - тези файлове (в html/ ) са примерни как да ни изглежда темата, като цяло нямат никакво участие в самия изглед на сайта. OSM-то трябва да се вкара в wordpress-а, или някъде в transportations template-а.

Остава като интересно нещо, което да се свърши, но ще затворя pull request-а, ако имаш време виж къде точно трябва да стане и отвори нов. Ако цяло ти се занимава със сайта, може да измислим как да ти вдигна един devlopement setup с реални данни.

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.

Картата за събитието да е OpenStreetMaps
3 participants