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

fix headlines font-size #481

Closed

Conversation

wadzari
Copy link
Contributor

@wadzari wadzari commented Aug 30, 2022

#468 Добавил разделение заголовков по семантике и размерам

image

@batyshkaLenin
Copy link
Collaborator

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

@wadzari
Copy link
Contributor Author

wadzari commented Aug 31, 2022

@batyshkaLenin @philippranzhin Ну тут я накидал размеры чисто по наитию 😅 (взял те, что были в константах). Я тоже за дизайнерский взгляд)

@AndyAbramov
Copy link

AndyAbramov commented Oct 20, 2022

на странице может быть 3 заголовка:

  1. название материала,
  2. основной заголовок внутри материала,
  3. дополнительный заголовок внутри материала.

на практике такое визуальное ограничение сужает лишнюю стилизацию статьи, снижает визуальный шум и делает материалы консистентнее.

если автор указал заголовок как следующий ниже по иерархии, то он должен определяться как дополнительный (третий в списке).

примерил и показал в фигме: https://www.figma.com/file/NO0RmwhfZnsOjW1cexIVEW/%F0%9F%94%B8-rzrbs?node-id=4904%3A11117

исходя из этого и ориентируясь на скрин:

  • заголовок первого уровня - это название материала. в "теле" по идее не должен использоваться,
  • заголовок второго уровня - основной заголовок внутри материала,
  • заголовок третьего уровня - дополнительный заголовок. по стилям почти попали: следует скорректировать letter-spacing, и посмотреть отступы, как будут стыковаться с абзацами и другими заголовками. в макете показано.

остальные следует упразднить или определить по стилям как заголовок третьего уровня.

@raantoniv
Copy link
Member

Не актуально

@raantoniv raantoniv closed this Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants