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? #9

Closed
igoradamenko opened this issue Jun 10, 2016 · 3 comments
Closed

menu? #9

igoradamenko opened this issue Jun 10, 2016 · 3 comments

Comments

@igoradamenko
Copy link
Contributor

igoradamenko commented Jun 10, 2016

Могу ошибаться, но всё же тег menu не используется в том смысле, в каком он описан в последнем примере. В спеке примеры описывают построение интерфейсных меню.

Думаю, в примере должен быть div. PR не создаю, т. к. может быть чего-то не знаю :)

P. S.: Вчитался в текст примеров. Он прекрасен!

@jt3k
Copy link
Contributor

jt3k commented Jun 10, 2016

Я смотрю здесь http://html5doctor.com/element-index/#menu , и кажыцо всо ок.
Надо читать "if the element has no li element children, flow content describing available commands."

@igoradamenko
Copy link
Contributor Author

Во-первых, menu в спеке находится в разделе интерактивных элементов, что, наверное, намекает на то, что элемент будет как-то обрабатываться браузером. Рядом с ним там details/summary, так что можно примерно прикинуть, как.

Во-вторых, menu определяет список «команд». Команда — это абстракция, которая меняет состояние. У неё может быть label, access key и пр.

В-третьих, чаще всего у этого тега есть атрибут type, возможные значения которого: context и popup, что тоже не совсем похоже на то, что в примере.

В-четвёртых, есть довольно-таки популярный вопрос на СО, где англоязычные ребята трактуют написанное в спеке. В целом они сходятся к тому, что toolbar и context — это не просто так, и в общем-то этот тег для веб-приложений, а не веб-страниц. Тут конечно можно развести дискуссию о том, что такое веб-приложение, но предлагаю считать в этом контексте веб-приложением то, у чего есть контекстное меню. Сейчас на моей памяти ничего, кроме Slack мне в голову не приходит. Т. е. реально приложение.

Даже если мы не сойдёмся в этой дискуссии на том, для чего же нужен этот тег, считаю, что из примера его стоит убрать, т. к. только путает. Примеры для новичков, а потому если они заметят тег menu в подобном контексте то будут пихать в него меню сайтов и пр., для чего явно есть другие теги (тот же nav, например).

@yoksel
Copy link
Owner

yoksel commented Jun 11, 2016

@igoradamenko вы правы, я слишком вольно распрядилась тегом. Поправила примеры.

@yoksel yoksel closed this as completed Jun 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants