New Landing How can we help? Themeforest Theme Support Uplift every link on an image seems like a lightbox link

Viewing 15 posts - 1 through 15 (of 22 total)
  • Posted in: Uplift
  • #323821
    Silentrob
    Member
    Post count: 24

    Hi,

    On my homepage you’ll see a row of 4 pictures (the ones with the tomatoes), the left one has a normal link on it. And the lightbox thing is disabled (not toggled). But still I see the zoom in icon on hover and it seems like it tries top en something before going to a link.

    Can you help me?

    Tnx!
    R

    #323899
    Silentrob
    Member
    Post count: 24

    for me you don’t need to look into it anymore, client changed its mind, but I think its still a bug 🙂

    #323903
    Rui Guerreiro – SUPPORT
    Keymaster
    Post count: 25779

    Ok. Thanks for the heads up.

    -Rui

    #323995
    Silentrob
    Member
    Post count: 24

    client changed it’s mind again, i now have it on the page: https://www.stoffels-tomaten.be/onze-producten/

    So on hover i see the “zoom in” icon, and i don’t think that’s the right thing here when it’s a link.

    Pls help ^^

    #324136
    Rui Guerreiro – SUPPORT
    Keymaster
    Post count: 25779

    Hi,

    Can you add the code below to the functions.php of your child theme

    remove_filter('sf_view_icon_svg', 'uplift_lightbox_svg_icon');
    add_filter('sf_view_icon_svg', 'uplift_url_svg_icon');

    Let us know if it worked.

    -Rui

    #324148
    Silentrob
    Member
    Post count: 24

    I tried it on several lines but didnt seem to work, which line would you recommend?

    Attachments:
    You must be logged in to view attached files.
    #324281
    David Martin – Support
    Moderator
    Post count: 20834

    Below line 73, below the last function ideally.

    Thanks.

    #324286
    Silentrob
    Member
    Post count: 24

    Nope, doesn’t work. =>https://www.stoffels-tomaten.be/onze-producten/

    I still a zoom-in icon instead of the link icon when hovering. When I had just one image there, it didnt give the issue though.

    Attachments:
    You must be logged in to view attached files.
    #324310
    Rui Guerreiro – SUPPORT
    Keymaster
    Post count: 25779

    Can you provide me ftp to give it a try?

    -Rui

    #324340
    Silentrob
    Member
    Post count: 24
    This reply has been marked as private.
    #324508
    Rui Guerreiro – SUPPORT
    Keymaster
    Post count: 25779

    It was necessary to hook that code in a specific WordPress hook.
    Final code in the functions.php

    function sf_remove_filters(){
    	remove_filter('sf_view_icon_svg', 'uplift_lightbox_svg_icon');
    	add_filter('sf_view_icon_svg', 'uplift_url_svg_icon');
    }
    add_action('init', 'sf_remove_filters');

    -Rui

    #324523
    Silentrob
    Member
    Post count: 24

    thanks, resolved!

    #324600
    David Martin – Support
    Moderator
    Post count: 20834

    Marking as resolved.

    If you have found the theme/support useful, we appreciate it if you can leave feedback on our item if you have the time as it really helps us out.

    Thanks.

    #324603
    Silentrob
    Member
    Post count: 24

    Hi david,

    it’s not 100% resolved, now the items that are image links to open a lightbox show the link icon and not the zoom icon.
    (on my site you can see it when selecting the other tab, those are image links instead of deeper pages)

    So I think there’s an issue/buyg more profoundly wrong?

    Cheers,
    Robby

    #324680
    Rui Guerreiro – SUPPORT
    Keymaster
    Post count: 25779

    Can you let me know one or two examples? Probably it’s some error.

    -Rui

Viewing 15 posts - 1 through 15 (of 22 total)

You must be logged in to reply to this topic.