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

Странно работает список поисковых запросов #25

Closed
Krulk opened this issue Aug 8, 2015 · 3 comments
Closed

Comments

@Krulk
Copy link

@Krulk Krulk commented Aug 8, 2015

Когда ищешь, поисковые запросы сохраняются в выпадающем меню. Сейчас оно работает неправильно (Как до конца не понял):

  1. Поисковые запросы сохраняется в нём сверху, но если закрыть окно а потом открыть его, последние запросы каждый раз оказываются в конце списка.
  2. Старые поисковые запросы пока список растёт перемещаются вверх и вообще не вытесняются когда список полон, если в 1 окне поиска не начать искать 10 раз.
  3. Список сортируется рандомно.

По идее оно должно работать так:

  1. Новое слово добавлять вверх, список растёт, удаляется самое нижнее слово, если список полон.
  2. Если слово уже есть в списке, переместить его вверх.
  3. Список запросов сохраняется после каждого поиска, чтобы если открыть новое окно поиска или закрыть старое и открыть новое, список был всегда актуальным.
@pavel-pimenov
Copy link
Owner

@pavel-pimenov pavel-pimenov commented Aug 19, 2015

Принятно - думаю там сортировка стоит по алфавиту.
и при открытии комбик все записи сам тасует

Loading

@pavel-pimenov
Copy link
Owner

@pavel-pimenov pavel-pimenov commented Aug 19, 2015

Исправлено - ожидай след бетку

Loading

@Krulk
Copy link
Author

@Krulk Krulk commented Aug 21, 2015

Не исправлено. Чтобы повторить нужно очистить поиск, искать последовательно: 01, 02, 03, 04, 05, 06, 07, 08, 09, 10. Закрыть поиск. Открыть поиск и увидеть, что запросы в списке перепутаны: http://f5.s.qip.ru/GjeazMDL.png
03 постоянно перескакивает во 2 строчку. Это стабильный пример на цифрах, с буквами всё ещё более непредсказуемо.

Loading

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants