• Hello,

    I have created a Kadence Fluid carousel Gallery, which opens the images in Lightboxes. I need some of those images to go externally to another page, so I thought of adding a link in the caption of the image with the code <a href="" target="_blank">TEXT</a>.

    [Issue] In the thumbnail carousel the images are displayed correctly and the link is visible at the bottom of the image. However when I click to view it?in lightbox: the linked captions lose the link and only the text is displayed. Also,?images that have links as captions are duplicated?in the lightbox carousel.

    [Request] Regarding the request (I don’t know if there is a specific section for it), it is related to the problem of losing the links in the Lightbox:

    Can you add the tag ‘a’ for the links in the lightbox captions?

    I was checking your code in case I could do it myself through some filter, but I only found ‘kadence_blocks_pro_dynamic_gallery_caption’ which is applied before.

    I found that on line 769 of class-kadence-blocks-advancedgallery-block.php there is a call to ‘strip_tags( html_entity_decode($caption, ENT_QUOTES, ‘UTF-8’) )’ with no tags arguments allowed. It would be interesting if you could include link tag as a second argument:

    $allowed_tags = '<a>';
    
    [...] (Line 769) 
    
    if ( ! empty( $href ) ) {
    	$output .= '<a href="' . esc_url( $href ) . '"' . ( $link_to === 'media' && $lightbox === 'magnific' && $lightbox_cap && ! empty( $caption ) && is_string( $caption ) ? ' data-description="' . esc_attr( strip_tags( html_entity_decode($caption, ENT_QUOTES, 'UTF-8'), $allowed_tags ) ) . '"' : '' ) . '' . ( $link_to === 'media' && $lightbox === 'magnific' && ! empty( $image_alt ) && is_string( $image_alt ) ? ' data-alt="' . esc_attr( $image_alt ) . '"' : '' ) . ' class="kb-gallery-item-link" ' . ( ( $link_to === 'custom' && '_blank' === $link_target ) || ( $link_to === 'media' && $lightbox === 'new_tab' ) ? 'target="_blank"' : '' ) . ' ' . ( ( $link_to === 'custom' && ! empty( $rel_attr ) ) || ( $link_to === 'media' && ! empty( $rel_attr ) ) ? 'rel="' . esc_attr( $rel_attr ) . '"' : '' ) . '>';
    }	
    

    Also the ‘strong’ tag for bold would be interesting.

    As for the image duplication problem when opening in Lightbox images with links in caption I haven’t found yet in the code what might be happening.

    Thanks for your work on this very complete plugin.

    I look forward to your reply, thank you very much in advance.

    P.S.: I don’t know if I posted in the right place, initial post here: Initial post in Github

    Best regards,
    Marina.

Viewing 6 replies - 1 through 6 (of 6 total)
  • Plugin Support Bonn

    (@bonnlevelup)

    Hi there,

    Thanks for the request.

    I will forward this to our development team.

    Let me know if there’s anything else we can help you with.

    Thread Starter margr

    (@mar1984)

    Hi,

    Thanks for passing it on to the development team, I hope they find it useful and approve its inclusion.

    My problem is precisely because when I put a link in the image caption the slide comes out double in the carousel in full screen, if there is any way to solve it.

    [Issue]?In the thumbnail carousel the images are displayed correctly and the link is visible at the bottom of the image. However when I click to view it?in lightbox: the linked captions lose the link and only the text is displayed. Also,?images that have links as captions are duplicated?in the lightbox carousel.

    Plugin Support michael-levelup

    (@michaeltarongoy)

    Hi there,

    Currently, this is escalated to the development team via our internal issue tracker. We have no new update as of the moment. We will send you an update once we learn more.

    We appreciate your continued patience and understanding regarding this matter.

    Kind Regards,
    Michael Taro

    Thread Starter margr

    (@mar1984)

    Hello,

    Thank you, I look forward to hearing from you.

    Best regards,
    Marina

    Plugin Support michael-levelup

    (@michaeltarongoy)

    Hi there,

    Just a quick update.

    This has been added to the upcoming development sprint. That doesn’t mean it will be finished and the update will be released once the development sprint ends though.

    Please note that the actual timing is still subject to change.

    We appreciate your continued patience and understanding regarding this matter.

    Kind Regards,
    Michael Taro

    Thread Starter margr

    (@mar1984)

    Hi Michael,

    Thank you very much for the update, I understand the development timing.

    I remain on standby.

    Kind regards,
    Marina

Viewing 6 replies - 1 through 6 (of 6 total)
  • You must be logged in to reply to this topic.