web-standards.ru icon indicating copy to clipboard operation
web-standards.ru copied to clipboard

Неприятности типографа

Open pepelsbey opened this issue 4 years ago • 15 comments

Вставил два пробела, заменил дефисы на тире

- 4. Далее производится проверка на пересечение множеств:
+ 4.  Далее производится проверка на пересечение множеств:
-    - если найдены общие ключевые слова — **положительный вердикт,** теги можно вкладывать.
+    — если найдены общие ключевые слова — **положительный вердикт,** теги можно вкладывать.

Заменил дефисы на тире, продублировал запятую снаружи

- - У вкладываемого тега нужно проверить секцию **Categories,** а у родительского тега — **Content model.**
+ — У вкладываемого тега нужно проверить секцию **Categories,**, а у родительского тега — **Content model.**

pepelsbey avatar May 05 '20 12:05 pepelsbey

@Mavrin, как думаешь, хотя бы списки поправить можно?

pepelsbey avatar May 05 '20 12:05 pepelsbey

С запятой однозначно баг и думаю исправляется. Пробелы это наверное remark вставляет, но проверю, возможно, нельзя починить или будет в другом месте переделывать, и потом падать на editorConfigLint. С дефисами и тире, не должно менять по причине того что это список? И можно весь файл целиком.

Mavrin avatar May 05 '20 13:05 Mavrin

С дефисами и тире, не должно менять

В маркдауне дефисы в начале строки делают эту строку элементом списка. И если строка в исходнике начинается с дефиса, то такой её и нужно оставить :)

Весь файл: src/articles/can-i-include/index.md

pepelsbey avatar May 05 '20 13:05 pepelsbey

Открой на гитхабе https://github.com/web-standards-ru/web-standards.ru/blob/master/src/articles/can-i-include/index.md заметь он не парсит список. Поэтому и типограф считает это, просто пердложением (но это не точно). Screen Shot 2020-05-05 at 16 27 14

Mavrin avatar May 05 '20 13:05 Mavrin

С пробелами должно помочь listItemIndent: mixed в .remarkrc.js. Нужно попробовать заменить mixed на 2.

Holiden avatar May 05 '20 13:05 Holiden

Там нельзя 2. Но все верно эта настройка, как раз может помочь, но будет другой сайд-эффект с вложенными списками. https://github.com/remarkjs/remark/tree/master/packages/remark-stringify#optionslistitemindent

Style of indentation for list items ('tab', 'mixed' or '1', default: 'tab').

Mavrin avatar May 05 '20 13:05 Mavrin

Там нельзя 2. Но все верно эта настройка, как раз может помочь, но будет другой сайд-эффект с вложенными списками. https://github.com/remarkjs/remark/tree/master/packages/remark-stringify#optionslistitemindent

Style of indentation for list items ('tab', 'mixed' or '1', default: 'tab').

Точно, имел ввиду 1. А с вложенными кажется может помочь плагин lint-list-item-content-indent.

Holiden avatar May 05 '20 14:05 Holiden

Посмотри внимательно на что ругается это плагин. это ок, для еditor config не ок

1.·[x] Alpha
···1. Bravo

это не ок, для еditor config ок

1.·[x] Charlie
····1. Delta

Mavrin avatar May 05 '20 14:05 Mavrin

@pepelsbey со списками все работает хорошо. https://github.com/web-standards-ru/web-standards.ru/pull/64/files. На сайте из-за этого не отображаются списки. Screen Shot 2020-05-05 at 21 27 08

Mavrin avatar May 05 '20 18:05 Mavrin

Посмотри внимательно на что ругается это плагин. это ок, для еditor config не ок

Я не очень понимаю (

pepelsbey avatar May 05 '20 19:05 pepelsbey

пример ошибки https://github.com/web-standards-ru/web-standards.ru/pull/66 можно еще так настроить https://github.com/web-standards-ru/web-standards.ru/pull/67

Mavrin avatar May 05 '20 20:05 Mavrin

Скажу честно, я запутался — поэтому не могу взяться. Я вижу три пулреквеста: #65, #66, #67. Какой из них нужен и зачем два остальных? :)

pepelsbey avatar May 09 '20 14:05 pepelsbey

#65 я исправил проблему с пунктуацией (про лишнюю запятую). Его можно мержить. #66 и #67 это для обсуждения. #66 это пример, что получится, если вы ставить в настройках remark на стройку для отступов 1, в этом случае видишь падает editor config linter. #67 это пример, что получится, если вы ставить в настройках remark на стройку для отступов tab.

Mavrin avatar May 09 '20 15:05 Mavrin

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

pepelsbey avatar May 09 '20 15:05 pepelsbey

Посмотри внимательно на что ругается это плагин. это ок, для еditor config не ок

Я не очень понимаю (

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

Mavrin avatar May 09 '20 15:05 Mavrin