Форумы / National / Russian / Модули и плагины / Нет тегов

Sergeich
#1 25.01.2009 16:42
Плагин "Tags", в случае отсутствия тегов выводит сообщение
Нет тегов
На мой взгляд, это избыточная информация. В листах и страницах это не очень заметно, т.к. блок тегов всего один. На форуме же это сразу заметно - откройте список тем и насладитесь этим "нет тегов" под каждой темой. Скажу больше, ситуация сохранится и на обычном форуме, никто себя не будет утруждать вбивать ещё и теги какие-то.

Для начала предлагаю убрать показ тегов из списка тем. В перспективе - обучить плагин, в случае отсутствия тегов, не выводить свой блок вообще.
Trustmaster
#2 25.01.2009 17:27
Предлагаю убрать тэги совсем и забыть это слово :-)

Надпись "Нет тэгов" - не моя идея. Но согласен, что нетрудно организовать блок и не выводить его в случае отсутствия тэгов. Делается это неутрудно, так что оформляем тикет, и уже сегодня он будет закрыт.
May the Source be with you!
Sergeich
#3 25.01.2009 17:39
Создал тикет, надеюсь поймёт народ, что я там написал :)
esclkm
#4 25.01.2009 17:39
уж лучше их тогда автоматом генерировать) из 3-5 самых употребляемых слов длиннее 4 символов. хотя наверное это сделать оч тяюело

А если по сабжу то нет тегов ПРАВИЛЬНО что выводдиться! (хотя если сделать опцией вкл выкл то еще правильнее) когда написано нет тегов - это какаято небольшая стимуляция пользователя их создавать
littledev.ru - мой маленький зарождающийся блог о котонти.
снижение стоимости программирования и снижение стоимости производства разные вещи. Первое можно скорее сравнить с раздачей работникам дешевых инструментов, чем со снижением зарплаты
Sergeich
#5 25.01.2009 18:52
Нету никакой стимуляции, поверь :). Теги категорически нельзя автоматом генерировать, в итоге получится страшная неюзабельная каша.
Ratibor
#6 25.01.2009 19:07
Фигней вы господа занимаетесь :-)
Есть тэги, нет теги, главное сейчас чтоб они работали,
а проблемм поважней сейчас хватает,
например глюк с расширенными темплейтами.

P.S. Согласен с esclkm если переделывать, то лучше зделать настройку в админке и пусть админ сам выбирает нужна ему эта надпись или нет.

P.P.S. Так же со времен LDU тянется баг с кнопкой "Новая тема".
Зачем ее выводить для гостей, если им все равно не разрешены посты на форуме ?
Не задавай глупых вопросов, не услышишь вранья.
Отредактировано: Ratibor (25.01.2009 19:21, 15 лет назад)
Sergeich
#7 23.03.2009 08:12
Продолжаем доводить теги до ума :). Возникла маленькая проблемка: В шаблоне page.tpl мне нужно разделять теги запятой (или любым другим разделителем), сейчас я это реализовал так:
<!-- BEGIN: PAGE_TAGS_ROW -->
	<a href="{PAGE_TAGS_ROW_URL}">{PAGE_TAGS_ROW_TAG}</a>,   
<!-- END: PAGE_TAGS_ROW -->
Проблема в том, что после последнего тега также вставляется запятая (разделитель), что есть не красиво. Как бы убрать эту лишнюю запятую?
Отредактировано: Sergeich (23.03.2009 08:25, 15 лет назад)
Kort
#8 23.03.2009 17:14
Это к кодерам. Мне, кстати, тоже не нравится отсутствие разделителей о облаке.
Btw, не забудьте про тикет переводчикам если что-то будете выкидывать/менять.
SED.by - создание сайтов, разработка плагинов и тем для Котонти
Trustmaster
#9 23.03.2009 18:45
Не было изящного решения сей проблемы на тот момент. С IF-ами можно будет решить, думаю.
May the Source be with you!
NovoKain
#10 23.03.2009 18:50
А не проще ли просто не выводить «Нет тегов» в ланг файле затереть это? А далее по умолчанию если тегов в форумс.постс нет то пусть будет пустая строка.

Кнопка новая тема для гостей это не баг вовсе. А мотивация зарегистрироватся.

Оффтопик: Я думаю что стоит посмотреть как реализованны те или иные вещи в других скриптах а не придумывать велосипед. Это адресованно в первую очередь тем кто считает что в Седе просто мегакрутой и очень функциональный форум.
dervan
#11 23.03.2009 19:57
# Trustmaster : С IF-ами можно будет решить, думаю.
Конечно, с IF'ом такое сделать можно (смотри код в plugins/tags/tags.page.php):
		<!-- BEGIN: PAGE_TAGS_ROW -->
		<a href="{PAGE_TAGS_ROW_URL}">{PAGE_TAGS_ROW_TAG}</a><!-- IF $GLOBALS['tag'] != $GLOBALS['tags'][count($GLOBALS['tags']) - 1] -->,<!-- ENDIF -->
		<!-- END: PAGE_TAGS_ROW -->
Только вот есть сомнения, правильно ли это - раз уж убирать HTML из кода, IMHO тогда не следует злоупотреблять кодом в шаблонах. А тут при каждом проходе парсера будет вызываться функцища count(). По-хорошему, в plugins/tags/tags.page.php надо добавить флажок, чтобы IF определял по нему последний проход парсера в блоке PAGE_TAGS_ROW.