Skip to content

[WIP] feat(navbar): Dropdown item #34

Merged
merged 22 commits into from
Aug 15, 2018

Conversation

kuflash
Copy link
Contributor

@kuflash kuflash commented Aug 10, 2018

No description provided.

@pimenovoleg
Copy link
Member

Congratulations on your first PR!!!

@mikeozornin
Copy link
Contributor

http://d.mikeozornin.ru/s095qI
Сейчас при хувере мышкой по выпавшим пунктам показывается состояние фокус, а не хувер.
Хувер можно посмотреть в листе, у него бледно-серый фон, хувер в дропдауне навбара такой же.
Вот правильный: http://d.mikeozornin.ru/Hi9QkH

http://d.mikeozornin.ru/x1Gwqb
Фокуса при этом нет, вместо него показывается стандартный браузерный аутлайн.

http://d.mikeozornin.ru/dRE1ch
Не очень понял что здесь. Зачем-то внутрь дропдауна вставлены кнопки с дропдауном. Что это?

Клавиатура
Здесь есть ошибки в работе с клавиатурой, но возможно их надо починить сделав навбар клавиатура-френдли. @fost, может сразу сделает навигацию с клавиатуры как обсуждали? Фокус на меню целиком, ←→ для навигации по меню и ↑↓ по выпадающим? Или это в рамках другого бранча?

Вот что сейчас не ок в работе клавиатуры:
Если пункт меню свернут, то по нему можно ходить табуляцией, как будто он есть.
Воспроизведение:

  1. Дойти табуляцией до Dropdown item 1
  2. Нажать Tab
    Ожидается: фокус перемещается на следующий элемент (Dropdown item)
    В реальноси: фокус перемещается на следующий элемент через 4 нажатия Tab

При фокусе на элементе с дропдауном он должен выпадать при нажатии на пробел, как это делает кнопка с дроплауном.
Элемент без дропдауна в меню должен выбираться по нажатии на пробел.

@mikeozornin
Copy link
Contributor

Олег сказал, что хоткеи потом и отдельно. Остальное все хорошо.

Не понимаю, кому нужно будет вставлять кнопки в меню, но пускай.

@pimenovoleg
Copy link
Member

@mikeozornin уберем кнопки, надо придумать пример с custom template.

@pimenovoleg pimenovoleg merged commit f8f40a2 into positive-js:master Aug 15, 2018
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.

3 participants