51 (edited by sempai 2017.06.16 14:36)

Re: Pan Preview Image - предпросмотр изображений, превью

kisa, легко, только думаю все "гифки" уж не нужно исключать, ибо бывают и большие нормальные картинки, можно сделать проверку на размер изображения и если оно скажем меньше 50*50, просто исключать из слайд шоу!


P.S. в общем сделал так, если изображение больше размера миниатюры, то оно добавляется в FancyBox, если меньше - то и увеличивать его смысла нет!

52 (edited by kisa 2017.06.16 14:42)

Re: Pan Preview Image - предпросмотр изображений, превью

PunBB_INFO_ICQ/yes Мы с PunBB, немного поднимали этот вопрос. Я предложил исключить из обработки вообще все изображения, если их размер составляет 110-120% от заданного размера превью. Ибо это глупо. Тыкаешь в картинку, а она не увеличивается, а просто рядом дублируется такая же маленькая.

Так большая нормальная гифка не обрабатывается - она законо и красиво радует в натуральный размер, что от неё и требуется.

Spoiler

https://punbb.softplaza.net/uploads/images/2017/06/b4bc5ea819e8aabd7a68af6992b245bc.gif

Не нужно их вообще трогать - им в тексте место. Это ж обычно просто приколы. И свежесть их теряется от любой обработки.

53

Re: Pan Preview Image - предпросмотр изображений, превью

Да, вычислять размер для каждого изображения оказалось слишком ресурсозатратным занятием, ограничился фильтрацией по типу (тип gif - не учитывается скриптом).

54

Re: Pan Preview Image - предпросмотр изображений, превью

PunBB_INFO_ICQ/yes это по уму. Уже завтра поиграюсь.
А  классная всё-таки связка получается PunBB_INFO_ICQ/music

55

Re: Pan Preview Image - предпросмотр изображений, превью

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

manifest, строка 40.

вместо:

if (!class_exists('ImageResize'))

нужно:

if (!class_exists('Eventviva::ImageResize'))

Причина ошибки, там где помимо класса используются пространства имен - необходимо их писать перед объявлением класса.

----------------------------------------------------------------------------------------------------------------------------------------------

Рац предложение, так как Uploader у нас в зависимостях, то и библиотека resizeimage.php само собой не куда не денется.

Предлагаю в манифесте строки 40-41 заменить на:

if (!class_exists('Eventviva::ImageResize'))
    include FORUM_ROOT.'extensions/pan_uploader/resizeimage.php';

А в settings.php строку 14 заменить на:

require FORUM_ROOT.'extensions/pan_uploader/resizeimage.php';

И удалить из расширения библиотеку resizeimage.php, попутно облегчив его на 13кб.

1

56 (edited by PunBB 2017.06.17 13:27)

Re: Pan Preview Image - предпросмотр изображений, превью

sempai, отличное решение. Доберусь до ПК, исправлю  PunBB_INFO_ICQ/thumbsup

Там еще у самого аплоадера много стилей для старого диалогового окна прописано. Видимо у Маракат из за них и сдвиги с с визуальным редактором. Еще почему то не было закрыто соединение с бд, хотя помню точно, что с самого начала добавлял  PunBB_INFO_ICQ/scratch
Помимо этого в некоторых местах файл удалялся без проверки на его существование. Все это было обнаружено, когда совмещал работу с превьюхами.  PunBB_INFO_ICQ/bigsmile

57

Re: Pan Preview Image - предпросмотр изображений, превью

Еще почему то не было закрыто соединение с бд, хотя помню точно, что с самого начала добавлял

Это наверное я удалил, когда "кастрировал" расширение, так как не нашел где это самое соединение открывалось!  PunBB_INFO_ICQ/jokingly

58 (edited by kisa 2017.06.16 23:53)

Re: Pan Preview Image - предпросмотр изображений, превью

sempai, С пропуском гиф - хорошо. Слайд-шоу как-то лучше получается, нет ощущения мусора.
PunBB, нехорошо, что маленькие картинки обрабатываются. С фанси боксом вроде неплохо, а с чистым Pan Preview Image  PunBB_INFO_ICQ/no

На тестовом (ссылка выше) сейчас Фанси Бокс стоит, вот здесь посмотри.  Картинка маленькая изначально.  Ну глуповато же - при клике она дублируется в том же размере.
https://punbb.softplaza.net/uploads/images/2017/06/eeb3350bd1156e4f0094ed74c29d2f80.jpg

Сдаётся мне, если она больше превьюхи хотябы  на 140-150%, только тогда имеет смысл обрабатывать. На каком этапе? Наверное, стОит просто не делать из таких превью.

$matches[1]

Вот картинка размером 140% от превью.
https://punbb.softplaza.net/uploads/images/2017/06/331d9e5e9184d99c28e652b344962b8c.jpg

$matches[1]

Можно ещё обойти глупый вид тем, что скрывать превьюху в тексте, когда открывается большая картинка. Это красиво, логично, и возможно - будет очень полезно позже.

$matches[1]

Да.
Пользователь будет видеть, что механизм работает.
А то тыкаешь в одну маленькую картинку - она открывается. В другую - http://krokovod.org/forum/extensions/pan_smiles/pack/img/fig.gif. Оно как-то... можно подумать, что что-то не срабатывает. Раздражать будет, короче.

А так картинка сработала - ничего не раздражает. Просто она маленькая изначально в посте - всё понятно.

$matches[1]

Но совсем маленькие нужно обходить.Может, от 80% размера превью. Размер НЕобрабатываемой картинки должен быть таким, чтобы пользователь внятно на глаз видел, что картинка маленькая, и нечего в неё тыкать.

59

Re: Pan Preview Image - предпросмотр изображений, превью

kisa wrote:

Но совсем маленькие нужно обходить.Может, от 80% размера превью.

может опцию добавить? кому надо какие пусть настраивают.

Например. Поле для ввода процентного соотношения с реальной превью картинкой. Только мне кажется больше времени на мануал уйдет  PunBB_INFO_ICQ/jokingly

60 (edited by kisa 2017.06.17 14:48)

Re: Pan Preview Image - предпросмотр изображений, превью

Думаю лишнее. Посто задать. Если угадаем с процентом, никто и не почешется что-то менять - хорошо будет. А кому надо будет очень - спросят либо найдут, где в коде циферку поменять.
Кстати, чем дальше, тем больше нравятся в нынешнем варианте размеры разных картинок на разных экранах. Но на ПК чуть гуляет позиционирование, но трогать страшно  PunBB_INFO_ICQ/bigsmile Если в идеале - алгоритм сложноватый получается.
Если картинка влезает по вертикали в экран, её нужно позиционировать по центру. Если не влезает - по верхнему краю.
Ну его?

61 (edited by PunBB 2017.06.17 14:51)

Re: Pan Preview Image - предпросмотр изображений, превью

kisa wrote:

Кстати, чем дальше, тем больше нравятся в нынешнем варианте размеры разных картинок на разных экранах.

из за того, что картинка адаптировалась при $matches[2] экрана? или еще что есть?

62 (edited by kisa 2017.06.18 11:13)

Re: Pan Preview Image - предпросмотр изображений, превью

И то, что на достаточно большом экране она показывается в натуральную величину. А сравнительно небольшие не увеличиваются неестественно.
Это логично и приятно: по возможности показывать в натуральную, а уменьшать или резать - только при необходимости. Вот оно так и работает сейчас.

И режутся только те, что по вертикали не лезут - тут скролл в помощь. Это лучше, чем уменьшать.

Короче, с размерами классно.

$matches[1]

У меня ж 1024 максимальный ".fancy_video_tag_parse($matches[1])." и вертикали и горизонтали выствлен, и довольно много таких картинок - нужны. Так что оценил.

$matches[1]

Оговариваясь, что сейчас расширение очень приятно (удивительно, в нашем очень кипешном сообществе НИКТО не высказал не то что НИ ОДНОЙ претензии, но и пожеланий, кроме мультизагрузки - что сюда не относится, и не нужно!), тем не менее, позволю себе чётче сформулировать вид со стороны пользователя некоторых мелочей, отделяющих нынешнее состояние от идеала  PunBB_INFO_ICQ/smile Ну, чтоб было  PunBB_INFO_ICQ/bigsmile В описание входит и то, что уже есть.

$matches[2]
$matches[2]
1. Размер по ширине имеет приоритет перед размером по высоте при выборе показываемой картинки, и однозначно задаёт её размер.
2. Картинка показывается в натуральную максимальную ширину до тех пор, пока помещается на экране. Соответственно, вертикальные картинки позже претерпевают уменьшение на малых экранах.
3. С уменьшением экрана меньше ширины картинки,  картинка уменьшается, занимая всю ширину имеющегося экрана.

$matches[2]
1. Высота задаётся только исходя из ширины картинки (см выше) пропорционально.

$matches[2]
$matches[2]
1. если картинка по ширине меньше экрана, она располагается точно по центру. Допустимо небольшое смещение, которое визуально скрадывается ассиметричностью дизайна топика.
2. Если картинка занимает весь экран, она автоматически располагается  симметрично.

$matches[2]
1. Если высота картинки не превышает высоты экрана, картинка центруется симметрично по высоте.
2. Если высота картинки превышает высоту экрана, картинка выравнивается по верхнему краю с минимальным отступом от верхнего края экрана.  Это даёт возможность воспользоваться скроллом, логично просматривая картинку сверху вниз.

$matches[2]
1. При клике на превью и открытии основной большой картинки, превью скрывается (не нарушая форматирования топика!).
2. При этом из обработки исключаются изображения, чей размер явно, на глаз пользователя, меньше остальных превью. Хорошим порогом видится примерно 70% .

$matches[1]

FancyBox for Pan Preview of Images, как надстройка над сабжем, в связке с ним почти идеален, и такая связка не требует усовершенствований прямо на данном этапе. Для любителей такого оформления и функционала эта связка - уже законченный продукт.  Всё-таки вызывает сомнение тот момент, что подбираются картинки со всей страницы, а не из одного поста. Может быть нелогичный венигрет из картинок. Внутри поста они более-менее объединены по смыслу и связаны с текстом, а на целой странице могут быть смысловые недоразумения от странной последовательности картинок. .

$matches[1]

Очень хорошо, что стили  Pan Preview Image простые, без прыжков @media, вносящих смуту.

63 (edited by sempai 2017.06.18 16:05)

Re: Pan Preview Image - предпросмотр изображений, превью

kisa, проверь этот вариант, должен группировать изображения по постам.

FancyBox for Pan Preview of Images

64

Re: Pan Preview Image - предпросмотр изображений, превью

Забрал

$matches[1]

PunBB_INFO_ICQ/thumbsup Любуйся  PunBB_INFO_ICQ/bigsmile
вот ссылка

Таки лучше, логичнее. Хоть понимаешь (если там реальный контент, а не мой макет), к чему те картинки.

65

Re: Pan Preview Image - предпросмотр изображений, превью

kisa, не торопись, это всего лишь тестовый вариант, он актуален только для viewtopic, для остального нужно будет еще поковыряться. PunBB_INFO_ICQ/secret

1

66

Re: Pan Preview Image - предпросмотр изображений, превью

sempai, это постоянная ссылка на обновляемое расширение? Может мне ее в первый пост выложить? Или хочешь создай тему с описанием.

67 (edited by kisa 2017.06.18 15:25)

Re: Pan Preview Image - предпросмотр изображений, превью

Побаловался ещё Боксом . Та ваще - готовый продукт PunBB_INFO_ICQ/yes

$matches[1]

sempai, А где ещё ты картинки хочешь показывать? Аватары прокручивать на главной?  PunBB_INFO_ICQ/bigsmile

68

Re: Pan Preview Image - предпросмотр изображений, превью

А ты нажми на кнопочку ответить (в теме), и посмотри что получилось!  PunBB_INFO_ICQ/wink

69 (edited by kisa 2017.06.18 15:34)

Re: Pan Preview Image - предпросмотр изображений, превью

А, в етам смысле PunBB_INFO_ICQ/scratch
Ну, работает PunBB_INFO_ICQ/pardon И в предпросмотре.

70

Re: Pan Preview Image - предпросмотр изображений, превью

kisa wrote:

Ну, работает  И в предпросмотре

Объединяет картинки по группам в постах?

71

Re: Pan Preview Image - предпросмотр изображений, превью

А, не, по одной показывает. Ну.. я б годами пользовался таким форумом, и ни разу не тыкал увеличить там картинку  PunBB_INFO_ICQ/bigsmile Чистые превьюхи там вообще не работают, и не надо - прямо считаю это лишним. Эти посты в обратном порядке там служат для ориентировки, а не для любования картинками.

Да, получается, у тебя сейчас вариант - ни к умным, ни к красивым, да. Или не трогать там, как в пан превью, или уже по полной обрабатывать. Я б не трогал,   зачем в том месте эти навороты? Особенно, если это как-то на ресурсопотреблении сказывается.

$matches[1]

Не место там для спецэффектов. Там и аватар нет, и ещё всякого.

72

Re: Pan Preview Image - предпросмотр изображений, превью

Ок, тогда "умную" группировку оставлю только в темах, в остальных местах все найденные картинки будут зацикливаться как и было раньше.  PunBB_INFO_ICQ/smile

73 (edited by kisa 2017.06.18 16:22)

Re: Pan Preview Image - предпросмотр изображений, превью

А может в этих ответах и предпросмотрах вообще не трогать, отключить обработку? Ну кто будет в тот подвал безаватарный лазить за слайд-шоу? То справочнное место, чтоб если нужно, освежить в памяти предыдущие посты, сочиняя свой.
Цитирование оттуда - полезно. Воможно, обращение по стрелочке было бы полезно. А иллюзион там устраивать  PunBB_INFO_ICQ/pardon

74

Re: Pan Preview Image - предпросмотр изображений, превью

Нет, так как эта библиотека подхватывает изображения не только в темах, у меня к примеру она активно используется в чате, может у кого еще где ни будь, зачем же обрезать имеющийся функционал!  PunBB_INFO_ICQ/wink

75 (edited by kisa 2017.06.18 16:27)

Re: Pan Preview Image - предпросмотр изображений, превью

PunBB_INFO_ICQ/pardon
Ну этак можно до прокрутки аватар дофунциклироваться  PunBB_INFO_ICQ/bigsmile
Тогда почему не оставить поштучное открытие?  Памятуя о том, что именно там наврядли кто-то додумается вообще на те превьюхи тыкать?
Разве что, одну какую-то фотку увеличить, если как раз её коментирую в создаваемом посте.
Прокрутка не особо и нужна, получается..

76 (edited by sempai 2017.06.18 21:15)

Re: Pan Preview Image - предпросмотр изображений, превью

kisa wrote:

Прокрутка не особо и нужна, получается

Да, но и вреда от нее нет, не так ли. Никто же силой не заставляет тыкать на эти стрелочки!  PunBB_INFO_ICQ/bigsmile


Если уже так сильно будет мешать, можешь открыть хук ps_handle_img_tag_end, и поудалять из ссылок:

 rel="group"

77 (edited by sempai 2017.06.18 21:15)

Re: Pan Preview Image - предпросмотр изображений, превью

PunBB, да, конечно можно вынести в первый топик, так как расширение является дополнением, а целевое расширение прописано у него в зависимостях!

постоянная ссылка

1

78

Re: Pan Preview Image - предпросмотр изображений, превью

PunBB_INFO_ICQ/ok есть. Добавил. Теперь если кому то захочется больше визуальных эффектов там есть ссылка.

1

79

Re: Pan Preview Image - предпросмотр изображений, превью

sempai, Пойдёт  PunBB_INFO_ICQ/bigsmile

80

Re: Pan Preview Image - предпросмотр изображений, превью

Скоро!!!

Превью из внешних изображений  PunBB_INFO_ICQ/dance

81

Re: Pan Preview Image - предпросмотр изображений, превью

Обновлено расширение для создания превью картинок.

+ Добавлена возможность установки процента сжатия превьюшек.
+ Создание превью для внешних картинок.
+ Удаление превьюшек с сервера при удалении поста или темы.
+ Удален файл сжатия. Используется файл аплоадера.

1

82

Re: Pan Preview Image - предпросмотр изображений, превью

всем привет. в логах появилась такая строка:

PHP Notice:  Undefined index: path in...
/www/extensions/pan_preview_image/hooks/pan_uploader_ed_pre_redirect_end.php on line 14

проверьте пожалуйста.

83

Re: Pan Preview Image - предпросмотр изображений, превью

Here is my French lang file for pan_preview-image
This content is available only to registered users.

84 (edited by DronSF 2019.06.09 09:10)

Re: Pan Preview Image - предпросмотр изображений, превью

А если фото загружены на другой сайт и в сообщения вставлены bbcode img на эти фото, как сделать миниатюры без создания отдельных превью-файлов для таких изображений?
Есть расширение такое или нужно скриптами делать вручную?

85 (edited by kisa 2019.06.09 11:19)

Re: Pan Preview Image - предпросмотр изображений, превью

@DronSF, Такое пойдёт? https://punbb.softplaza.net/t-1045.html  PunBB_INFO_ICQ/cool

86

Re: Pan Preview Image - предпросмотр изображений, превью

@kisa, То что надо. PunBB_INFO_ICQ/ok  Как я пропустил это  PunBB_INFO_ICQ/scratch
Я там вопросик задам