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

Autocomplete on angularjs #1

Closed
enginer opened this issue Jul 8, 2014 · 4 comments
Closed

Autocomplete on angularjs #1

enginer opened this issue Jul 8, 2014 · 4 comments
Labels

Comments

@enginer
Copy link
Contributor

enginer commented Jul 8, 2014

Нужно переделать аутокомплит чтобы не зависеть от библиотеки кендо (она достаточно объёмная, не хочется всю тянуть только ради аутокомплита)

Первый вариант аутокомплита можно увидеть по ссылке http://ii.ayfaar.org/new/#/home, но он далёк от совершенства. За эталон предлагаю взять аутокомплит на этой странице http://ii.ayfaar.org/#main

Требования:

  • если человек выбрал термин из списка предложенных, то переходим сразу на описание этого термина сейчас (это реализовано не верно - если нажать стрелочку вниз на клавиатуре, то происходит автоматический переход на первое значение)
  • если человек проигнорировал предложения и просто нажал ентер или поиск - переходим на поиск по введённому слову или фразе
  • Аутокомплит должен срабатывать после ввода более 2х символов.

Сам список желательно стилизовать (длина, подстветка выделения) по аналогии с http://ii.ayfaar.org

Изменения производить в ветке issue1 https://github.com/devstarter/ii/tree/issue1/src/main/webapp/new

@enginer enginer added this to the Переезд на angularjs milestone Jul 8, 2014
@enginer enginer added UI and removed enhancement labels Jul 10, 2014
@maks1m
Copy link

maks1m commented Jul 11, 2014

А не лучше чтоб автоподстановка начинала срабатывать где то с третьего символа?
А то по одной букве можна половину слов вытянуть.
И можете еще учитывать релевантность поиска?
Что чаще ищут то первым подставлять, ну пока введено только несколько букв?

@enginer
Copy link
Contributor Author

enginer commented Jul 11, 2014

Да! Имеено так сейчас и работает на http://ii.ayfaar.org, аутокомплит срабатывает после ввода более 3х символов. Нужно тоже самое, но на ангуларе.

@hroot
Copy link

hroot commented Jul 14, 2014

Все что ниже выношу на обсуждение:

  1. Реакция на 2 символа должна быть т.к. часть слов в русском языке двухбуквенная. Как минимум должно срабатывать реакция на ENTER или на кнопку поиска.
  2. Приведенный пример запроса использует GET который не является кросс доменным. В случае переезда АПИ на другой сервер вся структура перестанет работать.

@enginer
Copy link
Contributor Author

enginer commented Jul 14, 2014

Благодарю за хорошие вопросы!

  1. Согласен со всем, кнопку поиска можно как у гугла сделать.
  2. Эту проблему я исправил на сервере путём добавления Access-Control-Allow-Origin хедера в респонс. Ведь пример http://plnkr.co/edit/MCSwIK?p=preview какрас работает из другого домена. Так что можно не беспокоиться о перезде.

enginer added a commit that referenced this issue Jul 14, 2014
@hroot hroot added bug and removed bug labels Jul 17, 2014
@enginer enginer removed the bug label Jul 18, 2014
enginer pushed a commit that referenced this issue Jul 25, 2014
@Ekvi Ekvi modified the milestones: Переезд на angularjs, SEO, Голосование за найденные цитаты Aug 11, 2014
@enginer enginer removed the training label Sep 16, 2014
@enginer enginer modified the milestone: Голосование за найденные цитаты Oct 20, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants