-
-
Notifications
You must be signed in to change notification settings - Fork 59
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
add docs mol_link_source #629
add docs mol_link_source #629
Conversation
Lyumih
commented
Sep 9, 2023
link/source/demo.view.tree
Outdated
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.
Демки должны быть в отдельным модулях иначе поедут в прод вместе с компонентом.
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.
Как вариант на подумать, добавить к компоненту component.view.tree файл с расширением component.view.tree.demo (или другая последовательность c demo) и на уровне бандла выпиливать её. Сейчас папка /demo/ для каждого компонента выглядит неказисто если честно
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.
Модуль в MAM - это директория. Демки - это отдельные модули со своими отдельными зависимостями.
@@ -0,0 +1,20 @@ | |||
# $mol_link_source | |||
|
|||
Hyperlink on source with GitHub icon. |
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.
Думаю стоит поменять иконку на не связанную с гитхабом.
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.
Захардкожено в самом https://github.com/hyoo-ru/mam_mol/blob/master/link/source/source.view.tree#L4
<= Icon $mol_icon_github_circle
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.
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.
Можно на $mol_icon_script_text_outline
поменять.