Skip to content

Добавляет черновик CSS-индекса - #1116

Merged
solarrust merged 9 commits into
mainfrom
edit/css-index
Jul 30, 2021
Merged

Добавляет черновик CSS-индекса#1116
solarrust merged 9 commits into
mainfrom
edit/css-index

Conversation

@pepelsbey

Copy link
Copy Markdown
Contributor

Можно всё выбросить и оставить только структуру :)

@pepelsbey pepelsbey added кухня Документация, репозиторий, задачи редакции css Контент по CSS labels Jul 7, 2021
@pepelsbey
pepelsbey requested a review from solarrust July 7, 2021 11:05
@pepelsbey
pepelsbey marked this pull request as draft July 7, 2021 11:05
@github-actions

github-actions Bot commented Jul 7, 2021

Copy link
Copy Markdown

Ошибка публикации превью для контента из 261afbf. Подробнее в логах.

@solarrust

Copy link
Copy Markdown
Member

Пока просто заполнила, есть сомнения по сортировке разделов, к этому вернусь позже

@pepelsbey

Copy link
Copy Markdown
Contributor Author

Первые мысли:

  1. Я бы положил vertical-align в текст, рядом с text-align, поскольку он действует в инлайновом, текстовом контексте
  2. Общие свойства языка: глобальные ключевые слова, important, кастомные свойства я бы положил вместе

@solarrust

solarrust commented Jul 9, 2021

Copy link
Copy Markdown
Member

Тоже раздумывала над vertical-align в разделе работы с текстом. Но не решилась. Спасибо что подтвердил мои мысли.
Я старалась избегать таких общих разделов, где навалено всё подряд. Тем более что в разделе про те же кастомные свойства планируется как минимум три статьи: общая, про var() и про :root. Глобальные ключевые слова, пожалуй, можно убрать в основы, в первый раздел. А импортная так и будет болтаться отдельно, как его не ставь: хоть по алфавиту, хоть по смыслу, хоть по принадлежности =)

@solarrust
solarrust marked this pull request as ready for review July 26, 2021 09:20
Comment thread css/index.md Outdated
Comment thread css/index.md
Comment thread css/index.md Outdated
@solarrust
solarrust merged commit cbc9414 into main Jul 30, 2021
@solarrust
solarrust deleted the edit/css-index branch July 30, 2021 13:43
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.

2 participants