Davedub

I’ve updated the settings, feedback welcome!

Hi Kathy,

I’ve added this to the ‘known issues’ list on the website ( https://davedub.co.uk/davedub/wordpress/dd-contact-form/ )

However, on fresh install, the plugin currently send an email to the site admin, but with instructions on how to change the email address.

That said, I do plan to change it to simply set the site admin as the form enquiry recipient on install in the next update.

– Davedub

Hi,

Yep, I’d tend to agree with the theme developer and would be very keen to get to the bottom of this. To do so, I’d need to at least see a page where the problem is happening. Ideally, I’d get a copy of the theme so I could test it out.

~ Could you email me directly using the form on my site?

( https://davedub.co.uk/davedub/wordpress/contact/ )

– Davedub

Hi manosdp,

Because the form is responsive over many different screen sizes and layout configurations, adding new fields to the form could be quite tricky. So, I wouldn’t recommend adding a text field. Just so I’m clear, you are talking about adding more than the two text fields you can enable in the settings correct?

To change verbage, unless there is an entry in the plugin settings to do so, you’d currently need to edit the plugin files. Obviously, this is not ideal, as your changes would get wiped every time I updated the plugin. I do plan to add label configuration and multilingual support at some stage.

– for my future reference, can I ask which bits of verbage you want to change?

– Davedub

Hi Kathy,

Sorry for the delay replying, illness decided to strike the day after I released the plugin!

I don’t quite understand your question – when you say ’email address’, do you mean the email address the plugin sends the filled out forms to, as set in the settings, or are you talking about the email address field on the actual form?

Kind Regards,

Davedub

After the above fix, the images were not centering. To get the images to center horizontally, I changed

margin:0;

to

margin:auto;

for slide img in common.min.css.

However, I’m now having trouble getting the vertical centering to work – any suggestions?

– Davedub

I think there’s a problem with the first few lines of the getSizeByAuto funtion in class-image-resizer.php – I could think of a couple of ways the existing method might fail. Would soemthing like this not be better?


private function getSizeByAuto($newWidth, $newHeight)
{
$ratio = $this->height / $this->width; // aspect ratio of image
$sliderRatio = $newHeight / $newWidth; // aspect ratio of slider

if ($ratio < $sliderRatio)
// *** Image to be resized is wider (landscape)
{
$optimalWidth = $newWidth;
$optimalHeight= $this->getSizeByFixedWidth($newWidth);
}
elseif ($ratio > $sliderRatio)
// *** Image to be resized is taller (portrait)
{
$optimalWidth = $this->getSizeByFixedHeight($newHeight);
$optimalHeight= $newHeight;
}
else ...etc

Hi kosinix,

I tried adding the resize option to the cyclone_slide_image_url as directed, but the only changes seems to be that the images are no longer centred!

I have it running here, with a fresh install of the plugin: https://thegallerychaloklum.com/

Any help very much appreciated!

– Dave

@kosinix – please? I really, really need to just be able to set the resize method to ‘portrait’!

Update 2: I’ve been digging around the code, trying to set the resize option to portrait, but with no results – vexed!

I have a feature suggestion I’d like to put forward:

Add ‘Resize Method’ dropdown select to the slider settings (slider-properties.php) and individual slide settings (box.php) with choice of:

• exact
• portrait
• landscape
• auto
• crop

I’ve already added the vars and dropdowns to my own copy of your code (let me know if you’d like a copy) but am still unable to actually change the resizing method!
Time’s marching on, I can’t do too much more on this right now – awaiting your reply with interest…

Update: It does look like the solution above was for an older version. I now have version 2.5.3 which includes class-image-resizer.php

Inside class-image-resizer.php I found a function called:

resizeImage($newWidth, $newHeight, $option="auto")

As a trial hack, I added this as the first line to this function:

$option="portrait";

This had no effect. So I searched through ALL the slider’s php files, and was unable to find a single reference to the resizeImage function – so I’m now stuck!

Any advice greatly appreciated. Link for slider on my site:
https://thegallerychaloklum.com/gallery/

Hi kosinix,

I too am having this issue and your instructions make complete sense to me.

However, when I check the slider.php file, there is no reference to cycloneslider_thumb.

Has the slider code been updated since this post? Here’s the code I have for slider.php (default):

<?php if($slider_count>0) $slider_id = $slider_id.’-‘.$slider_count; ?>
<div class=”cycloneslider cycloneslider-template-default” id=”cycloneslider-<?php echo $slider_id; ?>” style=”max-width:<?php echo $slider_settings[‘width’]; ?>px”>
<div class=”cycloneslider-slides cycle-slideshow” <?php echo cyclone_settings($slider_settings, $slider_id); ?>>
<?php foreach($slider_metas as $i=>$slider_meta): ?>
<div class=”cycloneslider-slide” <?php echo cyclone_slide_settings($slider_meta, $slider_settings); ?>>
<?php if ($slider_meta[‘type’]==’image’) : ?>
<?php if ($slider_meta[‘link’]!=”) : ?>” href=”<?php echo $slider_meta[‘link’];?>”><?php endif; ?>
<img src=”<?php echo cyclone_slide_image_url($slider_meta[‘id’], $slider_settings[‘width’], $slider_settings[‘height’], array(‘current_slide_settings’=>$slider_meta, ‘slideshow_settings’=>$slider_settings) ); ?>” alt=”<?php echo $slider_meta[‘img_alt’];?>” title=”<?php echo $slider_meta[‘img_title’];?>” />
<?php if ($slider_meta[‘link’]!=”) : ?><?php endif; ?>
<?php if(!empty($slider_meta[‘title’]) or !empty($slider_meta[‘description’])) : ?>
<div class=”cycloneslider-caption”>
<div class=”cycloneslider-caption-title”><?php echo $slider_meta[‘title’];?></div>
<div class=”cycloneslider-caption-description”><?php echo $slider_meta[‘description’];?></div>
</div>
<?php endif; ?>
<?php elseif ($slider_meta[‘type’]==’video’) : ?>
<?php echo $slider_meta[‘video’]; ?>
<?php elseif ($slider_meta[‘type’]==’custom’) : ?>
<?php echo $slider_meta[‘custom’]; ?>
<?php endif; ?>
</div>
<?php endforeach; ?>
</div>
<?php if ($slider_settings[‘show_nav’]) : ?>
<div class=”cycloneslider-pager”></div>
<?php endif; ?>
<?php if ($slider_settings[‘show_prev_next’]) : ?>
<div class=”cycloneslider-prev”>Prev</div>
<div class=”cycloneslider-next”>Next</div>
<?php endif; ?>
</div>