Skip to content

feat(select): select search cosmetic and behavior fixes #136

Merged
merged 11 commits into from
Jun 11, 2019
Merged

feat(select): select search cosmetic and behavior fixes #136

merged 11 commits into from
Jun 11, 2019

Conversation

roll314
Copy link
Contributor

@roll314 roll314 commented Jun 5, 2019

@fost
@lskramarov
@mikeozornin

TFS 185145

Паста от Мишы

Собрал разное про поиск в селекте. Если что-то не про поиск, а про пример, то можно обсудить тоже.

Если нажать Esc в поле поиска, то закроется дропдаун, хотя хочется всего лишь сбросить поиск.

http://d.mikeozornin.ru/zcWIxL
Если ничего не найдено, то так ↑. А надо как-то так: http://d.mikeozornin.ru/DCAebd

http://d.mikeozornin.ru/5JBXYd
До выбранного элемента не всегда проматывается.
Первый раз (Moscow) ок, второй раз (Благовещенск) нет.

http://d.mikeozornin.ru/lZgq46
Если дропдаун свернут, то при наличии поиск внутри навигация ↓↑ ходит с учетом его. Если поиск скрыт, то он очищен и влиять не должен.
В скринкасте сначала ходит норм, потом поиск поиска начинает ходить по профильтрованным элементам.

http://d.mikeozornin.ru/KPm9bP
Теги немного ниже, чем нужно.

Давай ещё пересортируем опции по алфавиту (именно по англ)?

@pimenovoleg pimenovoleg requested a review from lskramarov June 5, 2019 13:41
@pimenovoleg
Copy link
Member

@roll314 добавь пжл детали по PR - название и описание

@roll314 roll314 changed the title bug(select): search fixes Select search cosmetic and behavior fixes Jun 5, 2019
@roll314 roll314 changed the title Select search cosmetic and behavior fixes feat(select): select search cosmetic and behavior fixes Jun 5, 2019
@mikeozornin
Copy link
Contributor

mikeozornin commented Jun 5, 2019

Если нажать Esc в поле поиска, то закроется дропдаун, хотя хочется всего лишь сбросить поиск.
А нельзя сделать так, чтобы первый Esc сбрасывал поиск, а второй дропдаун?

http://d.mikeozornin.ru/X4NtoR
Ничего не найдено появилось, нужно чуток внешний вид докрутить (http://d.mikeozornin.ru/DCAebd)
Высота такая же как у обычного элемента, отступы такие же, просто не ховерится и цвет бледный.

Если мы можем пересортировать опции в примере, то давай сделаем? Вроде редакторы умеют сортировать строки сейчас.

Остальное стало хорошо.

@pimenovoleg pimenovoleg requested a review from mikeozornin June 6, 2019 08:08
@roll314
Copy link
Contributor Author

roll314 commented Jun 6, 2019

Обновил работу esc, отсортировал опции в примере по английски (были по русски), поправил сообщение

@roll314
Copy link
Contributor Author

roll314 commented Jun 6, 2019

добавил тесты для esc

Copy link
Contributor

@mikeozornin mikeozornin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Норм

Upd. заапрувил не тот PR, сейчас посмотрю этот.

@mikeozornin mikeozornin self-requested a review June 6, 2019 20:13
Copy link
Contributor

@mikeozornin mikeozornin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Здесь тоже все хорошо

packages/mosaic/select/select.component.ts Outdated Show resolved Hide resolved
packages/mosaic/select/select.component.ts Outdated Show resolved Hide resolved
packages/mosaic/select/select.component.ts Outdated Show resolved Hide resolved
packages/mosaic/select/select.component.ts Outdated Show resolved Hide resolved
@pimenovoleg pimenovoleg merged commit a25a60b into positive-js:master Jun 11, 2019
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.

4 participants