Показать по условию скрытые "QR by Татьяна5" заголовки сообщений в темах

Обсуждение новой системы расширения функциональности phpBB 4.0-dev, разработки новых расширений.
Аватара пользователя
Alecto
Сообщения: 571
Зарегистрирован: 05 авг 2014, 21:41

Показать по условию скрытые "QR by Татьяна5" заголовки сообщений в темах

Сообщение Alecto »

расширение не оформлял.

- скрипт работает только с БО Татьяны, только с отключенными заголовками!
- проверяется совпадение заголовка сообщения и темы, если не совпадают - показываем.
- заголовки первых сообщений показываются всегда - их не трогаем.
- если заголовок был пустой - пропускаем и оставляем скрытым.
- если в любом встреченном заголовку попадается RE: - убираем его и далее проверяем на равенство заголовку темы.
- для того, чтобы избежать "наплывов" кнопок на изображения (смайлы и картинки в тексте) в сообщениях с отключенными заголовками, увеличиваем отступ сверху.
- в сообщениях создаются кнопки "Скопировать ссылку на сообщение".

Создаем кнопку "Скопировать ссылку на сообщение"

Создаем файл viewtopic_body_postrow_post_content_footer.html

Код: Выделить всё

	<!-- IF S_USER_LOGGED_IN -->
<a id="directPostLink_p{postrow.POST_ID}" href="{postrow.U_MINI_POST}" onclick="prompt('Ссылка на сообщение #{postrow.POST_ID}',this.href); return false;" style="background-color: #fff; border: 1px solid #ccc; border-radius: 4px; display: none; padding: 0 4px; position: absolute; top: -14px; z-index: 1;" title="Скопировать ссылку на сообщение #{postrow.POST_ID}">#{postrow.POST_ID}</a>
	<!-- ENDIF -->
Переопределяем отображение кнопок для сообщений в собственном CSS

Код: Выделить всё

.postbody .post-buttons { display: none; margin-top:-4px; }
Подключаем скрипт коррекции заголовка + кнопок копирования ссылок

В файле overall_footer_after.html

Код: Выделить всё

<script type="text/javascript">
// <![CDATA[

	<!-- IF S_VIEWTOPIC -->

// обработка заголовков сообщений
	$( document ).ready(function() {
		var topic_title = $('.topic-title a').text();

		$('.post .postbody div h3').each(function() {
			var post_title = $(this).text();
			var n = post_title.search('Re: ');
				if ( n == 0 ) {
					post_title = post_title.replace('Re: ', '');
					$(this).text(post_title);
				}

	 		if (!$(this).hasClass('first') &&  post_title != topic_title && post_title != '') {
					$(this).removeAttr('style');
					$(this).next().next().removeAttr('style');
				} else if ($(this).css('display') == 'none') {
					$(this).next().next().css({'padding-top': '5px'});
				}
		});


	});

// показываем-скрываем кнопки
		<!-- IF S_USER_LOGGED_IN -->
		$('.post.has-profile').hover(function() {
			$('#directPostLink_' + $(this).attr('id')).show();
			$(this).find('.post-buttons').show();
		}, function() {
			$('#directPostLink_' + $(this).attr('id')).hide();
			$(this).find('.post-buttons').hide(); 
		});
		<!-- ENDIF -->


	<!-- ENDIF -->

// ]]>
</script>
Последний раз редактировалось Alecto 29 сен 2014, 13:55, всего редактировалось 11 раз.


Аватара пользователя
Alecto
Сообщения: 571
Зарегистрирован: 05 авг 2014, 21:41

[ЗАПРОС] Скрыть заголовки сообщений в темах и поиске

Сообщение Alecto »

скрывать? или условие к скрытию?

Отправлено спустя 1 минуту 25 секунд:
хотя без разницы.
встречный вопрос: какой смысл в заголовках сообщений на страницах тем с RE или без, дублирующих название темы 10 раз на каждой странице?

Аватара пользователя
владимир1983
Сообщения: 164
Зарегистрирован: 21 янв 2014, 21:44
Контактная информация:

[ЗАПРОС] Скрыть заголовки сообщений в темах и поиске

Сообщение владимир1983 »

Например для логики оптимизации.
Конструкция вида seo валидна.

Код: Выделить всё

<article>
<h3>Заголовок поста</h3>
<aside>Кнопки, время, профиль и прочая лабуда</aside>
<div class="content">Содержание поста<div>
</article>
Без заголовка будет ошибка логики.

Код: Выделить всё

<article>
<aside>Кнопки, время и прочая лабуда</aside>
<div class="content">Содержание поста<div>
</article>

Аватара пользователя
Alecto
Сообщения: 571
Зарегистрирован: 05 авг 2014, 21:41

[ЗАПРОС] Скрыть заголовки сообщений в темах и поиске

Сообщение Alecto »

а чем нынешняя реализация невалидна?
вот как работает скрытие БО сейчас.

Код: Выделить всё

<div id="post_content74406">
<h3 style="display: none;">
<a href="#p74406">Фотоконкурс</a>
</h3>
...
все то же самое, конструкция валидна, только не мусорит страницу.
замечательно работает, за исключением того, что в таком виде вместе с мусорными автоматическими заголовками скрываются и полезные заголовки, которые могут быть введены вручную и нести смысловую нагрузку.

Аватара пользователя
владимир1983
Сообщения: 164
Зарегистрирован: 21 янв 2014, 21:44
Контактная информация:

[ЗАПРОС] Скрыть заголовки сообщений в темах и поиске

Сообщение владимир1983 »

Я не говорю что нынешняя разметка не валидна.
Я хочу верить что phpbb будет двигаться в ногу со временем и все больше и больше будет внедрятся в код структура и логика HTML5 и микроразметки.
Alecto писал(а):QR_BBPOST не мусорит страницу
Не считаю заголовки сообщения лишними. Мнения могут быть разными, но у тебя специфический запрос, и я не думаю что требуемое тебе расширение будет востребовано. Хочешь убрать заголовки, есть опция в расширении БО. Показывать или не показывать заголовки по каким либо условиям это ИМХО лишнее.

Аватара пользователя
Alecto
Сообщения: 571
Зарегистрирован: 05 авг 2014, 21:41

[ЗАПРОС] Скрыть заголовки сообщений в темах и поиске

Сообщение Alecto »

владимир1983 писал(а):QR_BBPOST есть опция в расширении БО
абсолютно бессмысленная опция в такой реализации.
владимир1983 писал(а):QR_BBPOST по каким либо условиям
не "по каким либо", а по вполне логичным, обоснованным и универсальным - эти условия подойдут на любом форуме, если админ включит голову.
и на оборот, без них скрытие, реализованное в БО - скорее досадное недоразумение.
посмотрите такую тему http://motobratva.com/viewtopic.php?f=21&t=701 или такую http://motobratva.com/viewtopic.php?f=6&t=703
на гуру, да и на любом другом форуме найдете что-то аналогичное.
в таких темах НЕЛЬЗЯ тупо скрывать заголовки.

а теперь посмотрите сюда http://www.phpbbguru.net/community/topic16034.html
нет заголовков, потом они есть - да пофигу на них всем, потому что это мусор.

Аватара пользователя
владимир1983
Сообщения: 164
Зарегистрирован: 21 янв 2014, 21:44
Контактная информация:

[ЗАПРОС] Скрыть заголовки сообщений в темах и поиске

Сообщение владимир1983 »

Я же тебе говорю у тебя индивидуальные потребности в данном расширении. Ну почему ты не хочешь признать что данный функционал нужен по большому счету тебе и очень ограниченному количеству людей? В своем подавляющем большинстве на форумах не требуется менять заголовки.
Тебе это нужно для удобного поиска как я понимаю, верно?
Я бы на твоем месте сходил в рекрутинг и там решил эту проблему.

Отправлено спустя 1 минуту 34 секунды:
Alecto писал(а):QR_BBPOST нет заголовков, потом они есть
Это проблема старого seo мода и переноса сообщений из тему в тему. Это не юзеры удаляли заголовки.

Аватара пользователя
Alecto
Сообщения: 571
Зарегистрирован: 05 авг 2014, 21:41

[ЗАПРОС] Скрыть заголовки сообщений в темах и поиске

Сообщение Alecto »

владимир1983 писал(а):QR_BBPOST Тебе это нужно для удобного поиска как я понимаю, верно?
я очень редко пользуюсь поиском, хотя если приходится, то ДА - искать намного удобнее.
в данном вопросе меня больше заботит оптимизация отображаемой информации.
владимир1983 писал(а):QR_BBPOST Ну почему ты не хочешь признать что данный функционал нужен по большому счету тебе
на счет "только" - поспорю, а то что нужен - признаю.
встречный вопрос: почему, если "этот функционал нужен только мне одному" - мои аргументы были услышаны и скрытие заголовков реализовано в БО? почему, когда я указал на отсутствие логики в работе данной функции - вы отсылаете меня в рекрутинг, вместо того, чтобы принять аргументы и сделать работу корректной в самом расширении?
начинали мы с обсуждения с вопроса: "как удалить заголовки", после чего пришли к тому, что удалять - не совсем красиво, а скрывать - вполне себе хорошо.
вот только оказалось, что: удалять заголовки по-умолчанию, но предоставлять возможность их вводить - это не одно и то же, что скрывать их безразбору.
вы ведь здесь ловите баги, ошибки, обсуждаете полезные советы?
я вам указал на ошибку, некорретную работу скрытия.
если вы считаете, что "нужен по большому счету тебе" - о чем тогда спор? "очень ограниченному количеству людей" - удивитесь, насколько "ограниченное число" окажется большим.
в любом случае, с таким подходом - давайте наплюем на все ошибки, баги и оптимизацию кода, ведь 99% этого всего либо не найдут, либо "и так сойдет".
владимир1983 писал(а):QR_BBPOST Я бы на твоем месте сходил в рекрутинг и там решил эту проблему.
спасибо за совет, но я подожду единомышленников.

Отправлено спустя 5 минут 1 секунду:
владимир1983 писал(а):QR_BBPOST данный функционал нужен по большому счету тебе
http://www.phpbbguru.net/community/topi ... ml#p205597
http://www.phpbbguru.net/community/topi ... ml#p227404
http://www.phpbbguru.net/community/topi ... ml#p243130

дальше лениво искать, на гуру наковыряете сотни или тысячи таким постов, но ... нужен ИСКЛЮЧИТЕЛЬНО мне одному

и, кстати, если бы в той же теме о пользовательских ббкодах использовались заголовки в смысловых сообщениях, например, в сообщениях с работающими кодами, а остальные бы заголовки скрывались - насколько бы тема была читабельна, сколько бы тупых постов "а где?", "не могу найти", на которые постоянно отвечает Пчелкин - отпали бы сами собой, но... такой функционал нужен ИСКЛЮЧИТЕЛЬНО мне одному, потому имеем как в анекдоте "мыши плакали кололись но продолжали жрать кактус".

Отправлено спустя 1 минуту 25 секунд:
а еще есть вот такой мод http://www.phpbbguru.net/community/topic22964.html, который, надеюсь будет портирован либо появится его аналог для 3.1, который очень даже работает с заголовками сообщений, но ... он тоже нужен ИСКЛЮЧИТЕЛЬНО мне одному
Последний раз редактировалось Alecto 03 сен 2014, 18:31, всего редактировалось 1 раз.

Аватара пользователя
владимир1983
Сообщения: 164
Зарегистрирован: 21 янв 2014, 21:44
Контактная информация:

[ЗАПРОС] Скрыть заголовки сообщений в темах и поиске

Сообщение владимир1983 »

Alecto писал(а):QR_BBPOST не совсем красиво, а скрывать - вполне себе хорошо.
По заголовкам осуществляется поиск. Поэтому нельзя удалять заголовки не кастрировав возможности поиска по форуму. Поэтому только скрывать.
Alecto писал(а):QR_BBPOST наковыряете сотни или тысячи таким постов... нужен ИСКЛЮЧИТЕЛЬНО мне
Исключительно тебе. Те ссылки что ты привел в качестве примера когда юзеры изменяют заголовок примерами не являются. Эти заголовки отличны от заголовка темы только потому что тема была объединена или же в неё были перенесены сообщения из другой темы.
На гуру командой ежедневно выполняются десятки а иногда сотни правок сообщений, объединение и удаление тем. Уверен что 99.9% отличающихся заголовков сообщения от заголовка темы это последствие модераторских действий с темой.

Ответить