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

Добавляет схемы в доки про tablist, tab, tabpanel #4997

Merged

Conversation

TatianaFokina
Copy link
Member

@TatianaFokina TatianaFokina commented Nov 23, 2023

Описание

Добавила одну и ту же схему с устройством вкладок для наглядности в тексты.

https://content-4997.dev.doka.guide/a11y/role-tablist/#kak-ponyat
https://content-4997.dev.doka.guide/a11y/role-tab/#kak-ponyat
https://content-4997.dev.doka.guide/a11y/role-tabpanel/#kak-ponyat

Чек-лист

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

@TatianaFokina TatianaFokina added the улучшение Доработка существующего label Nov 23, 2023
@TatianaFokina TatianaFokina self-assigned this Nov 23, 2023
@github-actions github-actions bot added дока Справочный материал a11y Контент про Доступность labels Nov 23, 2023
@solarrust
Copy link
Member

Строго имхо: из-за зелёной рамочки кажется, что стрелочки указывают на неё и весь этот блок целиком это и tab, и tablist, и tabpanel. Но тут @skorobaeus лучше разбирается

@TatianaFokina
Copy link
Member Author

TatianaFokina commented Nov 28, 2023

Строго имхо: из-за зелёной рамочки кажется, что стрелочки указывают на неё и весь этот блок целиком это и tab, и tablist, и tabpanel. Но тут @skorobaeus лучше разбирается

Это моя версия схемы до графов, которая тоже этим грешила. Было не понятно, что где что. Надо переделывать, наверное, так же. Убрать рамку как минимум.

@skorobaeus, добавила исправленный вариант схемы рядом со старым.

@skorobaeus
Copy link
Member

Всё справедливо!

@TatianaFokina ещё немножечко причесала обновлённый вариант и положила рядом :)

Я попробовала покрутить и повыделять рамочкой только части, например только tab или tablist, но понятнее от этого не становится)

@TatianaFokina
Copy link
Member Author

@solarrust, вот такая обновлённая схемка.

tabs

@solarrust
Copy link
Member

Простите, что я так докопалась к этой схеме 😅

Что вы думаете, если мы всё же добавим пару направляющих? (ниже очень грубый набросок)

CleanShot 2023-11-30 at 11 32 42@2x

@skorobaeus
Copy link
Member

Что вы думаете, если мы всё же добавим пару направляющих? (ниже очень грубый набросок)

CleanShot 2023-11-30 at 11 32 42@2x

Это как раз то, что я имела в виду под "попробовала покрутить и повыделять рамочкой только части, но понятнее от этого не становится".

Направляющие это дополнительные визуальные элементы (а в инфографике чем лаконичнее, тем лучше), которые вызывают вопросы. Например: почему у таблиста и табпанели есть направляющие, а у таба нет? Потому что таб маленький и про него всё понятно? А почему тогда про таблист и табпанель не понятно? А надо ли вообще это объяснять? :) Я не говорю, что есть пользователи, которые прям задают такие вопросы, но это те вопросы, которые задаю себе я :)

Если правда кажется, что указания стрелочками недостаточно, то я бы переделала бы тогда схему вообще, надо пробовать подсвечивать разными цветами, мб подписывать что-то прямо вместо рыбного текста.

@TatianaFokina
Copy link
Member Author

А как вам такой вариант?

tabs

@skorobaeus
Copy link
Member

Классная идея с увеличением таба!

Продолжаем крутить :)

image

Copy link

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

@TatianaFokina TatianaFokina merged commit afe8cf1 into doka-guide:main Dec 12, 2023
7 checks passed
@TatianaFokina TatianaFokina deleted the a11y/add-schemes-for-tabs branch December 12, 2023 18:19
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.

3 participants