Image Handler 5 (for v1.5.5 - v1.5.8) Support Thread

[swguy]

Documentation says "For additional information, please refer to the documentation on the plugin's GitHub repository; see above for the link." - but there is no link to Github on this page.

[lat9]

Documentation says "For additional information, please refer to the documentation on the plugin's GitHub repository; see above for the link." - but there is no link to Github on this page.

That link is present in the first note in the overview.

[swguy]

Sure enough. Might be worth adding text-decoration:underline to make that a bit more obvious, and making the link span both words (Github repository). It's really hard to spot right now.

[IATIA]

It would appear that the problem with Chrome is that Google is attempting to become the new IE. They played nice for some time and now that they have their 'customer' base, they are starting to ignore the browser standards in favor of their own extensions. They have their own 'popup' extension. I guess it's time for me to drop Chrome into the same bit bucket as IE. Haven't tried out Edge yet.

[strelitzia]

ZC 1.5.6a, clean install of IH5.1.1 and I'm getting these errors logged
Image Handler: Invalid 'small' background specified (-transparent-), using default.
This, even though the admin config section clearly states "Set to -transparent- to keep transparency."

I see that the notes on version 5.1.0 said "#101: Correct incorrect warning about background definitions."
Was this the same issue, and has 5.1.1 broken/overwritten some previous bug fixes?

And what is the recommended fix for the warnings logged re LARGE_IMAGE_HEIGHT and LARGE_IMAGE_WIDTH, as mentioned in post #148
Should I just create those two defines in the Images config page so that I can assign values to them?

[lat9]

ZC 1.5.6a, clean install of IH5.1.1 and I'm getting these errors logged
Image Handler: Invalid 'small' background specified (-transparent-), using default.
This, even though the admin config section clearly states "Set to -transparent- to keep transparency."

I see that the notes on version 5.1.0 said "#101: Correct incorrect warning about background definitions."
Was this the same issue, and has 5.1.1 broken/overwritten some previous bug fixes?

And what is the recommended fix for the warnings logged re LARGE_IMAGE_HEIGHT and LARGE_IMAGE_WIDTH, as mentioned in post #148
Should I just create those two defines in the Images config page so that I can assign values to them?

What is your setting for Images->IH small images background?

It does look like I missed the incorporation of some of the zc155-related changes (like #101) on the transition to v5.1.1. I'll update GitHub as I know more.

[strelitzia]

What is your setting for Images->IH small images background?

It's set to -transparent-

I'll keep an eye on github for any updates

[lat9]

It's set to -transparent-

I'll keep an eye on github for any updates

I'll need to review that documentation, as the value should be set to transparent (without the leading/trailing minus signs).

[lat9]

ZC 1.5.6a, clean install of IH5.1.1 and I'm getting these errors logged
Image Handler: Invalid 'small' background specified (-transparent-), using default.
This, even though the admin config section clearly states "Set to -transparent- to keep transparency."

I see that the notes on version 5.1.0 said "#101: Correct incorrect warning about background definitions."
Was this the same issue, and has 5.1.1 broken/overwritten some previous bug fixes?

And what is the recommended fix for the warnings logged re LARGE_IMAGE_HEIGHT and LARGE_IMAGE_WIDTH, as mentioned in post #148
Should I just create those two defines in the Images config page so that I can assign values to them?

Regarding those two constants, it looks like the changes got into the base IH class (/includes/classes/observers/ImageHandlerObserver.php) but didn't make it into the FualSlimBoxObserver.php file in the same directory.

I'll be changing the constants there (LARGE_IMAGE_HEIGHT => LARGE_IMAGE_MAX_HEIGHT, LARGE_IMAGE_WIDTH => LARGE_IMAGE_MAX_WIDTH) to align with the IH-provided configuration values.

[lat9]

What is your setting for Images->IH small images background?

It does look like I missed the incorporation of some of the zc155-related changes (like #101) on the transition to v5.1.1. I'll update GitHub as I know more.

After further review, all changes were incorporated on the transition to v5.1.1.