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

Добавляет якоря для мнений авторов #279

Merged
merged 4 commits into from
Jul 28, 2021

Conversation

furtivite
Copy link
Member

@furtivite furtivite commented Jul 27, 2021

В блоке «В работе» у мнений авторов не было ссылок-якорей. Этот PR:

  • Добавляет якоря для каждого мнения
  • Добавляет быстрые ссылки для каждого мнения
  • Якоря уникальны
  • Якоря содержат уникальный для статьи aria-label

Было

Снимок экрана 2021-07-27 в 10 19 30

Стало

Снимок экрана 2021-07-27 в 10 20 00

src/includes/practices.njk Outdated Show resolved Hide resolved
Copy link
Contributor

@pepelsbey pepelsbey left a comment

Choose a reason for hiding this comment

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

Мне кажется, что собирать схему block__element для ID это плохая затея. Это добавляет шум, как будто в шаблоне появились новые элементы блока. Но это не так, это просто айдишники. Плюс они потом попадают в адреса и ссылки, а значит будет

doka.guide/html/doctype/#practices__something

Что нарушает нашу договорённость в адресах писать через дефисы и ненужно удлинняет адрес. Лучше и привычнее так:

doka.guide/html/doctype/#practices-something

@furtivite
Copy link
Member Author

Мне кажется, что собирать схему block__element для ID это плохая затея. Это добавляет шум, как будто в шаблоне появились новые элементы блока. Но это не так, это просто айдишники. Плюс они потом попадают в адреса и ссылки, а значит будет

doka.guide/html/doctype/#practices__something

Что нарушает нашу договорённость в адресах писать через дефисы и ненужно удлинняет адрес. Лучше и привычнее так:

doka.guide/html/doctype/#practices-something

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

@furtivite furtivite requested a review from pepelsbey July 28, 2021 09:32
@pepelsbey pepelsbey merged commit 18857bc into main Jul 28, 2021
@pepelsbey pepelsbey deleted the issue/136-practices-ankors branch July 28, 2021 10:40
furtivite added a commit that referenced this pull request Aug 27, 2021
* Добавляет базовую вёрстку страницы статьи

* Добавляет шаблон для работы с шапкой

* Удаляет лишний margin

* Добавляет моноширинный шрифт для лого

* Фикс fallback-фона для header

* Чистит index.css от комментариев

* Удаляет устаревший якорь из #279

* Рефакторит использование трансформаций

* Уменьшает время анимации исчезновения

* Правит `hotkey` в шапке

* Правит стиль кода

Fix: #301

* Удаляет лишний `aside.js`

* Меняет теги

* Меняет приоритет поиска ресурсов для разработки

* Добавляет шрифт Graphik

* Удаляет старые шрифты

* Правки шрифтов в шапке

* Фиксит подсветку для невалидных меток кода

* Правит генерацию блоков кода

* Правки header

* Добавляет стили для iframe

* Стили для placeholder и времени чтения

* Чинит расстановку строк

Co-authored-by: Egor Levchenko <antytoto@gmail.com>
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.

Добавить якоря на имена авторов в блок «В работе»
3 participants