Skip to content

Редактирует статью про font-synthesis - #5925

Open
gazievri wants to merge 12 commits into
doka-guide:mainfrom
gazievri:fix/font-synthesis
Open

Редактирует статью про font-synthesis#5925
gazievri wants to merge 12 commits into
doka-guide:mainfrom
gazievri:fix/font-synthesis

Conversation

@gazievri

@gazievri gazievri commented Dec 4, 2025

Copy link
Copy Markdown

Описание

  1. Дополнил статью про font-synthesis в разделе css
  2. Добавил нового автора в people: gazievri

Идут проверки пулреквеста, следом соберётся превью контента из 9e1b178. Подробнее

@github-actions github-actions Bot added css Контент по CSS дока Справочный материал labels Dec 4, 2025
Comment thread css/font-synthesis/index.md Outdated
Comment thread css/font-synthesis/index.md Outdated
@vitya-ne

vitya-ne commented Dec 6, 2025

Copy link
Copy Markdown
Contributor

Привет,
спасибо за доку и отдельно за конфиг baseline )

Посмотри, пожалуйста, пару мелких коментов

@gazievri
gazievri requested a review from vitya-ne December 12, 2025 22:43
Comment thread css/font-synthesis/index.md Outdated
Comment thread css/font-synthesis/index.md Outdated

@vitya-ne vitya-ne left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Я тут ещё кое-что добавил, посмотри пожалуйста

@gazievri
gazievri requested a review from vitya-ne December 14, 2025 22:38
Comment thread css/font-synthesis/index.md Outdated

@vitya-ne vitya-ne left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Спасибо за дополнение. Оставил один комент.

@gazievri
gazievri requested a review from vitya-ne December 15, 2025 07:05
@vitya-ne

Copy link
Copy Markdown
Contributor

@gazievri Привет,
посмотри, пожалуйста, на порядок блоков доки: https://github.com/doka-guide/content/blob/main/docs/examples/doka.md

  • 'Как понять' и 'Как пишется' надо поменять местами.
  • части с подзаголовками из раздела 'Как пишется' (Variable fonts и font-synthesis, Когда использовать font-synthesis: none и остальные), по-моему лучше перенести в 'Как понять' ведь там не описывается синтаксис font-synthesis

Comment thread css/font-synthesis/index.md Outdated
Comment thread css/font-synthesis/index.md Outdated

@vitya-ne vitya-ne left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ещё пара коментов

@vitya-ne
vitya-ne self-requested a review December 18, 2025 20:02
Comment thread css/font-synthesis/index.md Outdated

@vitya-ne vitya-ne left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Посмотри, пожалуйста на описание в разделе При использовании Google Fonts

@gazievri
gazievri requested a review from vitya-ne January 24, 2026 20:18

@vitya-ne vitya-ne left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Спасибо за дополнения! Возможно некоторые тонкости выходят за рамки темы, но при окончательной редактуре это можно поправить. )
Добавил ещё несколько мелочей.

}
```

Это может приводить к неожиданному поведению в сложных layouts, поэтому важно учитывать наследование при использовании `font-synthesis` на верхних уровнях DOM-дерева.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Предлагаю:

  • поднять это предложение выше (над примерами) - ведь оно является продолжением мысли;
  • немного уточнить.

Это может приводить к неожиданному поведению в сложных layouts

  • "сложный layouts" лучше заменить на какой-то аналог полностью на русском, например:
    "в глубоко вложённой структуре элементов"

Comment thread css/font-synthesis/index.md

@nasty23-star nasty23-star left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Привет! Большое тебе спасибо за желание закончить незавершённую статью!
(@^◡^)つ❀✾✿
Оставила несколько комментариев, посмотри, пожалуйста.

Comment thread css/font-synthesis/index.md

### Когда синтез особенно заметен

Синтетические варианты шрифта особенно плохо смотрятся:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Может быть, есть возможность прикрепить пару скриншотов с примерами синтетических и обычных шрифтов? Было бы здорово.

Comment thread css/font-synthesis/index.md
Comment thread css/font-synthesis/index.md
Comment thread css/font-synthesis/index.md
Comment thread css/font-synthesis/index.md

@nasty23-star nasty23-star left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Спасибо!

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.

4 participants