search

feincms / feincms3-cookiecontrol

BSD 3-Clause "New" or "Revised" License
7 stars 0 forks source link

add conscious embedding functionality #30

Closed yoshson closed 2 years ago

yoshson commented 2 years ago

@matthiask was denkst du zu der Erweiterung?

Ich find es nicht gut gelöst wie die JS Resourcen eingebunden werden. Basiert alles auf dem build.js des Panels, aber wenn ich die Sourcen separiere, dann kann es sein, dass sie mehrfach eingebunden werden.

matthiask commented 2 years ago

Ich finde es hübsch und auch sehr passend zum cookiecontrol Banner.

Fragen tue ich mich über das Naming des Template Tags ({% conscious_embed %} würde reichen?), über mögliche Synergie mit https://github.com/matthiask/feincms3/blob/main/feincms3/embedding.py , und ob die Art und Weise der Provider-Übergabe Sinn macht. Vielleicht könnte anhand der URL oder des Embed-Codes der Provider automatisch rausgesucht werden? Vielleicht wäre ein schlauer Fallback, auf den Inhalt zu verlinken statt ihn einzubinden, wenn der Provider nicht automatisch erkannt werden kann? Die aktuelle Lösung kann nur mit kleinerem Handstand integriert werden mit dem External Plugin (feincms3.plugins.external), deshalb meine ich.

yoshson commented 2 years ago

und ob die Art und Weise der Provider-Übergabe Sinn macht.

Seh ich auch so. Man kann den Provider auch einfacher bestimmen.

Vielleicht wäre ein schlauer Fallback, auf den Inhalt zu verlinken statt ihn einzubinden, wenn der Provider nicht automatisch erkannt werden kann?

Ich weiss nicht genau wie du das meinst. Wie kann ich den Content verlinken? Du meinst das src-Attribut suchen?

Die aktuelle Lösung kann nur mit kleinerem Handstand integriert werden mit dem External Plugin (feincms3.plugins.external), deshalb meine ich.

Würde das mit dem jetzigen Wrapper besser funktionieren? Ich meine, die Render-Funktion für das external plugin muss man ohnehin aufrufen, dann kann man auch den Wrapper davor setzen.

matthiask commented 2 years ago

Ich habs nicht vergessen, will heute noch gründlich reinschauen!

matthiask commented 2 years ago

Das einzige was mir etwas Sorgen macht ist: window.localStorage kann schiessen, zB im Safari im Incognito Mode (wenns mir recht ist), und das wäre ziemlich unschön für die Einbindung.

matthiask commented 2 years ago

Hui, warum schliesst es diese PRs immer mit einer Rejection? Der PR ist gemergt, und noch 1-2 Dinge angepasst (zB das localStorage Zeugs)