Skip to content

Comments

Coord search#263

Open
progserega wants to merge 9 commits intoopenstreetmap-ru:masterfrom
progserega:coord_search
Open

Coord search#263
progserega wants to merge 9 commits intoopenstreetmap-ru:masterfrom
progserega:coord_search

Conversation

@progserega
Copy link
Contributor

@progserega progserega commented Aug 2, 2016

Поддержка поиска по координатам (десятичным и град/мин/сек).
Если в строке поиска ввести что либо вида:
43.15 132.01
или (разделители - символы одинарного и двойного апострова):
43°9′0″ 132°0′36″
или (разделители - символы одинарной и двойной кавычек):
43°9'0" 132°0'36"

или всё то же самое, но разделитель между lat и lon не пробел, а ';' или ':' или ',', окружённый или нет пробелами),

То в списке поиска появится переход на соответствующие координаты.
Порою это бывает нужно. Осуществлять переход по известным координатам, посредством правки строки адреса сайта - крайне неудобно. Более того, порою в интернете координаты даются именно в град/мин/сек - в результате нужно искать калькулятор по переводу координат в десятичные, потом править адрес сайта... Неудобно.
Данный патч решает эту проблему.

@ErshKUS
Copy link
Collaborator

ErshKUS commented Aug 29, 2016

закончил? :)

@progserega
Copy link
Contributor Author

да :-)

@ErshKUS
Copy link
Collaborator

ErshKUS commented Aug 30, 2016

Что то у тебя уж больно сложный код. Не проще ли было бы реализовать всё на JS и если найдены координаты, то не выполнять другой поиск? Нет желания переделать?

@progserega
Copy link
Contributor Author

Да, согласен. Делал просто по аналогии. А серверной части тут в принципе и не нужно - можно на стороне клиента всё сделать. Постараюсь переделать. :-)
П.С. Питон ближе... :-)

@ErshKUS
Copy link
Collaborator

ErshKUS commented Aug 31, 2016

кстати, если между координатами поставить штук 5 пробелов, то не работает

@progserega
Copy link
Contributor Author

Да. надо перерабоатывать код. :-(

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.

2 participants