OpenFest / initfest

OpenFest theme, built on top of Krasimir Tsonev's html+css work
GNU General Public License v2.0
0 stars 10 forks source link

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

Closed dinamic closed 6 years ago

dinamic commented 6 years ago

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

Changes:

bozhodimitrov commented 6 years ago

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

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

dinamic commented 6 years ago

Промяната включва 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 commented 6 years ago

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

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

bozhodimitrov commented 6 years ago

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

krokodilerian commented 6 years ago

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

dinamic commented 6 years ago

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

krokodilerian commented 6 years ago

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

krokodilerian commented 6 years ago

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

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