Slider - Open slide-links in new tab

Currently links set for slides are always opened in the current tab.
It might be useful to open the link in a new tab.
Therefore a setting should be added that enables opening a link, when clicking a slide, in a new tab.

mentioned in commit 2427e29b

@henbug
I thought this might be a useful addition.
Could you check if you find any major flaws in the commit above (2427e29b)?
I tested it and everything seems to work just fine.

window.open(url, '_blank'); ... this solution does not work with popup blocker. this must be solved completely differently. probably with target blank and a elements.

default must definitely not be this option.

you can theoretically import this but give some hints that this might not work with popup blocker

mentioned in commit 79882a44

mentioned in commit 5215cad9

mentioned in commit 437e5b7b

I was worried about that as well.
Therefore I now removed the JavaScript-referral and changed to a-elements in the sliders (see referenced commits above).
This seems to work fine too.

ok you can add it to the dev

mentioned in merge request !4 (merged)

feature is now in the dev-branch, therefore closing this issue

closed

mentioned in commit 2cfa2abf

Ich muss das hier reopen @Panni weil ich dazu paar Fragen / Anregungen habe ;)

1. Wenn man keinen Link im Backend setzt, sollte auch kein <a> Tag gesetzt werden, oder sieht ihr das anders? So kann man die Slides nicht als Informations-Slides (mit nur Text) benutzen. Auch wenn ich eigene Links (Buttons) in den Inhalt setze, kann man sie nicht klicken.

2. Außerdem verwirrt mich der Cursor (pointer) über die kompletten Slides.

3. In verstehe nicht ganz wo die ganzen <a> Tags herkommen. In Code sehe ich nur den äußeren Tag:

{foreach $desktopSlides as $key => $slide}
<li data-image="{image image=$slide.image onlysrc=1}" data-href="{$slide.url}" data-newTab="{$slide.newTab}">
    <a href="{$slide.url}" {if $slide.newTab} target="_blank" {/if}>
    {assign var=extraClass value=""}

    {if isset($slide.pos) && $slide.pos == 'quiqqer-bricks-promoslider-slide-right'}
    {assign var=extraClass value=" quiqqer-bricks-promoslider-wallpaper-text__right"}
    {/if}

    <div class="quiqqer-bricks-promoslider-wallpaper-image" style="display: none">&nbsp;</div>

    {if $slide.title != '' || $slide.text != ''}
    <div class="grid-container">
        <div class="quiqqer-bricks-promoslider-wallpaper-text{$extraClass}">
            {if $Utils->isImage($slide.title) || $slide.title != ''}
            <header>
                {if $Utils->isImage($slide.title)}
                {image image=$slide.title}
                {else}
                {$slide.title}
                {/if}
            </header>
            {/if}

            {if $slide.text != ''}
            <div class="quiqqer-bricks-promoslider-wallpaper-text-description">
                {$slide.text}
            </div>
            {/if}
        </div>
    </div>
    {/if}

    &nbsp;
    </a>
</li>
{/foreach}

(https://dev.quiqqer.com/quiqqer/package-bricks/blob/dev/src/QUI/Bricks/Controls/Slider/PromosliderWallpaper.html)

In Chrome sind aber mehrere zu sehen (einige dazu leer):

Ist das gewollt @Panni?

Ich habe nur den Baustein \QUI\Bricks\Controls\Slider\PromosliderWallpaper getestet, aber gehe ich davon aus, dass bei anderen die gleiche Probleme / Fragen (1,2 und 3) auftauchen können.

reopened

unassigned @Panni

assigned to @Panni

mentioned in commit 965615eb

@michael.daniel
Sollte jetzt auf dem dev-Branch gefixt sein.
Bitte einmal testen.
Wenn alles passt dann den Issue bitte schließen.

assigned to @michael.daniel and unassigned @Panni

mentioned in issue #102 (closed)

getestet auf dev - funktioniert wie gewünscht ;)

closed