Site
/
initfest
11
0
Fork 0
Code Issues 4 Pull Requests Packages Projects Releases Wiki Activity

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

Closed
dinamic wants to merge 1 commits from master into master
pull from: master
merge into: Site:master
Conversation 8 Commits 1 Files Changed 4 +30 -30
dinamic commented 2018-11-05 10:46:46 +02:00 (Migrated from github.com)
Copy Link

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

Changes:

  • use OSM instead of GMaps
  • minor refactoring; removed trailing spaces
Изглежда като че тези файлове да не се използват вече, но.. fixes #25 Changes: - use OSM instead of GMaps - minor refactoring; removed trailing spaces
bozhodimitrov commented 2018-11-05 14:32:46 +02:00 (Migrated from github.com)
Copy Link

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

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

Промяната включва OSM embed линк, който не е точен. А самия OSM все още е доста назад от към информация спрямо Google Maps. За пример - улицата на лаба дори я нямаше на картата, а на нейно място имаше друга улица без име. Самия init Lab също не беше добавен като POI. Добавих малко промени в OSM, за да може локацията на лаба да се намира коректно при търсене/навигиране - [OSM changes - add init Lab](https://www.openstreetmap.org/changeset/64195065#map=18/42.70681/23.32473) Според мен е по-добре да има и двете карти. или поне да има статичен линк към Google Maps (ако не искаме iframe-ове от Google).
dinamic commented 2018-11-06 13:39:18 +02:00 (Migrated from github.com)
Copy Link

Промяната включва 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 програмист които да го направи да изглежда приятно. Как си го представяш - с табове или имаш нещо друго предвид?

> Промяната включва 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 2018-11-06 14:40:15 +02:00 (Migrated from github.com)
Copy Link

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

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

@dinamic по-скоро OSM-а да е по подразбиране, а за хората, които все пак са свикнали с Google Maps - да имат опцията със статичен линк, за да го отворят в нов прозорец (target blank). Не е нужно да викаме дизайнер за подобно нещо. Относно местоположението - OpenFest отдавна не е в сградата на Интерпред. Новото място (за сега) е [сградата на форум "Джон Атанасов" (OSM link)](https://www.openstreetmap.org/#map=19/42.66615/23.37551)
bozhodimitrov commented 2018-11-06 14:53:49 +02:00 (Migrated from github.com)
Copy Link

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

Btw, ако не греша - тази тема май вече не се ползва или по-скоро промените от prod, не са добавени в това хранилище. На сайта никъде не виждам Google Maps-a.
👍 1
krokodilerian commented 2018-11-06 15:07:56 +02:00 (Migrated from github.com)
Copy Link

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

Ще налея последните промени от живия сайт (въпреки че би трябвало да са тук), и ще видя как ще се получи. Ще имам време тия дни :)
👍 1
dinamic commented 2018-11-06 15:16:28 +02:00 (Migrated from github.com)
Copy Link

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

Да, също така OpenFest вече не е в Интерпред. :)
krokodilerian commented 2018-11-20 10:58:40 +02:00 (Migrated from github.com)
Copy Link

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

Sry, че толкова се бавя, днес ще го погледна и вероятно ще го вкарам
krokodilerian commented 2018-11-22 17:58:00 +02:00 (Migrated from github.com)
Copy Link

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

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

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

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

   
duplicate

   
enhancement

   
help wanted

   
invalid

   
question

   
wontfix
No Label
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
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: Site/initfest#33
No description provided.
Delete Branch "master"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?