Домой Wp-Recall WP-Recall: Предложения и идеи Что меня удивило в реализации плагина
8 ответ(ов) в теме
Kirill Zzz
не в сети 2 недели
На сайте с 27.03.2015
Участник
Тем 23
Сообщения 134
1
13:03
Версия WP-Recall:16.1.5

Здравствуйте, не могу понять почему? В плагине есть шаблоны для вывода разметки, но при этом разметка присутствует в данных. Почему??? Не разумнее вынести всю разметку в шаблоны? К примеру генерировать не весь _box целиком , а отдельно названия классов, содержимое тегов итд. Для css классов неплохо бы метод который возвращает родной класс и принимает параметром(ми)(arr) классы которые нужно добавить к нему + bool параметр оставлять ли родной класс или удалить. Или $Cart_Button->get_price(array('тег_обертки="пустая строка по умолчанию", тег_прайса="не передан по умолчанию"'), array('какие классы добавить') удалить_или_оставить_родной класс = false ) нет аргументов вывод по умолчанию

Редакции сообщения
03.06.2020 13:14Kirill ZzzПричина: не указано
03.06.2020 13:29Kirill ZzzПричина: не указано
03.06.2020 14:25Kirill ZzzПричина: не указано
03.06.2020 14:25Kirill ZzzПричина: не указано
03.06.2020 14:34Kirill ZzzПричина: не указано
03.06.2020 14:37Kirill ZzzПричина: не указано
03.06.2020 14:44Kirill ZzzПричина: не указано
03.06.2020 14:47Kirill ZzzПричина: не указано
03.06.2020 14:49Kirill ZzzПричина: не указано
0
Владимир (Otshelnik-Fm)
На сайте с 27.01.2013
Модератор
Тем 30
Сообщения 16954
2
14:54

Здравствуйте.

Разве возникает сложность стилизовать через css?

Некоторые элементы выведены в шаблоны:

Вы не можете просматривать опубликованные ссылки

Если что не хватает - предлагайте, добавим.

А так - вордпресс аналогично делает. Я не вижу критической проблемы. Есть фильтры, шаблоны, css - многое можно реализовать. Хуки можно отменить и реализовать так как вам нужно и повесить на этот же хук.
Подробней в статье

Вы не можете просматривать опубликованные ссылки
0
Kirill Zzz
не в сети 2 недели
На сайте с 27.03.2015
Участник
Тем 23
Сообщения 134
3
14:58

Я не говорил же о сложности стилизации, об архитектуре шла речь. И не критическую проблему я искал, а просто предложил. Если вам обратная связь не нужно, просто скажите я больше не буду писать всякую чушь) Спасибо! вот что то типа такого было бы очень гибко на мой взгляд $Cart_Button->get_price(array('тег_обертки="если не передан то по умолчанию", класс_обертки="если не передан то по умолчанию", удалить_или_оставить_родной класс = false'), array('тег_цены="если не передан то по умолчанию", класс_цены="если не передан то по умолчанию", удалить_или_оставить_родной класс = false')) нет аргументов вывод по умолчанию. Повторюсь если вам не нужна обратная связь и здоровая критика, я могу ми чем то другим заняться)))

Редакции сообщения
03.06.2020 15:01Kirill ZzzПричина: не указано
03.06.2020 15:03Kirill ZzzПричина: не указано
03.06.2020 15:03Kirill ZzzПричина: не указано
0
Владимир (Otshelnik-Fm)
На сайте с 27.01.2013
Модератор
Тем 30
Сообщения 16954
4
15:07

Kirill Zzz сказал(а)
$Cart_Button->get_price(array('тег_обертки="если не передан то по умолчанию", класс_обертки="если не передан то по умолчанию", удалить_или_оставить_родной класс = false'), array('тег_цены="если не передан то по умолчанию", класс_цены="если не передан то по умолчанию", удалить_или_оставить_родной класс = false'))

а кто из плагинов так делает?

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

0
Kirill Zzz
не в сети 2 недели
На сайте с 27.03.2015
Участник
Тем 23
Сообщения 134
5
15:20

У вас css классы и теги разметки захардкожены в классах. У вас есть шаблоны в которых используется вывод. Юзер например юзает css фреймворк и хочет использовать встроенные в него классы для стилизации. Создал свой шаблон внес нужные классы и вуаля. Я изначально вел речь о возможности кастомизации встроенных шаблонов а не о плагинах, в плагинах само собой фильтры уже как бы предусмотрены в самом wp. И да с практической точки зрения оно именно мне не надо, просто писал некоторое расширение для когда то давно сделанного магазина клиента увидел этот момент и решил поделиться. Ничего обосновывать я не хочу, не нужно я не настаиваю. Всего хорошего!

0
Андрей CS
На сайте с 30.11.-0001
Администратор
Тем 71
Сообщения 14380
6
15:22

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

0
Kirill Zzz
не в сети 2 недели
На сайте с 27.03.2015
Участник
Тем 23
Сообщения 134
7
15:32

Конкретное предложение скорее всего можно внедрить без угрозы сломать совместимость. Просто добавить новые методы в классах не убирая старые. Хотите так пользуйте а хотите так. Мне плагин нравиться в целом, по сравнению с другими магазинами на WP он гораздо легче и быстрее, во многом проще, поэтому как бы вот захотелось поучаствовать в развитии со своим предложением. Спасибо что выслушали. Желаю успехов в развитии!

Редакции сообщения
03.06.2020 15:32Kirill ZzzПричина: не указано
0
Андрей CS
На сайте с 30.11.-0001
Администратор
Тем 71
Сообщения 14380
8
15:37

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

Вы не можете просматривать опубликованные ссылки

Будем с нетерпением ожидать ваших предложений там.

0
Вы не имеете права на публикацию сообщений в этой теме