Re: Image Handler 5 (for v1.5.5) Support Thread
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.
Re: Image Handler 5 (for v1.5.5) Support Thread
Quote:
Originally Posted by
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.
That link is present in the first note in the overview.
Re: Image Handler 5 (for v1.5.5) Support Thread
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.
Re: Image Handler 5 (for v1.5.5) Support Thread
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.
Re: Image Handler 5 (for v1.5.5) Support Thread
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?
Re: Image Handler 5 (for v1.5.5) Support Thread
Quote:
Originally Posted by
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?
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.
Re: Image Handler 5 (for v1.5.5) Support Thread
Quote:
Originally Posted by
lat9
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
Re: Image Handler 5 (for v1.5.5) Support Thread
Quote:
Originally Posted by
strelitzia
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).
Re: Image Handler 5 (for v1.5.5) Support Thread
Quote:
Originally Posted by
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?
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.
Re: Image Handler 5 (for v1.5.5) Support Thread
Quote:
Originally Posted by
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.