jcampbell88

Hi Steve,

Thanks for getting back to me – that sounds ideal, will watch the Trello card.

Cheers,

J

Marking as resolved.

Hi Steve,

It was the apply_filters that was messing everything up, wasn’t calling the $image variable so the size wasn’t being applied.

Thanks for your help.

J

Nothing happens – original image is still loaded, here’s one where the orig is 2000px:

https://test.blackhay.co.uk/property/prestwick-ka9/

I have large defined as max. 700px so even if it’s 1000 it should still be resized?

Jonny

Regenerated all the thumbnails and disabled any caching plugins but the problem persists:

https://test.blackhay.co.uk/property/troon-ka10/

Here’s the full code from my property-images.php page:

<?php
/**
 * Single Property Images
 *
 * @author      PropertyHive
 * @package     PropertyHive/Templates
 * @version     1.0.0
 */

if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly

global $post, $propertyhive, $property;

$gallery_attachments = $property->get_gallery_attachment_ids();
?>
<div class="images">

    <?php
        if ( !empty($gallery_attachments) ) {

            echo '<div class="slider-for slick-slide-img--full">';

            foreach ($gallery_attachments as $gallery_attachment)
            {
                $image_title = esc_attr( get_the_title( $gallery_attachment ) );
                $image_link  = wp_get_attachment_url( $gallery_attachment );
                $image = wp_get_attachment_image( $gallery_attachment, 'large' );

                echo '<div>' . apply_filters( 'propertyhive_single_property_image_html', sprintf( '<img src="%s" alt="Placeholder" />', $image_link, $image_title, $image ), $post->ID ) . '</div>';
            }

            echo '</div>';

        } else {

            echo apply_filters( 'propertyhive_single_property_image_html', sprintf( '<img src="%s" alt="Placeholder" />', ph_placeholder_img_src() ), $post->ID );

        }
    ?>

    <?php do_action( 'propertyhive_product_thumbnails' ); ?>

</div>

Thumbnail support is added via my theme’s functions file, if that matters.

Would this only affect images uploaded after the change? All the images on the site have the necessary sizes generated.

Will also see if anything is being cached.

Thanks,

J

Hi Alan,

I’ve applied this fix, thanks for replying so quickly.

J

Hi there,

I’ve encountered a problem with this fix – when you applied the code:

.media-toolbar-secondary {
width: 100%;
}

in your update, it pushes the ‘Select’ button out of sight in the ‘Add Media’ in Microsoft Edge and below

Morning Steve,

Thanks for getting back to me. In regards to how useful it is, it was requested by one of my clients – if I’m honest I don’t see the value in it either, I would think that sorting by price, etc. is far more beneficial.

If you could add this as a feature request I’d really appreciate it, thank you ??

Jonny

• This reply was modified 7 years ago by jcampbell88.

Sorry, just wanted to give this a bump – is it doable?

Thanks,

Jonny

Looking to have it sorted by whatever gets output in the item_title hook i.e. what populates <?php the_title(); ?> in the template files, unless we can sort by the individual address elements? In that case it would be the ‘Street’ field.

Thanks,

Jonny

Hi Steve,

Just on the back of this thread – I’m looking to add the ability to sort the search results by address alphabetically, will this also be possible when the ability to override the default args is added?

Thanks,

Jonny

• This reply was modified 7 years, 1 month ago by jcampbell88.