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

Coord search #263

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

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