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

Дополняет и уточняет доку aria-required #5562

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

juliememe
Copy link

@juliememe juliememe commented Nov 24, 2024

Описание

Closes #3036.

https://content-5562.dev.doka.guide/a11y/aria-required/

Чек-лист

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

@github-actions github-actions bot added дока Справочный материал placeholder Заготовка материала a11y Контент про Доступность labels Nov 24, 2024
@juliememe juliememe changed the title update index.md Дополняет и уточняет доку aria-required Nov 24, 2024
@TatianaFokina TatianaFokina changed the title Дополняет и уточняет доку aria-required Дополняет и уточняет доку aria-required Nov 24, 2024
@TatianaFokina
Copy link
Member

@juliememe, спасибо за доку! Я ещё раз гляну на неё чуть позже.

@TatianaFokina TatianaFokina removed the placeholder Заготовка материала label Nov 24, 2024
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.

Фууух, добралась! Извини, что так долго шла до твоего пиара.

a11y/aria-required/index.md Outdated Show resolved Hide resolved
a11y/aria-required/index.md Outdated Show resolved Hide resolved
a11y/aria-required/index.md Outdated Show resolved Hide resolved
a11y/aria-required/index.md Outdated Show resolved Hide resolved
a11y/aria-required/index.md Outdated Show resolved Hide resolved
a11y/aria-required/index.md Outdated Show resolved Hide resolved
a11y/aria-required/index.md Outdated Show resolved Hide resolved
a11y/aria-required/index.md Outdated Show resolved Hide resolved
a11y/aria-required/index.md Outdated Show resolved Hide resolved
a11y/aria-required/index.md Outdated Show resolved Hide resolved
Copy link
Member

@skorobaeus skorobaeus left a comment

Choose a reason for hiding this comment

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

За демку аппрув, а финальную версию текста я ещё чекну :)

@TatianaFokina
Copy link
Member

@juliememe, привет! Нужна моя помощь с этой докой?

@TatianaFokina
Copy link
Member

@nasty23-star, @skorobaeus, чекните текст?

Copy link

github-actions bot commented Jan 4, 2025

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

@TatianaFokina
Copy link
Member

@skorobaeus и @solarrust, забежите в этот пиар?

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

Successfully merging this pull request may close these issues.

aria-required
4 participants