Домой Каталог: Дополнения WP-Recall PrimeForum Изменения вывода иконок в дефолтном шаблоне
7ответ(ов) в теме
Андрей CS
не в сети 2 часа
На сайте с 30.11.-0001
Администратор
Тем 71
Сообщения 16855
1
11:50
Версия WP-Recall: 16.8.7
Версия дополнения: 1.4.0

Заметил большой запрос на возможность изменения иконок в списке групп, форумов и тем, поэтому решил внести изменения в дефолтный шаблон и сделать вывод иконок в указанных списках более гибким. Для решения этого вопроса уже было выпущено несколько дополнений, поэтому решил сначала вынести на обсуждение данное решение.
Итак, предполагаю добавление следующих функций:

function pfm_get_icon($icon_class = 'fa-folder'){
    return apply_filters('pfm_icon', '<i class="fa '.$icon_class.'" aria-hidden="true"></i>');
}

function pfm_the_icon($icon_class = 'fa-folder'){
    echo pfm_get_icon($icon_class);
}

соответственно, в самих файлах шаблона, например групп, использовать функцию следующим образом:

<div class="prime-forum-icon">
    <?php pfm_the_icon('fa-database'); ?>
</div>

При необходимости кастомного изменения иконки или замены на изображение использовать фильтр 'pfm_icon', а данные текущего форума, группы или темы получать из глобальных объектов $PrimeGroup, $PrimeForum и $PrimeTopic.
Хотелось бы услышать насколько необходима и удобна такая реализация.

Редакции сообщения
21.11.2017 11:53Андрей CSПричина: не указано
1
Вова (Otshelnik-Fm)
не в сети 2 дня
На сайте с 27.01.2013
Участник
Тем 43
Сообщения 18647
2
12:00

Я просто бы переопределил стилями если очень надо

0
Андрей CS
не в сети 2 часа
На сайте с 30.11.-0001
Администратор
Тем 71
Сообщения 16855
3
12:03

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

1
Андрей CS
не в сети 2 часа
На сайте с 30.11.-0001
Администратор
Тем 71
Сообщения 16855
4
12:05

например так, в списке тем, можно будет вывести аватарки их авторов, ну и этот функционал можно будет использовать в любом другом шаблоне форума

Редакции сообщения
21.11.2017 12:09Андрей CSПричина: не указано
1
Вова (Otshelnik-Fm)
не в сети 2 дня
На сайте с 27.01.2013
Участник
Тем 43
Сообщения 18647
5
12:20

Да - если нужна динамика или отдавать это как доп - то мое переопределение стилями не сработает.

Поэтому - почему бы и не внедрить?

Я работал с этими глобальными переменными - они много полезного внутри содержат

1
garry
не в сети 3 часа
На сайте с 04.01.2014
Участник
Тем 146
Сообщения 2925
6
12:22

Ага, вы сер сильно упростите задачу этим))
По крайней мере мне так говорит не проснувшийся мозг.
Просто выводить свою настройку без лишнего гемора, я только за.

1
Андрей CS
не в сети 2 часа
На сайте с 30.11.-0001
Администратор
Тем 71
Сообщения 16855
7
12:33

ок, если не будет больше предложений по изменению, то включу это в след. обновление в таком виде

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