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

Добавляет статью про Псевдоприватные кастомные свойства #5577

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

Conversation

Alex-Andr-19
Copy link

@Alex-Andr-19 Alex-Andr-19 commented Dec 6, 2024

Описание

Создание статьи в раздел Рецепты про перенос миксинов SASS'а на переменные CSS в атомарных классах.

Превью: https://content-5577.dev.doka.guide/css/pseudo-private-custom-property/

Чек-лист

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

@github-actions github-actions bot added the рецепт Контент для Рецептов label Dec 6, 2024
@solarrust solarrust changed the title feat: Create article about virtual variables Добавляет статью про Виртуальные переменные Dec 6, 2024
@Alex-Andr-19 Alex-Andr-19 changed the title Добавляет статью про Виртуальные переменные Добавляет статью про Псевдоприватные кастомные свойства Dec 6, 2024
@ra1nbow1

This comment was marked as resolved.

@Alex-Andr-19

This comment was marked as resolved.

Inventoris

This comment was marked as resolved.

@Alex-Andr-19
Copy link
Author

@Inventoris Хорошо, спасибо за совет

@Alex-Andr-19

This comment was marked as resolved.

@github-actions github-actions bot added the статья Расширенный материал label Dec 25, 2024
@TatianaFokina

This comment was marked as resolved.

@Alex-Andr-19

This comment was marked as resolved.

solarrust

This comment was marked as resolved.

@github-actions github-actions bot added the css Контент по CSS label Jan 9, 2025
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.

Принесла небольшие правки по тексту. Ничего глобального, смысл не должен был нарушиться.

Получилось очень круто! Спасибо тебе =)

От меня аппрув.
Зову @skorobaeus помочь с демками 🧚‍♀️

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.

Привет! С демками всё хорошо :) К ним только нужно прикрутить стили отсюда: https://github.com/doka-guide/content/blob/main/docs/demos/style.md

Нам потребуются цвета и шрифты. Если возникнут сложности, я сделаю всё сама, но, видимо, уже после того, как уйдёт рассылка :)

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.

Привет ещё раз!

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

У док и статей есть типовая структура, давай попробуем её соблюсти.

Эту статью очень украсит вводная часть, но не формальная, для галочки, а по существу. В такой вводной части первым делом стоит объяснить простыми словами что такое псевдоприватные кастомные свойства. Рассказать, зачем они нужны: чтобы изолировать стили, избегать конфликтов, давать пользователям рычаги для настройки компонентов, не трогая внутреннюю логику, и т.д. И наконец, отметить, что это синтаксический сахар и показать, как пишется. Чем чётче и яснее здесь пример кода, тем лучше; он не обязан быть связан с демкой.

После вводной части переходим к разбору практического примера: ставим задачу (вёрстка карточек), обозначаем проблему (код раздувается до нечитаемости, из-за взаимозависимостей стилей его становится очень легко сломать), опционально (!) нагнетаем ситуацию (если прикрутить миксины, то получится жесть ваще бррр), даём решение проблемы (показываем идентичный результат на чистом CSS).

Наконец, завершаем статью: договариваем всё, о чём не договорили, делаем выводы.

Прости, пожалуйста, за такую простыню фидбека 😅 Я очень обрадовалась возможности избавиться от миксинов и теперь очень хочу, чтобы из этой статьи получилась конфетка :)

@Alex-Andr-19
Copy link
Author

Alex-Andr-19 commented Feb 11, 2025

@skorobaeus Спасибо большое за такую крутую обратную связь! Мне это действительно важно – нужно учиться писать красиво)

@Alex-Andr-19
Copy link
Author

Alex-Andr-19 commented Feb 13, 2025

@skorobaeus Добавил вводную часть. Надеюсь вышло по-лучше)

@Alex-Andr-19
Copy link
Author

@skorobaeus Извини, что лишний раз пишу, но я очень жду ревью.

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.

Привет! Спасибо тебе большое за быстрый отклик!

Мне кажется, стало очень последовательно и понятно ❤ Я немного покомментировала и позадавала последние вопросы. Надо с ними разобраться, а дальше только с запятыми пройдусь и будет огонь!

@Alex-Andr-19
Copy link
Author

@skorobaeus Внёс все нужные правки

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.

Ура, получилось шикарно! Спасибо тебе огромное за работу ❤

@Alex-Andr-19
Copy link
Author

@ra1nbow1 @Inventoris @baileys-li Жду ваших ревью)

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.

Привет! Оставил пару комментариев

.product-card {
--_background-color: var(--background-color, #123E66);

--_lighter-color: hsl(from var(--_background-color) h s calc(l * 1.3));
Copy link
Member

Choose a reason for hiding this comment

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

Здесь используется l, давай в коде ее объявим

Copy link
Author

Choose a reason for hiding this comment

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

Так работает функция hsl, я же не объявлял h и s. Функция предоставляет интерфейс работы с цветом, который передан первым аргументом, через пространство hsl, где h, s, l будут конкретными числами

Copy link

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

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

Successfully merging this pull request may close these issues.

7 participants