Thanks for posts by Палыч

Обсуждение новой системы расширения функциональности phpBB 4.0-dev, разработки новых расширений.
Алг
Сообщения: 645
Зарегистрирован: 18 фев 2014, 17:28

Re: Опыты с экстами

Сообщение Алг »

а у вас видно кнопку? ( кеш чист, как слеза)
а может не будем писать аддон отдельным расширением, а добавим в это?

Отправлено спустя 0 часов 20 минут 37 секунд:
в ф12 кнопка показывалась с нулевыми размерами
изменила 2 посл строчки в thanks.css

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

.thanks-icon, .thanks-icon a	{ background-image: url("./images/thankposts.gif"); width: 20px; height: 20px; }
.removethanks-icon, .removethanks-icon a	{ background-image: url("./images/removethanks.gif");; width: 20px; height: 20px; }
теперь видно

Аватара пользователя
rxu
Сообщения: 1125
Зарегистрирован: 21 янв 2014, 21:20
Откуда: Krasnoyarsk
Github repo: https://github.com/rxu
Контактная информация:

Re: Опыты с экстами

Сообщение rxu »

Алг писал(а):а может не будем писать аддон отдельным расширением, а добавим в это?
Да можно и добавить, а что конкретно? Жду пулл реквест ;)
Алг писал(а):изменила 2 посл строчки в thanks.css
Спасибо, попробую. Хотя, есть мысль избавиться от графики, и сделать кнопку на css просто с плюсом, как на Гуру.

Алг
Сообщения: 645
Зарегистрирован: 18 фев 2014, 17:28

Re: Опыты с экстами

Сообщение Алг »

последний штрих, кнопка не нажимается
исправить в viewtopic_body_post_buttons_after.html всё, что стоит между условиями заменить на

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

    <li>
		<a href="{postrow.THANKS_LINK}" title="{postrow.THANK_ALT}{postrow.POST_AUTHOR}"><div  class="{postrow.THANKS_IMG}" >&nbsp;</div></a>
	</li>
rxu писал(а):Да можно и добавить, а что конкретно?
благодарность без перезагрузки страницы. У кого отключён js будет работать так, как сейчас, у остальных, как положено ;)
rxu писал(а):Спасибо, попробую. Хотя, есть мысль избавиться от графики, и сделать кнопку на css просто с плюсом, как на Гуру
а если кто картинкой хочет?

Отправлено спустя 0 часов 4 минуты 50 секунд:
Алг писал(а):последний штрих
это был предпоследний
последний( на данный момент), при установке расширения, у меня обнулились все существующие благодарности. Надо проверять, что если эти таблицы существуют, не "дропать " их

Отправлено спустя 0 часов 1 минуту 40 секунд:
и может это расширение будем в отдельной теме обсуждать? оно такое ж объемное и такое же популярное, как и быстрый ответ

Xisp
Сообщения: 155
Зарегистрирован: 02 фев 2014, 12:38

Re: Thanks for posts by Палыч

Сообщение Xisp »

Алг писал(а):а если кто картинкой хочет?
Тот возьмёт и изменит стили. Кнопки картинкой- прошлый век.

Алг
Сообщения: 645
Зарегистрирован: 18 фев 2014, 17:28

Re: Thanks for posts by Палыч

Сообщение Алг »

Xisp писал(а):Кнопки картинкой- прошлый век.
чегой-то? вон справа в каждом посте 6 кнопок с картинками(физически картинка-то одна, они применили css-спрайт технологию)

Xisp
Сообщения: 155
Зарегистрирован: 02 фев 2014, 12:38

Re: Thanks for posts by Палыч

Сообщение Xisp »

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

Алг
Сообщения: 645
Зарегистрирован: 18 фев 2014, 17:28

Re: Thanks for posts by Палыч

Сообщение Алг »

Xisp писал(а):Растеризованный иконочный шрифт это
icons_button.png
icons_button.png (5.56 КБ) 1385 просмотров
какая разница нарисовано ли векторно или по старинке? ;)
кстати, зачем нам две картинки
thankposts.gif
thankposts.gif (1.2 КБ) 1385 просмотров
removethanks.gif
removethanks.gif (1.2 КБ) 1385 просмотров
, лучше сделать одним спрайтом?

Аватара пользователя
rxu
Сообщения: 1125
Зарегистрирован: 21 янв 2014, 21:20
Откуда: Krasnoyarsk
Github repo: https://github.com/rxu
Контактная информация:

Re: Thanks for posts by Палыч

Сообщение rxu »

Алг писал(а):лучше сделать одним спрайтом?
Конечно, лучше. Надо только, чтобы кто-то сделал :)

Алг
Сообщения: 645
Зарегистрирован: 18 фев 2014, 17:28

Re: Thanks for posts by Палыч

Сообщение Алг »

да сейчас столько спрайт-генераторов

Аватара пользователя
rxu
Сообщения: 1125
Зарегистрирован: 21 янв 2014, 21:20
Откуда: Krasnoyarsk
Github repo: https://github.com/rxu
Контактная информация:

Re: Thanks for posts by Палыч

Сообщение rxu »

Кстати, нужное событие добавлено в код https://github.com/phpbb/phpbb/commit/4 ... 845cb81970
Теперь можно исправлять код рейтинга форумов.

Закрыто