Event Calendar v3

[LaCamus]

I am really struggling to explain with this phenomena....

The original code, before the introduction of the image manipulation code and before I inadvertantly did the double <img> thing, returned the actual image width and height in px (as listed in view source and as stored in the events_images folder) and, even though the image as displayed on the event page may have been smaller than the actual image size, the proportions were maintained.

This should still be the case after image manipulation, but somehow some function (zen_image, zen_image_OLD or whatever) is distorting this process. My limited understanding of coding prevents from seeing how the image width and height values as collected should not be maintained in whatever code does the proportioning for the display.

Does/can event_flink_small do any proportioning, or is the proportioning entirely dependent upon zen_image/zen_image_OLD?

the flink function does not resize on the store side. If the true parameter is used it will use zen_image_OLD to create the image tag, if false (or "blank") is used it creates an image tag without height and width. I am doing nothing out of the ordinary. I think what was happening before is that because of the double open on the IMG tag the height and width were not being detected by the browser.

[DivaVocals]

well just where the heck have YOU been mister???

the flink function does not resize on the store side. If the true parameter is used it will use zen_image_OLD to create the image tag, if false (or "blank") is used it creates an image tag without height and width. I am doing nothing out of the ordinary. I think what was happening before is that because of the double open on the IMG tag the height and width were not being detected by the browser.

Yep.. that's kind of what I suspected.. which probably explains why his old code "appeared" to work.. The real issue is that the CSS using to maintain image dimensions is not complete/correct as it will not work if the IMG tag height and width are there.. The CSS solution I posted (and included with my last fileset upload) should fix that, but it appears that (as usual) Internet Exploder 7 & 8 doesn't like it and needs it OWN solution..

[LaCamus]

...However regardless of whether this is on or off, one can rezsize images by simply selecting the resize radio button that appears on the events preview page. I've looked at this code and I'll admit I do not understand for the life of me what this define "does".

I would assumed that it turns off the resizing functions when set to "off" (re-sizing is not going to occur nor will the re-sizing functions be available to the user). I also would expect that when "on" the re-sizing would occur without an additional action on the part of the user (clicking the re-size button).. The actual behavior is quite different, so I'm confused as to what on/off here means.. Hoping Rus can shed some light..

This define is just for resizing the image in the IMG tag when on the admin side only. Meaning if you turn it off and then upload a large image the whole original image will be seen on the edit and review screens. If it is on then it resizes it only on the tag so that you can reasonably view the image when editing/reviewing. Does that make sense?

[DivaVocals]

This define is just for resizing the image in the IMG tag when on the admin side only. Meaning if you turn it off and then upload a large image the whole original image will be seen on the edit and review screens. If it is on then it resizes it only on the tag so that you can reasonably view the image when editing/reviewing. Does that make sense?

Ahhh yes.. that does.. So the "click here to resize" is an explicit action the user must take in order to resize the image file that is uploaded and this re-sized file will then displayed on the front-end?? Correct???

[LaCamus]

This define is just for resizing the image in the IMG tag when on the admin side only. Meaning if you turn it off and then upload a large image the whole original image will be seen on the edit and review screens. If it is on then it resizes it only on the tag so that you can reasonably view the image when editing/reviewing. Does that make sense?

Sorry sorry, scratch that...
No, resizing should take place with true, and it doesn't when false. Options are not shown when set to false....make sure you did not set the variable to the string 'false', that is not an actual false, if the string contains any text it will create a true condition. The field type in the db has to be boolean. If you can't achieve this try setting the false selection to blank ( '' empty quotes).

[LaCamus]

Ahhh yes.. that does.. So the "click here to resize" is an explicit action the user must take in order to resize the image file that is uploaded and this re-sized file will then displayed on the front-end?? Correct???

again, sorry, but yes the user has to click the resize radio button in order to resize the image, it is resized only in the img for viewing when oversized, it is not actually resized and saved until the button is clicked.

[DivaVocals]

Sorry sorry, scratch that...
No, resizing should take place with true, and it doesn't when false. Options are not shown when set to false....make sure you did not set the variable to the string 'false', that is not an actual false, if the string contains any text it will create a true condition. The field type in the db has to be boolean. If you can't achieve this try setting the false selection to blank ( '' empty quotes).

Great that explains why I wasn't seeing what I expected.. (that the re-sizing options are not displayed at all when this is set to the strong 'false') I have modified the install the script as follows:
from:

(NULL, 'Automatically re-size event images?', 'EVENTS_FILE_RESIZE', 'true', 'If this option is set to true, the module will re-size event images \(based on maximum width and height settings above\).<br /><br />Set to false to disable image resizing.<br /><br />Default is true.', @t4, 5, NOW(), NULL, 'zen_cfg_select_option(array(\'true\', \'false\'),'),

to

(NULL, 'Re-size event images?', 'EVENTS_FILE_RESIZE', 1, 'If this option is set to true, the module will display the re-size event images options on the create/edit preview page. \(based on maximum width and height settings above\).<br /><br />Set to false to disable image resizing.<br /><br />Set as follows:<br />0=False<br />1=True<br /><br />Default is true.', @t4, 5, NOW(), NULL, 'zen_cfg_select_option(array(\'1\', \'0\'),'),

Now this works as I expected.. Thanks..

again, sorry, but yes the user has to click the resize radio button in order to resize the image, it is resized only in the img for viewing when oversized, it is not actually resized and saved until the button is clicked.

Got it.. Now that I have modified the admin setting and supporting menu text this now makes more sense.. Gonna make one small change to the UI to make the "Click here to re-size" stand out a little more.

[LaCamus]

Great that explains why I wasn't seeing what I expected.. (that the re-sizing options are not displayed at all when this is set to the strong 'false') I have modified the install the script as follows:
from:

(NULL, 'Automatically re-size event images?', 'EVENTS_FILE_RESIZE', 'true', 'If this option is set to true, the module will re-size event images \(based on maximum width and height settings above\).<br /><br />Set to false to disable image resizing.<br /><br />Default is true.', @t4, 5, NOW(), NULL, 'zen_cfg_select_option(array(\'true\', \'false\'),'),

to

(NULL, 'Re-size event images?', 'EVENTS_FILE_RESIZE', 1, 'If this option is set to true, the module will display the re-size event images options on the create/edit preview page. \(based on maximum width and height settings above\).<br /><br />Set to false to disable image resizing.<br /><br />Set as follows:<br />0=False<br />1=True<br /><br />Default is true.', @t4, 5, NOW(), NULL, 'zen_cfg_select_option(array(\'1\', \'0\'),'),

Now this works as I expected.. Thanks..

Try using just true and false without the quotes.

[DivaVocals]

Try using just true and false without the quotes.

I did, but trying to make that into a selectable admin option looks strange.. I had to use 1 & 0 for another admin option for the same reasons.. Is there a reason I shouldn't use 1 & 0?? it appears to have the same affect..

[LaCamus]

I was looking at the description...it uses the word "true", but you could use 1 & 0.