icms2 icon indicating copy to clipboard operation
icms2 copied to clipboard

Доработать работу типографа.

Open maxisoft-git opened this issue 7 years ago • 12 comments

Используемая версия InstantCMS: 2.10.x

Версия PHP: 7.1

Как воспроизвести проблему:

Создать контент с полем html и обработкой типографом

Воспроизводится ли проблема на демо http://demo.instantcms.ru/: ДА

При оформлении текстов, используются теги span, при обработке типографом возможно использовать только атрибут style что не хорошо, так как стилей может быть много, логичнее добавить разрешенный тег id, class и data-* атрибуты.

maxisoft-git avatar Sep 12 '18 09:09 maxisoft-git

обавить разрешенный тег id, class и data-* атрибуты

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

fuzegit avatar Sep 16 '18 19:09 fuzegit

сделать отключение типографа для определенных групп или как то иначе.

не получится простой пример hook который просто подсвечивает в тексте слова, он проверяет текст на слова, записывает в базу при выводе в поле стоит типограф, если сделать исключение тогда при выводе у админа будет выводить а вот для юзера нет. тут как раз речь не в редакторе а типографе, он не как не влияет на работу юзеров.

maxisoft-git avatar Sep 17 '18 06:09 maxisoft-git

Права сделать на сам типограф, а не на вызов хука html_filter.

fuzegit avatar Sep 17 '18 08:09 fuzegit

Как вариант можно и так, главное чтобы можно было настроить эти права.

maxisoft-git avatar Sep 17 '18 10:09 maxisoft-git

Поддерживаю введение прав на типограф. Иногда типограф приходится выключать, например, для встраивания видео с хостинга, не упомянутого в типографе, или для вставки записи из вконтакте и т.п.

zau4man avatar Oct 10 '18 10:10 zau4man

Поддерживаю

bilarius avatar Nov 15 '18 20:11 bilarius

Что касается разрешенных тегов в типографе и прочего разрешенного, где все что не разрешено отсеивается - добавьте пожалуйста hook'и чтобы можно было добавлять свои "разрешения" не исправляя код системы. Это полезно для использования типографа в своих компонентах где нужно добавить прочие специфические теги.

san-sanych-k avatar Nov 22 '18 11:11 san-sanych-k

Я за возможность добавления своих тегов и возможность изменения параметров тегов.

geminisf avatar Feb 18 '19 11:02 geminisf

В типограф нужно добавить проверку опции "Проверять HTTP referer" компонента Редиректы. И в зависимости от опции формировать атрибуты внешних ссылок в строке https://github.com/instantsoft/icms2/blob/2c94377fc57a52b799186579854264feeae46d28/system/controllers/typograph/hooks/html_filter.php#L165 (версия 2.12.1). Сейчас в 2.12.1 при включенной опции и формировании внешних ссылок через редирект переход по ссылке ведет на 404, так как браузер не передает реферер.

zau4man avatar Apr 01 '19 14:04 zau4man

Это вы наверно не мне отправляли))) 

bilarius avatar Apr 01 '19 15:04 bilarius

@bilarius вам на почту приходит рассылка - уведомление от комментариев. Всё что вы отвечаете в почте, добавляется в комментарии на GitHub.

fuzegit avatar Apr 01 '19 15:04 fuzegit

Небольшая проблема с заменой тире на дефис в типографе решена, но, хотелось бы иметь такую возможность в настраиваемых опциях типографа. Спасибо! )

dvduck avatar Jul 13 '22 08:07 dvduck