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

Добавляет доку с описанием деструктуризации #5506

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

Conversation

vitya-ne
Copy link
Contributor

Описание

Рассказываю про деструктуризации.
Рассматриваю основные способы применения.

Чек-лист

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

@github-actions github-actions bot added js Контент по JavaScript дока Справочный материал labels Sep 20, 2024
@TatianaFokina
Copy link
Member

TatianaFokina commented Sep 21, 2024

Я из своего редакторского уголка оставлю одно предложение. Можешь без проблем убрать часть заголовков, а то их слишком много. Из-за них, кажется, прерывается повествование.

И ещё можно это из доки в статью смело проапдейтить.

@vitya-ne
Copy link
Contributor Author

Можешь без проблем убрать часть заголовков, а то их слишком много. Из-за них, кажется, прерывается повествование.

Моя идея заключалась в упрощении поиска по заголовкам. Но, попробую поменять.

И ещё можно это из доки в статью смело проапдейтить.

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

@vitya-ne
Copy link
Contributor Author

@TatianaFokina, я уменьшил количество заголовков. Как считаешь, что ещё поменять ?

js/destructuring-assignment/index.md Outdated Show resolved Hide resolved
js/destructuring-assignment/index.md Outdated Show resolved Hide resolved
js/destructuring-assignment/index.md Outdated Show resolved Hide resolved
js/destructuring-assignment/index.md Outdated Show resolved Hide resolved
js/destructuring-assignment/index.md Outdated Show resolved Hide resolved
js/destructuring-assignment/index.md Outdated Show resolved Hide resolved
@github-actions github-actions bot added the статья Расширенный материал label Sep 28, 2024
Copy link

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

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 помержим?

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

Successfully merging this pull request may close these issues.

3 participants