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

Улучшает блок с Baseline #1187

Merged
merged 2 commits into from
Aug 11, 2023

Conversation

TatianaFokina
Copy link
Member

@TatianaFokina TatianaFokina commented Aug 10, 2023

По мотивам ишью #1143. Смотреть можно в доке про grid-area.

Что исправила:

  • убрала лишние ARIA-атрибуты и title из <span>, скрыла повторяющуюся инфу от скринридеров;
  • слегка изменила скрытый текст (убрала заглавную букву, так как это всё одно предложение);
  • вернула классы на место в одном <span>;
  • использовала наш цвет для текста и пофиксила проблемы с контрастностью;
  • слегка изменила способ задавания отступов, пофиксила лишний на мобилках и планшетах;
  • добавила отступ ко всему блоку.

Остался только один вопрос: в итоге мы планируем делать «Поддержка в браузерах» заголовком или нет? Если да, нужна ваша помощь, чтобы заголовок не попадал в оглавление статьи.

Frame 1

@TatianaFokina TatianaFokina added улучшение Доработка существующего вёрстка Задачи по вёрстке дизайн Задачи по дизайну labels Aug 10, 2023
@TatianaFokina TatianaFokina self-assigned this Aug 10, 2023
@github-actions
Copy link

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

@skorobaeus
Copy link
Member

Красивое!

Остался только один вопрос: в итоге мы планируем делать «Поддержка в браузерах» заголовком или нет? Если да, нужна ваша помощь, чтобы заголовок не попадал в оглавление статьи.

Есть интуитивное ощущение, что это неразделимые вещи. То есть, если решаем, что это достаточно значимо, чтобы сделать заголовком, то и из оглавления убирать не нужно.

@TatianaFokina
Copy link
Member Author

Тогда нужна помощь зала. Для нас достаточно значим блок с поддержкой браузерами? @igsekor, @solarrust, @furtivite, @HellSquirrel?

@TatianaFokina
Copy link
Member Author

TatianaFokina commented Aug 11, 2023

Давайте потом решим про заголовки. Чувствую, может надолго затянуться обсуждение, а там на проде проблемы с контрастностью сейчас.

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 TatianaFokina merged commit 8ab30c9 into doka-guide:main Aug 11, 2023
4 checks passed
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.

Что нужно исправить/улучшить в Baseline-блоке
2 participants