search

FriendsOfREDAXO / consent_manager

Stellt ein Opt-In Cookie Banner zur Verfügung
MIT License
57 stars 15 forks source link

Details anzeigen/ausblenden lädt Startseite neu #312

Closed iceman-fx closed 10 months ago

iceman-fx commented 10 months ago

Der Link im Banner: Details anzeigen/ausblenden

lädt aufgrund des /# die Startseite der Webseite neu. Dies sollte abgefangen werden bzw. kein # genutzt werden.

Probleme dabei:

Consent-Manager: 4.2.0 REDAXO: 5.15.1 PHP: 8.2.6 Browser: Chrome

aeberhard commented 10 months ago

Hi Falko, hast Du hier einen Link zur Seite!? Ich denke das liegt an einem anderen Script auf deiner Seite.

https://github.com/FriendsOfREDAXO/consent_manager/blob/60f20214d342ec750c526f1e91c7c96658adb868/assets/consent_manager_frontend.js#L103C30-L103C30

Da ist ein return false; drin

iceman-fx commented 10 months ago

Ja, hierüber z.B.: https://www.provitus.de/leistungen/

Evtl. macht es aber Sinn, wenn kein "href" in diesem Link drin wäre.

IngoWinter commented 10 months ago

Irgendwas namens "plyr" funkt dazwischen, kein CM Fehler

Bildschirmfoto 2024-01-03 um 12 34 39

aeberhard commented 10 months ago

Ja Du hast da ja reichlich JS drin, und CM arbeitet richtig. Nimm mal plyr raus

iceman-fx commented 10 months ago

PLYR ist das PLYR-Addon für die Videoausgabe. Das wird benötigt.

iceman-fx commented 10 months ago

Ich habe PLYR mal kurz testweise deaktiviert. Dann klappt es, ist aber leider nicht die Lösung.

Beides sind Rex-Addons und werden benötigt.

IngoWinter commented 10 months ago

Mach ein Issue bei plyr auf. Das reagiert auf einen Klick, auf den es nicht zu reagieren hat.

iceman-fx commented 10 months ago

Man könnte aber auch im CM den Link ändern auf: <a id="consent_manager-toggle-details" class="icon-info-circled" tabindex="0">Details anzeigen/ausblenden</a>

Dann klappt es auch ;-)

IngoWinter commented 10 months ago

Gegenvorschlag: du lernst mit den Dev Tools umzugehen und machst das nächste Issue direkt im passenden Repo auf - dann klappt es auch ;-)

aeberhard commented 10 months ago

# Wurde soweit ich mich erinnern kann extra aufgenommen da ein Link ohne href von irgendeinem Google-Tool oder so angemeckert wurde. # im href ist ja auch nicht das Problem.

@iceman-fx Du kannst natürlich das fragment ändern und z.B, im Projekt-Addon ablegen.

Der Fehler liegt aber am plyr-Addon und nicht am CM.

olien commented 10 months ago

Warum nicht so?

<a href="javascript:void(0)" id="consent_manager-toggle-details">Detaila</a>

iceman-fx commented 10 months ago

@olien Auch ein Möglichkeit. Wird aber sicher nicht kommen ;-)

olien commented 10 months ago

Kannst du ja im eigenen Fragment nutzen…

IngoWinter commented 10 months ago

Kinders, hier ist das entsprechende Issue: https://github.com/FriendsOfREDAXO/consent_manager/issues/313 Das plyr Addon verursacht den Fehler - gebt doch dort gute Ratschläge ;)