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

Добавил плагин для встраивания на сайт #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Lyumih
Copy link

@Lyumih Lyumih commented May 11, 2024

image
image
image

Comment on lines 1 to 2
- Плагин для встраивания на сайт кнопки-формы с обратной связью.
- id - идентификатор встречи. Можно получит на сайте https://survey.hyoo.ru/ в url.
Copy link
Member

Choose a reason for hiding this comment

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

- \комментарии пишутся так.

Copy link
Member

Choose a reason for hiding this comment

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

Но вообще, этому место в readme.md

Copy link
Author

Choose a reason for hiding this comment

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

Убрал

Comment on lines 1 to 2
- Плагин для встраивания на сайт кнопки-формы с обратной связью.
- id - идентификатор встречи. Можно получит на сайте https://survey.hyoo.ru/ в url.
Copy link
Member

Choose a reason for hiding this comment

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

Но вообще, этому место в readme.md

id \
realm $hyoo_crus_realm
sub /
<= Feedback $mol_pick
Copy link
Member

Choose a reason for hiding this comment

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

Попапы с большими формами - такая себе идея. На малых экранах экранах это даёт плохой UX. Также попап можно закрыть случайно.

Copy link
Author

Choose a reason for hiding this comment

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

Пока лучше и надёжнее не придумал)
Если будет на что заменить, замени пожалуйста

@@ -0,0 +1,13 @@
- Плагин для встраивания на сайт кнопки-формы с обратной связью.
- id - идентификатор встречи. Можно получит на сайте https://survey.hyoo.ru/ в url.
$hyoo_survey_plugin $mol_page
Copy link
Member

@nin-jin nin-jin May 11, 2024

Choose a reason for hiding this comment

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

Плагин - слишком уж абстрактное название.

Copy link
Author

Choose a reason for hiding this comment

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

заменил на embed

<= Feedback $mol_pick
hint \Обратная связь
trigger_content /
<= Feedback_icon $mol_icon_thumb_up_outline
Copy link
Member

@nin-jin nin-jin May 11, 2024

Choose a reason for hiding this comment

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

Кнопка лайка вводит в заблуждение. У человека может быть негатив и он не поймёт, что надо жать на лайк, чтобы его выразить.

Copy link
Author

Choose a reason for hiding this comment

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

Заменил на звёздочки $mol_icon_creation_outline

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants