-
Notifications
You must be signed in to change notification settings - Fork 65
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
Приводит названия к единому шаблону #999
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Inventoris
requested review from
furtivite,
solarrust,
nlopin,
igsekor and
HellSquirrel
as code owners
October 12, 2022 13:09
nlopin
approved these changes
Oct 13, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
огонь, так точно лучше
igsekor
approved these changes
Oct 13, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Полетели!
igsekor
pushed a commit
that referenced
this pull request
Oct 19, 2022
solarrust
added a commit
that referenced
this pull request
Oct 27, 2022
* Дополняет список исключений * Дополняет список папку для рубрики «На собесе» * Добавляет поля для рубрики «На собесе» * Добавляет черновик шаблона для вопросов / ответов * Добавляет динамику работы для ответов на вопросы * Добавляет стили (такие же, как для советов) * Вносит правочку (#982) * Добавляет @HellSquirrel в codeowners (#991) * Добавляет дополнительные места для переноса текста (#983) * Исправляет нейминг (#989) * Запрещает оставлять один символ имени метода при переносе на следующую строку (#998) * Шаблонизирует названия (#999) * Добавляет поддержку ссылок на Telegram для страницы автора (#995) Co-authored-by: Igor Korovchenko <[email protected]> * Добавляет превью цвета в блоках кода (#996) * Добавляет код * Правит выравнивание * Добавляет роли для людей (#1004) * Добавляет конструктор тегов * Исправляет нейминг * Добавляет роль для автора * Окорачивает выражение * Добавляет проверку на наличие значения в ключах * Делает название роли в Практикуме с большой буквы * Прессует роль * Оставляет только один тип карточки для Твиттера (#1002) * Добавляет ярлыки к имени автора * Добавляет папку с интервью * Меняет заголовок * Добавляет описание к рубрике * Исправляет метод загрузки коллекций * Исправляет фильтрацию по ответам * Добавляет фильтр для меню навигации по материалу * Добавляет исключение для раздела a11y * Добавляет список статей с ответом автора * Правильно структурирует списки * Меняет название поля * Снова меняет название поля для ответов на вопросы * Выносит общий код повыше по абстракции * Добавляет авторов ответов в индекс * Рефакторит цвета ярлыков * Дорабатывает границы блоков * Меняет описание рубрики * Наклоняет текст описания * Добавляет фон для ховера на ярлыках * Поправляет ховер по умолчанию * Исправляет цвет для ховера * Повышает уровень свободы при выборе цвета * Исправляет нумерацию * Добавляет красный вопросик * Меняет структуру содержания материала * Брендирует рубрику в содержании * Исправляет фильтр для ответов на вопросы * Заворачивает вопрос в callout * Правит стили, добавляет условие, меняет ссылку * Меняет шило на мыло * Удаляет лишний стиль * Добавляет id к вопросу * Уточняет селектор * Возвращает содержание статьи в рабочее состояние * Подправляет стили и вёрстку для плейсходера * Приводит вёрстку в порядок * Чинит цвета для ховера на ярлыках * Добавляет трансформацию для картинок в ответах Co-authored-by: Alexey Nikitchenko <[email protected]> Co-authored-by: Egor Levchenko <[email protected]> Co-authored-by: Denis Putnov <[email protected]> Co-authored-by: Vadim Makeev <[email protected]> Co-authored-by: Alena Batitskaia <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Есть предложение привести названия, вроде
JS
илиJavaScript
, к единому шаблону.Сейчас то так:
То так:
В
src/transforms/article-code-blocks-transform.js
из-за этой путаницы даже есть код, который подглядывает в объектLANG_ALIASES
в поисках понятной призму записи.Можно перенести эти исключения из
article-code-blocks-transform
вsrc/markdown-it.js
и шаблонизировать названия до того, как вставлять их в атрибутdata-lang
.Плюсы:
article-code-blocks-transform
, поскольку туда всё будет попадать уже шаблонизированным;JavaScript
наJS
или что-то такое. Как хотят, так и пишут, код всё приведёт к единому стилю.Было:
Стало:
На мой взгляд, так больше порядка 😀