Viewing 5 replies - 1 through 5 (of 5 total)
  • Plugin Author Ash Durham

    (@ashdurham)

    Hi Iamhere,

    That is a valid point my friend however your solution isn’t going to work as you may think as the title when using the image output is the link of the advert 😉

    I will endeavor to add this option in the coming days. Keep an eye out.

    Thread Starter Iamhere

    (@iamhere)

    Hi Ash

    Works for me 😉

    Also you might want to consider limiting the admin thumbnail.
    Right now I am achieving that with this in my custom admin style, otherwise the image is too big

    /* adking pro images */
    tbody#the-list td.advert_image img
     {
    	max-width:150px !important;
    }

    Thread Starter Iamhere

    (@iamhere)

    Also – looks like my original code didn’t come out at the top;

    $output .= "<a href='".get_the_title()."' target='_blank' rel='".$post_id."'  title='click to visit'  >";
                            $output .= "<img src='".$image."' style='max-width: ".$term_meta['advert_width']."px; max-height: ".$term_meta['advert_height']."px;' alt='".get_the_title()."' />";
    Plugin Author Ash Durham

    (@ashdurham)

    Hi Iamhere,

    As you can see in your latest snippet of code, ‘get_the_title()’ is used for the href so I really wouldn’t recommend using that for the alt tag of the image. I will provide a solid solution for this in the next update.

    Regarding the admin thumb, I see what you mean. I’ll add some css on that in the next update also.

    Thanks

    Plugin Author Ash Durham

    (@ashdurham)

    Hi,

    Just letting you know, both the CSS and the option to have an alt tag have been added to version 1.9.0 that will be released in the coming hours.

    Thanks

Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘Suggested improvements’ is closed to new replies.