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

Добавляет статью про CSS-in-JS #5670

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

Conversation

RuslanPro01
Copy link
Contributor

@RuslanPro01 RuslanPro01 commented Jan 26, 2025

Описание

Добавляет статью про CSS-in-JS решения.
Это мой превый подобный опыт, так что буду рад обратной связи 😊

Closes #5238

Превью: https://content-5670.dev.doka.guide/tools/css-in-js/

Чек-лист

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

@github-actions github-actions bot added веб-платформа Контент по Веб-платформе статья Расширенный материал labels Jan 26, 2025
@TatianaFokina TatianaFokina changed the title Article/css in js Добавляет статью про CSS-in-JS Jan 26, 2025
Copy link
Member

@ra1nbow1 ra1nbow1 left a comment

Choose a reason for hiding this comment

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

Привет!

Спасибо за такую фундаментальную документацию) Я внес пару предложений

tools/css-in-js/index.md Outdated Show resolved Hide resolved
tools/css-in-js/index.md Outdated Show resolved Hide resolved
tools/css-in-js/index.md Outdated Show resolved Hide resolved
tools/css-in-js/index.md Outdated Show resolved Hide resolved
tools/css-in-js/index.md Outdated Show resolved Hide resolved
tools/css-in-js/index.md Outdated Show resolved Hide resolved
tools/css-in-js/index.md Outdated Show resolved Hide resolved
tools/css-in-js/index.md Outdated Show resolved Hide resolved
@RuslanPro01
Copy link
Contributor Author

@ra1nbow1 большое спасибо за апдейт, беру в работу!)

@RuslanPro01 RuslanPro01 requested a review from ra1nbow1 January 26, 2025 13:28
tools/css-in-js/index.md Outdated Show resolved Hide resolved
Статья получилась очень теоритической, кажется, что кусочки кода чуть-чуть добавят ясности
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.

Потрясно! С большим интересом прочитала. Спасибо!

Оставила несколько предложений по улучшению. Не пугайся, что их много. Все они больше про форматирование и про перестановку слов =)
Всё можем обсуждать!

tools/css-in-js/index.md Outdated Show resolved Hide resolved
tools/css-in-js/index.md Outdated Show resolved Hide resolved
tools/css-in-js/index.md Outdated Show resolved Hide resolved
tools/css-in-js/index.md Outdated Show resolved Hide resolved
tools/css-in-js/index.md Outdated Show resolved Hide resolved
tools/css-in-js/index.md Outdated Show resolved Hide resolved
tools/css-in-js/index.md Outdated Show resolved Hide resolved
tools/css-in-js/index.md Show resolved Hide resolved
tools/css-in-js/index.md Outdated Show resolved Hide resolved
tools/css-in-js/index.md Outdated Show resolved Hide resolved
@RuslanPro01 RuslanPro01 requested a review from solarrust January 29, 2025 14:14
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.

Кайф, мне всё нравится!

Copy link

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

@RuslanPro01
Copy link
Contributor Author

@solarrust, подскажи, пожалуйста, а мы тут ещё апрувы ждём или нужно сделать что-то еще?)

@solarrust
Copy link
Member

@solarrust, подскажи, пожалуйста, а мы тут ещё апрувы ждём или нужно сделать что-то еще?)

Ага, ждём ещё аппрув от редактора раздела @HellSquirrel =)

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.

Не хватает материала о CSS-in-JS решениях
3 participants