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

menu: проверить соответствие руководству по доступности #1458

Closed
aristov opened this issue Apr 13, 2015 · 9 comments

Comments

@aristov
Copy link
Contributor

aristov commented Apr 13, 2015

Проверить, соответствует ли блок menu описанию из руководства по доступности:

Механика меню
Фокус устанавливается только на кнопку, открывающую меню — фокус всегда установлен на этой кнопке. Элементы меню должны быть исключены из навигации, с использованием клавиши Tab.
Выпадающее меню открывается только при активации кнопки, закрывается — при нажатии Escape.
Первый элемент меню становится текущим после открытия. Текущий элемент выделяется визуально. Стрелка вниз делает текущим следующий элемент, стрелка вверх — предыдущий. Если следующего нет — текущим становится первый, если нет предыдущего — последний.
Меню может содержать вложенное меню. Вложенное меню открывается нажатием стрелки вправо и закрывается нажатием стрелки влево.
Текущий элемент меню активируется нажатием Spacebar или Enter. После активации текущего элемента меню закрывается.

Внести исправления, если найдутся несоответствия.

@Guria
Copy link
Contributor

Guria commented Apr 15, 2015

Блок menu кажется совсем не про меню. Скорее всего это некий list.
То что описано в руководстве, в библиотеке не реализовано.
Возможно стоит рассмотреть возможность переименования, по аналогии с #1447

@tadatuta
Copy link
Member

@Guria
не возьмусь судить про соответствие названию menu, но list — это скорее про контент/типографику типа:

  • раз
  • два
  • три

@Guria
Copy link
Contributor

Guria commented Apr 15, 2015

Может не list, но точно не меню. Это же список итемов.
А вообще похожие по смыслу сущности, правда более навороченные, чаще всего называются по типу ListView.
https://msdn.microsoft.com/en-us/library/windows/apps/br211837.aspx
http://demos.telerik.com/kendo-ui/mobile-listview/index

@qfox
Copy link
Member

qfox commented Apr 15, 2015

@Guria ;-) В Яндексе своя терминология, не спорь! Можно, правда, поставить вопрос при переезде на v3, но это опять всем всё сломает. С другой стороны, порог входа от этого только уменьшится. Можно пообсуждать.

@Guria
Copy link
Contributor

Guria commented Apr 15, 2015

Ну в руководстве по доступности под меню явно имеют ввиду именно меню :)
На фоне изменений i-bem в bem-core@v3 переименование пары блоков будет наименьшей проблемой.

@qfox
Copy link
Member

qfox commented Apr 15, 2015

@Guria Согласен, если возвращаться к issue — путаница.

@aristov
Copy link
Contributor Author

aristov commented Apr 16, 2015

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

Элементы меню должны быть исключены из навигации, с использованием клавиши Tab.

Текущий элемент выделяется визуально. Стрелка вниз делает текущим следующий элемент, стрелка вверх — предыдущий. Если следующего нет — текущим становится первый, если нет предыдущего — последний.

Текущий элемент меню активируется нажатием Spacebar или Enter.

Вложенные меню у нас не поддерживаются.

@aristov
Copy link
Contributor Author

aristov commented Apr 16, 2015

issue про select

@aristov aristov removed the v3 label Apr 29, 2015
@veged veged changed the title menu: механика меню menu: проверить соответствие руководству по доступности Apr 29, 2015
@deeonis deeonis added the ready label Apr 29, 2015
@tadatuta tadatuta mentioned this issue Jun 4, 2015
20 tasks
@aristov
Copy link
Contributor Author

aristov commented Sep 16, 2015

Проверено. Меню работает правильно, исправления не требуются. Закрываю.

@aristov aristov closed this as completed Sep 16, 2015
@aristov aristov removed the ready label Sep 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants