Dayver |
|
---|---|
И я согласен с делением на два .... Ratibor может займешся таким разделением? Тем более больше чем половину труда ты уже выполнил
Pavlo Tkachenko aka Dayver
|
|
This post was edited by Dayver (2009-02-12 01:02, 15 years ago) |
Ratibor |
|
---|---|
# Dayver : Ratibor может займешся таким разделением? Тем более больше чем половину труда ты уже выполнилНе вижу смысла в таком разделении. Что они будут делать по отдельности ? Выводить теже теги, но с разной сортировкой ? Дак сейчас я зделал, хочешь сортируй, хочешь нет. Так же хочешь цвета и шрифт, пропиши их в css, не надо цвета или шрифт, убери из css. Или я чтото не допонимаю ? Не задавай глупых вопросов, не услышишь вранья.
|
Sergeich |
|
---|---|
Плагин "Теги" - занимается добавлением и выводом тегов в статьях и форуме, а также поиском по тегам.
Плагин "Облако тегов" - занимается исключительно формированием облака тегов и его визуальными преобразованиями. Я могу вообще не устанавливать облако тегов, при этом активно использовать сами теги. |
Ratibor |
|
---|---|
# Sergeich : Я могу вообще не устанавливать облако тегов, при этом активно использовать сами теги.А в чем проблемма то ? Ну не вставляй тэг вывода тэгов в tpl. Не задавай глупых вопросов, не услышишь вранья.
|
Sergeich |
|
---|---|
А если тег не вставлен в шаблон, то пхп-код плагина не отрабатывает в холостую? Я не в курсе, просто интересно. Помнится была с рейтингами в седитио похожая ситуация, убирание тегов из скина вовсе не отменяло обработку пхп-кода, просто результат не выводился.
|
Ratibor |
|
---|---|
# Sergeich : А если тег не вставлен в шаблон, то пхп-код плагина не отрабатывает в холостую? Я не в курсе, просто интересно. Помнится была с рейтингами в седитио похожая ситуация, убирание тегов из скина вовсе не отменяло обработку пхп-кода, просто результат не выводился.Дак вот и надо решать эту проблемму, если нет тэга, то и нефиг обрабатывать. А так давай будем тупо все разделять, тем более то, что разделять не требуется. У меня к примеру куча тэгов не вставлена на главной, но я же тут шашкой не махаю, давайте типа будем разделять каждую фигню. Если уж решать, то решать глобально. В крайнем случае просто можно удалить tags.index.php Также еще можно внести один переключатель в настройки плагина. Сейчас в tags.index.php используется: if($cfg['plugin']['tags']['pages'])заменить его на: if($cfg['plugin']['tags']['index']) а, в конфиг внести переключатель index P.S. Так что не вижу причины для разделения. Не задавай глупых вопросов, не услышишь вранья.
|
|
This post was edited by Ratibor (2009-02-12 03:59, 15 years ago) |
Trustmaster |
|
---|---|
Объясняю, зачем нужно разделение. Базовая часть тэгов (все, кроме собственно облаков) по большей части всех устраивает, и будет дальше развиваться в направлении последовательного улучшения. А вот с облаками есть много вариантов.
В качестве примера берем tags_new. Что в нем здорово:
Что не здорово в варианте на jquery.tagcloud:
Несколько моментов (вроде того же index и нескольких вариантов сортировки, влючая случайный) необходимо реализовать во всех вариантах плагина. Но все же мы видим 2 разных взгляда на рендеринг этих облаков. И это далеко не предел, ведь кто-то может захотеть написать, к примеру, отображение трехмерных облаков на флеше. А кто-то еще что-то эдакое. Каждый раз клонировать весь плагин tags - это приведет к большой неразберихе с версиями. Неплохой вариант сделать универсальные облака, чтоб и на жабе работал и без нее. Но желающим сделать что-то свое придется эти части отключать (да, многие админы до сих пор не знают, что кнопочка Pause нужна для отключения частей плагина от хуков, чтобы не выполнялся ненужный код). В общем, вот вам еще пара дней на размышления, господа. И заранее попрошу без лишнего флейма (особенно по поводу Флеша или чего-то в этом роде). May the Source be with you!
|
Ratibor |
|
---|---|
# Trustmaster : Объясняю, зачем нужно разделение. Базовая часть тэгов (все, кроме собственно облаков) по большей части всех устраивает, и будет дальше развиваться в направлении последовательного улучшения. А вот с облаками есть много вариантов. Базовая часть - это какая ? Если выкинуть вывод облаков, то что останется ? А останутся только: tags.forums.delete.php tags.forums.editpost.php tags.forums.editpost.tags.php tags.forums.newtopic.php tags.forums.newtopic.tags.php tags.forums.topics.php tags.page.php tags.page.add.php tags.page.add.tags.php tags.page.delete.php tags.page.edit.php tags.page.edit.tags.php tags.setup.php А выкинуть придется: tags.php tags.forums.php tags.index.php tags.list.php Выкосить функцию sed_tag_cloud из functions.php Папку js Ну и покромсать папки lang и tpl Тогда нужно не просто разделять на два плагина, а тогда базовую часть вносить в ядро, а иначе нет никакого смысла делать ее отдельным плагином, т.к. она уже и так частично туда интегрирована. Вот на это я скрипя сердцем согласен. P.S. И думаю надо выкосить из движка транслит как класс. Звучит конечно глупо, если учесть что я его и предлагал ввести Но тогда мы не знали что можно получить нормально читаемые не латинские ссылки. Не задавай глупых вопросов, не услышишь вранья.
|
|
This post was edited by Ratibor (2009-02-13 07:33, 15 years ago) |
dervan |
|
---|---|
# Ratibor : P.S. И думаю надо выкосить из движка транслит как класс.Транслит нужен для преобразования имён файлов, которые посетители загружают на хост. |
Ratibor |
|
---|---|
# dervan : Транслит нужен для преобразования имён файлов, которые посетители загружают на хост.Сейчас транслит сделан только для русского языка,что не есть гуд. Это как минимум эгоизмом попахивает.Если его делать, то делать типа апи для всех языков, примерно как мы зделали с числительными окончаниями. P.S. Хотя я реализацию транслита и не смотрел, может он и зделан как апи. Если даже и так, то в любом случае, он преобразует только из русских имен. В общем еще надо подумать хорошенько над этим. Не задавай глупых вопросов, не услышишь вранья.
|
dervan |
|
---|---|
Ratibor, да уже отлично всё сделано - я недавно посмотрел. :) И API есть, см. функцищу sed_safename(), и добавить для любого языка просто - достаточно поместить в lang-файл массив $sed_translit, и сразу будет работать. Сейчас $sed_translit уже есть в русском и украинском lang-файлах.
|
Trustmaster |
|
---|---|
Тогда есть еще предложение соединить два облака в одно. То есть сделать опцию выбора рендеринга облака: серверное и клиентское. И внести все те изменения, о которых говорили ранее.
May the Source be with you!
|
Ratibor |
|
---|---|
# Trustmaster : Тогда есть еще предложение соединить два облака в одно. То есть сделать опцию выбора рендеринга облака: серверное и клиентское.Сейчас переделываю, позже выложу, посмотришь. Суть в том что добавил к своему варианту в настройки тип сортировки RAND(), также поправил лимиты, они у тебя на главной не задействованы, ну и возможность сортировки на клиенте. Но в любом случае придется отказаться от jquery.tagcloud.js, т.к. он глючит неимоверно. Не задавай глупых вопросов, не услышишь вранья.
|
Trustmaster |
|
---|---|
Примеры глюков?
May the Source be with you!
|
Ratibor |
|
---|---|