New Landing How can we help? Themeforest Theme Support Dante Delete target="_blank" in Clients (Featured)

Viewing 7 posts - 16 through 22 (of 22 total)
  • Posted in: Dante
  • #327369
    Rui Guerreiro – SUPPORT
    Keymaster
    Post count: 25779

    You need to remove that semicolon.
    https://cl.ly/2f3V2s1e3F2C/Image%202017-06-07%20at%2011.20.18%20AM.public.png

    -Rui

    #327377
    Pikay
    Member
    Post count: 48

    Line 329 is now
    $items .= '<a href="'.$client_link_url.'" "'.$target = apply_filters('sf_clients_link_target', '_blank').'"><img src="'.$image[0].'" width="'.$image[1].'" height="'.$image[2].'" alt="'.$image_alt.'" /></a>';

    By clicking a client the link opens in the same window/tab although there is a target="_blank" tag in the source-code … magic ๐Ÿ˜€

    That’s all for now?

    #327379
    Rui Guerreiro – SUPPORT
    Keymaster
    Post count: 25779

    If it’s working, let it be like so that in the next update David code will replace it and you will be able to choose in that option if it opens in a new window or not.

    -Rui

    #327397
    Pikay
    Member
    Post count: 48

    OK, thank you guys! ๐Ÿ™‚

    #327398
    Rui Guerreiro – SUPPORT
    Keymaster
    Post count: 25779

    No problem.

    -Rui

    #328408
    Pikay
    Member
    Post count: 48

    downloaded dante_4.8 now and the issue is still there ๐Ÿ™

    #328417
    David Martin – Support
    Moderator
    Post count: 20834

    Hi,

    I have just submitted the patch to actually fix the bug. You will now be able to use the correct option to control the same/new window.

    Next update will be out soon.

    Thanks.

Viewing 7 posts - 16 through 22 (of 22 total)

You must be logged in and have valid license to reply to this topic.

License required for one of the following items
Login and Registration Log in · Register