#41278 Macik:

 

В Котонти существовал свой собственный набор смайов (в `images\smilies\`), со своим собственным файлом описания на JS

Я помню эти конфиги, как раз когда начинал своё знакомство с Cotonti, пользовался Markitup , и работал с конфигами смайлов.

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

Если бы решение зависело исключительно от меня, то я бы смотрел в сторону глобального изменения.

1. Папку смайлов положил бы непосредственно в корень CKEditor.
2. Удалил бы дефолтный плагин смайлов скедитора.
3. Использовал бы для вставки смайлов в редактор API самого CKEditor.(например так). Уже давно использую этот метод для вставки рекламы в контент с помощью myads плагина.
4. Удалил бы Markitup из сборки, вместо этого добавил бы CKEditor + BBcode (например такой).

Парсинг BBcode по-прежнему происходил бы через плагин bbcode, мне он очень нравится.

В CKEditor есть много интересных возможностей при нестандартном подходе. Например у меня есть наработка в которой подсчитывается налету слова символы без пробелов и сразу выдается цена по формуле 1000*N сбп, или есть наработка вставки микроразметки для рецептов по кнопке, с дальнейшей обработкой парсером. Думаю смайлы реализовать таким способом не проблема.

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

п1. Не лучший вариант. Вот почему — редакторы предполагаются в CMF, как полностью независимый модуль и могут быть заменены один на другой. Поэтому, все что качается конкретного редактора имеет смысл держать в его папке, а все, что не касается надо куда-то выносить. Пример: заменили редактор — и у нас похерились все смайлы на сайте. Не хорошо. 
Во-вторых, теоретически, смайлы это графический элемент на сайте, а следовательно должны/могут иметь отношение к теме оформаения сайта. Заставлять каждого разработчика скина клать внутрь смайлы под нее — немыслимо. Значит надо иметь дефолтный набор. Что примерно мы и имели для Markitup.

п2. Он, в общем случае, удобен (идет в комплекте, находится на панеле). Его можно легко програмно кастомизировать (ссылку давал выше), сделать стандартную «точку входа». При его использовании, не надо городить огород с прочими кнопками или иными решенями.

п3. Внешний элемент — не самый удобный вариант. Плюс тянет за собой доп.код и проблемы поддержки. Встроенный же вариант можно очень просто включать/отключать для заданного редактора.

п4. В этом есть резон. Однако надо спросить у адептов Merkitup.

Я помню эти конфиги, как раз когда начинал своё знакомство с Cotonti, пользовался Markitup , и работал с конфигами смайлов.

Позже я решил адаптировать под Cotonti визуальный BB редактор SCEditor. В нём реализовал смайлики настраиваемые в отдельной папке, чтобы отвязаться от набора в корне, были добавлены свои наборы через BBcode плагин при установке. Необходимость ручнго конфига таким образом отпадала.

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

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

Если бы решение зависело исключительно от меня, то я бы смотрел в сторону глобального изменения.

1. Папку смайлов положил бы непосредственно в корень CKEditor.
2. Удалил бы дефолтный плагин смайлов скедитора.
3. Использовал бы для вставки смайлов в редактор API самого CKEditor. Уже давно использую этот метод для вставки рекламы в контент с помощью myads плагина.
4. Удалил бы Markitup из сборки, вместо этого добавил бы CKEditor + BBcode (например такой).

Парсинг BBcode по-прежнему происходил бы через плагин bbcode, мне он очень нравится.

В CKEditor есть много интересных возможностей при нестандартном подходе. Например у меня есть наработка в которой подсчитывается налету слова символы без пробелов и сразу выдается цена по формуле 1000*N сбп, или есть наработка вставки микроразметки для рецептов по кнопке, с дальнейшей обработкой парсером. Думаю смайлы реализовать таким способом не проблема.

А что насчет правки #1404, по ней идеи уже есть ? 

я так понимаю, суть заявки отвязать конфиг смайлов CKEditor от самого плагина?

В Котонти существовал свой собственный набор смайов (в `images\smilies\`), со своим собственным файлом описания на JS, который использовался в редакторе Markitup, который раньше был основным редактором для Котонти (корнями это уходит в 2008-09 года).
Этот набор позже использовался так же плагином BBCode (при соотв. настройках). Здесь на сайте даже представлено несколько кастомных наборов.

Так вот, суть, думаю в том, чтобы привязать этот штатный набор к CKEditor. 

Теоретически наборы в CKeditor настраиваемые через соотв. переменные конфига. Надо просто расширить `ckeditor.default.set.js`, чтобы была возможность задать доп. настройки через свой JS файл или переменную.

Вопрос в том, на сколько это сегодня актуально. Расширить `ckeditor.default.set.js`, для возможности задавать свои настройки смысл вижу. Прописывать ли через этот механизм дефолтные смайлы — для меня вопрос открытый.

Правки влил. Спасибо за исправления.

Ок, видел. 

А что насчет правки #1404, по ней идеи уже есть ? 

я так понимаю, суть заявки отвязать конфиг смайлов CKEditor от самого плагина?

В целом согласен. Но надо немного доработать — прокомментировал здесь.

Ок, код приготовил, сейчас на гитхабе добавлю, новый вариант будет такой:

// meta title
$out['subtitle'] = empty($qs) ? $L['Tags'] : htmlspecialchars(strip_tags($qs)) . ' - ' . mb_strtolower($L['tags_Search_tags']);
// meta descriptions
$out['desc'] = empty($qs) ? $L['Tags'] : mb_strtolower($L['tags_Search_tags'] . ': ' . strip_tags($qs) . ' - ' . $L['tags_Search_results']);
// meta keywords
$out['keywords'] = empty($qs) ? $L['Tags'] : mb_strtolower($L['Tag'] . ' ' . strip_tags($qs) . ' ' . $L['tags_Search_tags']);

Добавлено 17 минут спустя:

Правки внёс

Нашёл, я добавил заяку на гитхаб:  #1458

В целом согласен. Но надо немного доработать — прокомментировал здесь.

Еще замечание — ты правильно сделал открыв тикет 1458 (и рано его закрыл).
Если в пул-реквесте писать что-то типа `Fix #1458` или `Closes #1458` то открытый ранее тикет будет автоматом закрыт при принятии пулреквеста.

Спасибо за оперативный тест. 
Вот скорректировал .

Попутно поправил еще один баг — «кривое» кеширование страниц облака тегов. Из-за чего некоторые могли дублироваться, а некоторые не отображались.

Просьба протестировать повторно (не забудьте обнулить кеш).

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

Добавлено 10 минут спустя:

=================

Отлично, всё работает теперь, и в режиме Дружественная паджинация и без.

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

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

Добавлено 10 минут спустя:

Нашёл, я добавил заяку на гитхаб:  #1458

Попутно поправил еще один баг — «кривое» кеширование страниц облака тегов. Из-за чего некоторые могли дублироваться, а некоторые не отображались.

Просьба протестировать повторно (не забудьте обнулить кеш).

Как выяснилось баг проявлялся только при включенном режиме `easypagenav`.
Исправлено в 2285300.

Прошу проверить и сообщить о результатах. По факту тестирования обновим здесь на сайте.

Протестировал на нескольких сайтах, проблема осталась. Пробовал и с включенными чпу и без, на рабочем сайте и на новом, чистом.

Если установить чистую базу, создать 4-5 тегов, и в админке поставить настройку:   Тегов на странице в облаке всех тегов   1  

При включении Дружественная паджинация:  просто переключает теги, но сама пагинация не меняет цифры, остается на первой.

При выключении Дружественная паджинация:  тоже не работает, но при малом количестве тегов еще и появляется бесконечная пагинация, 1..5..100...5000...

Если выбрать в настройках :   Тегов на странице в облаке всех тегов, 0 - все теги сразу:     вместо 0 другое число, например 100, то появляется пагинация на странице всех тегов, даже при переключении что-то меняется, но кнопка подсветки остается 1 постоянно, куда бы я не переключал, класс pagenav pagenav_current постоянно находится на кнопке под номером 1, при переходе на страницу тегов , например /tags?a=all&dt=2      сами теги меняются, но в пагинации это никак не отображается, плюс в мета тайтл не появляется ( страница 2 ) . 

На тестовом пробовал в одном из файлов заменить переменную $t на $dt в одном месте , тогда в мета тайтл появляется ( страница 2 ) и тд, но сама пагинация не меняет класс, и остается на уровне 1 постоянно.

Я попробую обновить сайт до мастер ветки, посмотрим что будет. 

Добавлено 1 час спустя:

Обновил сайт мастер веткой, проблема осталась

Добавлено 13 минут спустя:

Вот живой пример проблемы:

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

Второй пример:

Сайт с несколькими сотнями тегов, обновлен до мастер ветки, настройка выводить по 100, итог можно посмотреть здесь:

Заметил что не работает пагинация на странице всех тегов если ограничить количество   тегов на странице в облаке всех тегов

Остальные места работают, проблема наблюдается именно на странице всех тегов.

Roffun, можешь подробнее расписать (не могу повторить на посленей версии из мастер ветки). Какая версия Котонти и какая версия плагина.
Еще лучше если ты сравниш все свои файлы плагина с файлами в мастер-ветке (если сделать это сложно — запакуй и пришли мне).

У себя проверял так — создал пару страниц с тегами (всего около десятка) в настройках поставил  «Tags displayed per page in standalone cloud = 2». Как результат на странице тегов (/tags?a=all) выводится 2 первых и показывает несколько страниц. Пагинация тоже работает нормально.

Ошибка на вот этой версии файла https://github.com/Cotonti/Cotonti/blob/master/plugins/tags/tags.php наблюдается ?

Наблюдается. 

Остальные места работают, проблема наблюдается именно на странице всех тегов.

Кстати на cotonti.com тоже проблема с тегами, при нажатии на тег вообще ошибка fatal error выскакивает: //www.cotonti.com/tags?a=all&t=siena - если заменить в адресе переменную t на dt (как в пагинации указано) то ошибка исчезает.