search

finec-mgimo / finec-mgimo.github.io

Финэк МГИМО - программы бакалавриата и магистратуры по экономике, менеджменту, туризму и бизнес-информатике в кампусе МГИМО-Одинцово.
https://finec.mgimo.ru
MIT License
1 stars 8 forks source link

отсутствует шорткод для mermaid #210

Closed epogrebnyak closed 2 years ago

epogrebnyak commented 2 years ago

нам нужен тэг mermaid = true в front matter, сейчас такого нет - hugo не распознает ключ mermaid в заголовке

Нужно для #192

epogrebnyak commented 2 years ago

Скорее проблема в отсутствии шорткода, а не поля в заголовке

epogrebnyak commented 2 years ago

https://getdoks.org/docs/how-to/extra-features/mermaid/

elisad5791 commented 2 years ago

Взяла в работу.

epogrebnyak commented 2 years ago

@elisad5791 - просьба сделать на отдельном бранче в этом репо

epogrebnyak commented 2 years ago

Вопрос открытый, я не знаю.Если разберётесь - предложите, если нет - разассайните себя из ишью, чтобы кто другой мог сделать.

Help wanted - это более сложные ишью в проекте, в отличие от good first issue.

вс, 17 июл. 2022 г., 18:06 elisad5791 @.***>:

там нужно при mermaid:true подключать скрипт mermaid в шаблоне. В каком? blog/single.html ?

— Reply to this email directly, view it on GitHub https://github.com/finec-mgimo/finec-mgimo.github.io/issues/210#issuecomment-1186543800, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACGWBLRK7V33TAPNYBMD35LVUQON3ANCNFSM53X7DTLQ . You are receiving this because you authored the thread.Message ID: @.***>

elisad5791 commented 2 years ago

Я сделала, как описано в документации. Но вот сам рабочий процесс не понимаю. Склонировала к себе, сделала ветку mermaid, внесла изменения. Хочу запушить git push origin mermaid Ругается, что прав нет.

epogrebnyak commented 2 years ago

Я сделала, как описано в документации. Но вот сам рабочий процесс не понимаю. Склонировала к себе, сделала ветку mermaid, внесла изменения. Хочу запушить git push origin mermaid Ругается, что прав нет.

Поднял права, должно проходить.

Документация нацелена на внешний форк (внешние котрибьютеры делают форк и PR в preview, внутренние открывают бранчи и ревьюим в ишью, до PR), ее надо менять в рамках #208.

По workflow - лучше сначала спросить, с каким ишью нужна помощь - сейчас получается, что для не очень приоритетного ишью мы тратим фокус внимания.

epogrebnyak commented 2 years ago

@elisad5791 - если можете - переименуйте бранч, чтобы был с номером ишью (см #208)

epogrebnyak commented 2 years ago

@elisad5791 - где у нас теперь референс на коммит в ишью?

epogrebnyak commented 2 years ago

@super16:

super16 commented 2 years ago

Доработал поддержку mermaid на оптимизированную (то как это реализовано в Doks), теперь поддерживается оба синтаксиса из инструкции:

{{< mermaid class="bg-light text-center" >}}
graph TD
  A[Hard] -->|Text| B(Round)
  B --> C{Decision}
  C -->|One| D[Result 1]
  C -->|Two| E[Result 2]
{{< /mermaid >}}
```mermaid
graph TD
  A[Hard] -->|Text| B(Round)
  B --> C{Decision}
  C -->|One| D[Result 1]
  C -->|Two| E[Result 2]


Теперь поддержка mermaid берётся не из публичного CDN, а генерируется во время сборки сайта, а также CDN не поддерживает Markdown-синтаксис. В макросе mermaid также можно передавать собственные CSS-классы.
super16 commented 2 years ago

Соответственно, в песочнице добавил примеры из инструкции и наработки из #192, а также рендеринг: https://finec.mgimo.ru/sandbox/mermaid/

epogrebnyak commented 2 years ago

@super16 - спасибо, то что нужно