Skip to content
This repository has been archived by the owner on Sep 11, 2019. It is now read-only.

(wip) ComboBox: Implement component #97

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

(wip) ComboBox: Implement component #97

wants to merge 5 commits into from

Conversation

narqo
Copy link
Owner

@narqo narqo commented Aug 15, 2016

  • Menu: Scroll to first selected item on mount
  • Menu: Implement typeahead
  • Select: Refactoring

@pasaran
Copy link
Collaborator

pasaran commented Aug 31, 2016

Пока что замеченные проблемы:

  • Сейчас берется value item'а в качестве значения инпута, а нужно брать текст item'а.
  • Когда я выбрал что-то в селекте, закрыл его, еще раз открыл, то в инпуте должен быть текст в виде текста, а не плейсхолдер, как сейчас.
  • Если попап открывается вверх, то при наборе он "поджимается" к своей верхней границе, т.е. между попапом и селектом растет дырка.
  • При наборе иногда пропадает выделение, хотя в попапе есть подходящие item'ы. Покажу на примере.
  • Иногда выбирается не тот item. Например, сверху есть "[x] Любая" и из всех марок отфильтровалась ровно одна марка, но выбран при этом item с "Любая".

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants