Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Details anzeigen/ausblenden lädt Startseite neu #312

Closed
iceman-fx opened this issue Jan 2, 2024 · 14 comments
Closed

Details anzeigen/ausblenden lädt Startseite neu #312

iceman-fx opened this issue Jan 2, 2024 · 14 comments

Comments

@iceman-fx
Copy link
Member

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:

  • man verlässt die Seite auf welcher man war
  • das Detailfenster öffnet sich nicht, da die Seite neu geladen wird, wenn man nicht bereits auf der Startseite ist.

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

@aeberhard
Copy link
Member

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
Copy link
Member Author

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
Copy link
Member

Irgendwas namens "plyr" funkt dazwischen, kein CM Fehler

Bildschirmfoto 2024-01-03 um 12 34 39

@aeberhard
Copy link
Member

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

@iceman-fx
Copy link
Member Author

iceman-fx commented Jan 3, 2024

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

@iceman-fx
Copy link
Member Author

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
Copy link
Member

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

@iceman-fx
Copy link
Member Author

iceman-fx commented Jan 3, 2024

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
Copy link
Member

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

@aeberhard
Copy link
Member

aeberhard commented Jan 3, 2024

# 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
Copy link
Member

olien commented Jan 4, 2024

Warum nicht so?

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

@iceman-fx
Copy link
Member Author

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

@olien
Copy link
Member

olien commented Jan 4, 2024

Kannst du ja im eigenen Fragment nutzen…

@IngoWinter
Copy link
Member

Kinders, hier ist das entsprechende Issue: #313
Das plyr Addon verursacht den Fehler - gebt doch dort gute Ratschläge ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants