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

Добавляет рецепт приготовления многоуровневого меню. #4992

Merged
merged 24 commits into from
Dec 7, 2023

Conversation

s-dudko
Copy link
Contributor

@s-dudko s-dudko commented Nov 18, 2023

Описание

Добавляет рецепт приготовления многоуровневого меню. Пример меню с двумя вложенными подменю. Пытается сделать меню доступным для вспомогательных технологий.

Closes #2737

https://content-4992.dev.doka.guide/recipes/dropdown-menu/

Чек-лист

  • Текст оформлен согласно руководству по стилю
  • Ссылки на внутренние материалы начинаются со слеша и заканчиваются слэшем либо якорем на заголовок (/css/color/, /tools/json/, /tools/gulp/#kak-ponyat)
  • Ссылки на картинки, видео и демки относительные (images/example.png, demos/example/, ../demos/example/)

@github-actions github-actions bot added веб-платформа Контент по Веб-платформе статья Расширенный материал рецепт Контент для Рецептов labels Nov 18, 2023
@s-dudko s-dudko changed the title [Draft] Feature/2737 Добавляет рецепт приготовления многоуровневого меню. Nov 18, 2023
Copy link
Member

@TatianaFokina TatianaFokina left a comment

Choose a reason for hiding this comment

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

Привет! Пока посмотрела демку и не стала ничего редачить пока, так как что-то пошло не так и в этом пиаре оказались лишние коммиты. Призываю @HellSquirrel на помощь с гитом.

Пока всё хорошо по структуре разметки, оставила только одно предложение-комментарий.

recipes/dropdown-menu/demos/menu/index.html Outdated Show resolved Hide resolved
@TatianaFokina TatianaFokina removed the веб-платформа Контент по Веб-платформе label Nov 27, 2023
@TatianaFokina TatianaFokina requested review from TatianaFokina and removed request for igsekor November 27, 2023 16:52
@TatianaFokina
Copy link
Member

TatianaFokina commented Nov 27, 2023

@s-dudko, я там поредачила немного и причесала демку. Давай подождём комментариев от @skorobaeus, а потом заменим стили на актуальные в самом тексте?

А display: none не срабатывал из-за приоритетности (то есть наследования). Я его пониже в стилях перенесла, чтобы не перезаписывалось display: flex.

Copy link
Member

@solarrust solarrust left a comment

Choose a reason for hiding this comment

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

Спасибо за рецепт! Подметила буквально пару моментов.

recipes/dropdown-menu/index.md Show resolved Hide resolved
recipes/dropdown-menu/index.md Outdated Show resolved Hide resolved
recipes/dropdown-menu/index.md Outdated Show resolved Hide resolved
recipes/dropdown-menu/index.md Outdated Show resolved Hide resolved
@s-dudko
Copy link
Contributor Author

s-dudko commented Nov 30, 2023

Спасибо за рецепт! Подметила буквально пару моментов.

@solarrust Привет, спасибо за замечания, исправил)

@skorobaeus
Copy link
Member

Привет! Я немного подтюнила демку. Не уверена, что все эти стили нужны в тексте статьи, но всё, что сочтёте нужным, можно забирать ✌

@TatianaFokina Спасибо тебе! Раньше у нас не было дизайна для дропдаун-меню, а благодаря тебе появился :)

Завтра утром принесу обновлённую схему (pattern.png) и аппрув :)

@TatianaFokina
Copy link
Member

Осталось только актуализировать в тексте CSS и можно мёрджить, ящитаю.

Copy link

github-actions bot commented Dec 1, 2023

Превью контента из 36f9ea9 опубликовано.

@TatianaFokina TatianaFokina merged commit 3cbeea5 into doka-guide:main Dec 7, 2023
6 of 7 checks passed
@TatianaFokina
Copy link
Member

Спасибо за чудесный рецепт!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
рецепт Контент для Рецептов статья Расширенный материал
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Выпадающее меню
4 participants