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

Добавляет рецепт плавного раскрытия выпадающего меню #4998

Merged
merged 8 commits into from
Dec 28, 2023

Conversation

solarrust
Copy link
Member

@solarrust solarrust commented Nov 23, 2023

Описание

Рецепт, в котором раскрытие выпадающего меню анимируется на чистом CSS.

@TatianaFokina если захочешь намазать сверху доступности, то я не против =)
В целом, пожалуйста, попридирайтесь максимально.

Демо: https://content-4998.dev.doka.guide/recipes/dropdown-animation/

Чек-лист

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

@solarrust solarrust added статья Расширенный материал рецепт Контент для Рецептов labels Nov 23, 2023
@solarrust solarrust self-assigned this Nov 23, 2023

This comment was marked as outdated.

@github-actions github-actions bot added the дока Справочный материал label Nov 23, 2023
@solarrust solarrust marked this pull request as ready for review November 23, 2023 16:47
@TatianaFokina
Copy link
Member

Ок, я посмотрю завтра.

recipes/dropdown-animation/index.md Outdated Show resolved Hide resolved
recipes/dropdown-animation/index.md Outdated Show resolved Hide resolved
recipes/dropdown-animation/index.md Outdated Show resolved Hide resolved
recipes/dropdown-animation/index.md Outdated Show resolved Hide resolved
recipes/dropdown-animation/index.md Outdated Show resolved Hide resolved
recipes/dropdown-animation/index.md Outdated Show resolved Hide resolved
recipes/dropdown-animation/index.md Outdated Show resolved Hide resolved
recipes/dropdown-animation/index.md Outdated Show resolved Hide resolved
recipes/dropdown-animation/index.md Outdated Show resolved Hide resolved
recipes/dropdown-animation/index.md Show resolved Hide resolved
@TatianaFokina
Copy link
Member

TatianaFokina commented Nov 24, 2023

Так, я поняла, что так-то контент от скринридеров не скрыт. И с клавы можно сделать фокус на невидимом пункте. Предлагаю навесить aria-hidden="true" и tabindex="-1" на выпадающий список в закрытом состоянии. Когда пользователь кликает на кнопку, атрибуты можно либо удалять вообще, либо менять их значения false и 0, соответсвенно. Как больше нравится. Ты подправишь демки или мне сделать?

@skorobaeus
Copy link
Member

Не возражаешь, если растянем по ширине маленько? У демок горизонтальный вайб, так сказать 😅

.menu-link {
    padding: 0.5lh 80px;
}

image

Co-authored-by: Tatiana Fokina <fokinatatian@gmail.com>
Copy link
Member Author

@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-animation/index.md Outdated Show resolved Hide resolved
recipes/dropdown-animation/index.md Outdated Show resolved Hide resolved
@solarrust
Copy link
Member Author

@TatianaFokina добавила раздел про скрытие меню для скринридеров и клавиатуры в закрытом состоянии.
@skorobaeus поправила стили, как ты предложила.

@HellSquirrel очень жду твоего ревью нашего совместного творчества =)

Copy link

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

@TatianaFokina TatianaFokina removed the дока Справочный материал label Dec 12, 2023
@TatianaFokina
Copy link
Member

А может помёрджим, а Полина, если что, потом придёт и посмотрит?

@solarrust
Copy link
Member Author

А давай

@solarrust solarrust merged commit ac56272 into main Dec 28, 2023
6 of 7 checks passed
@solarrust solarrust deleted the recepie/dropdown branch December 28, 2023 20:24
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.

3 participants