Комментарии

+2
maxisoft maxisoft Компонент "Реакции" 11 дней назад

Ну будет полезно если данная разработка не планируется дальше монетизировать его, то да имеет смысл сделать доступны на github, чтобы можно было строним разработчикам также принять участи в разработке. Простой пример понимаю что код еще будет дорабатываться но могу сразу указать на ряд ошибок и оптимизации в коде.

1. стоит оптимизировать код js 

  1. var reactionType = clickedReaction.data('reaction');
  2. var item_id = clickedReaction.closest('.content_reactions').data('item-id');
  3. var ctype_id = clickedReaction.closest('.content_reactions').data('ctype-id');
  4.  
  5.  

стоит 1 раз получить DOM элемент и потом получать сразу всю data-атрибут

2. проблема с отправкой реакции, стоит сделать проверку на ajax запрос, чтобы не постили простым POST запросом и не спамили. 

  1. public function actionAdd() {
  2.  
  3. if (!$this->request->isAjax()){
  4. cmsCore::error404();
  5. }
  6.  

3. столкнулся с тем что есть ошибка формирования запроса, например на последней версии хромиума (хром, яндекс) не будет нормально работать отправка запроса. Так как вы сделали MIXED тип отправки, сделали POST отправку даных но при этом сделали GET параметр в запросе, из за этого система не может получить данные, корректно 

  1. $.post('/reactions/add?' + $.now(), requestData, function (result) {
+1
Foxman23 Foxman23 Компонент "Реакции" 11 дней назад

Я вас поняла. Стиль самих эмодзи пока такой-  символьный. Будет — как в премиум решении от вордпресса — с анимацией и lotti анимацией

+1
Panda58dev Panda58dev Компонент "Реакции" 11 дней назад

Не совсем поняла что вы имеете ввиду

Я про эту картинку:

ИзображениеВыбранная реакция, больше, относительно остальных. Подобное решение, ещё было в доке OSX:

ИзображениеПравда, это программы, а не реакции 😅

Думаете это кому-то будет нужно?)))

Ну, например, вместо того, чтобы писать код в комментариях, я и другие разработчики, могли бы прислать pull request) Но, дело ваше, тут настаивать не буду, т.к. на роль мэйнтейнера не претендую :D

0
Foxman23 Foxman23 Компонент "Реакции" 11 дней назад

Здравствуйте! Спасибо за фидбек!

появляется высплывающие уведомление с текстом вроде «Что бы оставлять реакции, вы должны быть зарегестрированы!» или вроде того. Вообще, идеально, если текст можно будет настраивать

Да, это уже есть (правда пока только у меня 🤗)

В «beta» версии есть лайк (👍🏻), но нет дизлайка (👎🏻). Как понял, в будущем, будет поправлено

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

Имхо, выбранная реакция, лучше смотриться как в примере, в начале статьи, то бишь — увеличенная:

Не совсем поняла что вы имеете ввиду, но очень интересная идея)

И было бы круто, увидеть код на github/gitflic или т.п

Думаете это кому-то будет нужно?)))

+1
Panda58dev Panda58dev Компонент "Реакции" 11 дней назад

Штука нужная и клёвая!
Фидбэк пожалуй будет состоять из «новых плюшек»:
— Если юзверь не авторизирован, и пытаеться поставить реакцию, не просто «return false», а появляется высплывающие уведомление с текстом вроде «Что бы оставлять реакции, вы должны быть зарегестрированы!» или вроде того. Вообще, идеально, если текст можно будет настраивать;
— Добавить в CSS, что-то вроде:

  1. .content_reactions {
  2. -webkit-touch-callout: none;
  3. -webkit-user-select: none;
  4. -khtml-user-select: none;
  5. -moz-user-select: none;
  6. -ms-user-select: none;
  7. user-select: none;
  8. }

Иначе может произойти такое:

Изображение

— В «beta» версии есть лайк (👍🏻), но нет дизлайка (👎🏻). Как понял, в будущем, будет поправлено;
— Имхо, выбранная реакция, лучше смотриться как в примере, в начале статьи, то бишь — увеличенная:

  1. .content_reactions .reaction {
  2. margin: 0 4px;
  3. cursor: pointer;
  4. font-size: 24px;
  5. transition: scale 0.2s;
  6. display: inline-block;
  7. }
  8. .reaction.selected {
  9. scale: 1.5;
  10. }
  11.  

В идеале, если будут глобальные настройки модуля — возможность вставки произвольного css;

— И было бы круто, увидеть код на github/gitflic или т.п.

0
Foxman23 Foxman23 Компонент "Реакции" 11 дней назад

Здравствуйте! Да, видимо так и должно быть в идеале — но пока это бета, так что все предложения принимаются и внесутся в обновлении)

+1
Def Def Компонент "Реакции" 11 дней назад

хорошая идея. Только под гостем при клике надо предлагать авторизоваться, чтобы юзер понимал как проголосовать

0
ALASTAR ALASTAR InstantCMS 2.16.3 11 дней назад

С PHP 8.3 полностью совместимо?

0
F_a_R_i_D F_a_R_i_D Обновление вакансии и резюме 12 дней назад

Включил возможность делать вип, для админов 0. Всеравно пользователи могут делать вип. Если даже в доступе ставить им какую-то сумму, то всеравно эта услуга им видна.

0
Yar733 Yar733 IcmsBilling 12 дней назад

Приветствую. К сожалению, на гитхаб открывается 404((как скачать компонент?

0
My-InstantCMS.Ru My-InstantCMS.Ru Обновление вакансии и резюме 13 дней назад

Обновление всех компонентов можете получиться на моем сайте, в разделе Мои покупки

0
F_a_R_i_D F_a_R_i_D Авторизация с помощью социальных сетей 13 дней назад

Отлично. Обращусь к вам на днях.

0
Zau4man Zau4man Авторизация с помощью социальных сетей 13 дней назад

Постинг в телеграм уже есть версия с открытым исходным кодом, стоит дороже, 2 т.р. instantcms.ru/addons/ztelegram.html

0
Def Def Обновление вакансии и резюме 13 дней назад

а как получить обновление? НА почту jurist1@mail.ru

0
Саня Саня Авторизация с помощью социальных сетей 13 дней назад

Тоже готов скинуться

0
F_a_R_i_D F_a_R_i_D Авторизация с помощью социальных сетей 13 дней назад

Отлично.

Я готов сразу вступить в ряды этих 10 человек.

Также может заинтересовать постинг в телеграм.

0
Zau4man Zau4man Авторизация с помощью социальных сетей 13 дней назад

Есть. Как придет время, создам тему на форуме в поиске покупателей на версию с открытым кодом. Человек 10 надо будет насобирать, и выпущу для них компонент с открытым исходным кодом.

0
F_a_R_i_D F_a_R_i_D Авторизация с помощью социальных сетей 13 дней назад

Жаль. Хостер принудительно просит всех перейти на PHP 8, а менять хостера не хочется, больше 10 лет с ними уже работаю. Есть ли планы по переходу на PHP 8?

0
Zau4man Zau4man Авторизация с помощью социальных сетей 13 дней назад

Здравствуйте, нет. Только 7.2 — 7.4. Для работы нужен ioncube

0
Zau4man Zau4man Панель для мобильных 13 дней назад

Да, компонент продается с открытым исходным кодом, ioncube не нужен, версия php любая (в рамках icms сейчас 7.0+)

Используя этот сайт, вы соглашаетесь с тем, что мы используем файлы cookie.