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.

Sign up for GitHub

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

Jump to bottom

Меняет список на подзаголовки #4697

Merged
merged 2 commits into from
Oct 20, 2023
Merged

Меняет список на подзаголовки #4697

merged 2 commits into from
Oct 20, 2023

Conversation

tiMURtSE
Copy link
Contributor

@tiMURtSE tiMURtSE commented Sep 6, 2023

Описание

На страничке про тег поменял список атрибутов на подзаголовки

Чек-лист

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

@github-actions github-actions bot added html Контент по HTML дока Справочный материал labels Sep 6, 2023
@TatianaFokina TatianaFokina added the улучшение Доработка существующего label Sep 6, 2023
@tiMURtSE tiMURtSE mentioned this pull request Sep 6, 2023
Closed
skorobaeus
skorobaeus approved these changes Sep 7, 2023
View reviewed changes
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.

По-моему, хорошо! В части про атрибут name сниппеты кода немного сдвинуты, но даже так страница стала читаться проще ❤

@tiMURtSE
Copy link
Contributor Author

tiMURtSE commented Sep 7, 2023 •
edited
Loading

Может, стоит подзаголовок про content передвинуть на второе место сразу после charset? Чтобы изначально прочитать проcontent. Иначе пользователь видит код с этим атрибутом и только в самом конце читает про сам атрибут.

@skorobaeus
Copy link
Member

Может, стоит подзаголовок про content передвинуть на второе место сразу после charset? Чтобы изначально прочитать проcontent. Иначе пользователь видит код с этим атрибутом и только в самом конце читает про сам атрибут.

Да, отличная идея. Здорово, что ты заметил!

@solarrust
Copy link
Member

Привет! Нужна помощь с этим пиаром?

@github-actions
Copy link

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

solarrust
solarrust approved these changes Oct 20, 2023
View reviewed changes
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.

Спасибо большое за улучшение Доки! 🚀

@solarrust solarrust merged commit 322663c into doka-guide:main Oct 20, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skorobaeus skorobaeus skorobaeus approved these changes

@solarrust solarrust solarrust approved these changes

Assignees
No one assigned
Labels
html Контент по HTML дока Справочный материал улучшение Доработка существующего
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@tiMURtSE @skorobaeus @solarrust @TatianaFokina