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

Frickl Makeover #5

Open
Rolf-B opened this issue Oct 11, 2021 · 6 comments
Open

Frickl Makeover #5

Rolf-B opened this issue Oct 11, 2021 · 6 comments
Assignees
Labels
Initiative Größere Anregung, muss konkretisiert werden. Ggf. einzele Issues für Details Wer macht mit? Extra attention is needed
Projects

Comments

@Rolf-B
Copy link
Member

Rolf-B commented Oct 11, 2021

Platzierung von Script-Elementen

@Rolf-B Rolf-B added the Bug Etwas funktioniert nicht label Oct 11, 2021
@Rolf-B Rolf-B added this to the Aktivitäten Q4-2021 milestone Oct 11, 2021
@Rolf-B Rolf-B added this to Offen in SelfTodo via automation Oct 11, 2021
@Rolf-B Rolf-B self-assigned this Oct 11, 2021
@Rolf-B Rolf-B added the Wer macht mit? Extra attention is needed label Oct 11, 2021
@Rolf-B
Copy link
Member Author

Rolf-B commented Oct 14, 2021

Aktuell gemeldete Probleme (ohne Bewertung, ob man sie lösen sollte oder nicht):

  • Das Frickl-Ergebnis steckt in einer Data-URL und kann deswegen nicht mit den Entwickler-Tools untersucht werden
  • Der JS-Teil für das Frickl muss als script-Element im stehen, ansonsten zeigt Frickl es im Body UND Head an und es wird doppelt ausgeführt. Das bedingt, dass Event-Beispiele immer mit DOMContentLoaded Eventhandler laufen müssen
    • jsFiddle hatte mal einen Schalter, wo das User-Script geladen werden sollte. Mittlerweile setzen sie es immer ans Ende des Body
  • Es ist nicht möglich, in einem Frickl JS Libraries zu verwenden (z.B. jQuery)
  • Das Frickl-Ergebnis verwendet - im Gegensatz zum Vorschau-Tab - nicht das Selfhtml-Grundlayout. Folge: Vorschau und Frickl-Ergebnis können unterschiedlich aussehen.
    • Idee von Felix: Grundlayout per @import hereinholen
  • Es ist nicht möglich, ein Frickl-Script als ECMAScript-Modul zu schreiben
  • Jürgen: Höhe der Vorschau einstellbar (SetupFrickl in common.js + Beispielvorlage)

@Rolf-B Rolf-B added Initiative Größere Anregung, muss konkretisiert werden. Ggf. einzele Issues für Details and removed Bug Etwas funktioniert nicht labels Oct 14, 2021
@Matthias-Scharwies
Copy link

Style-Bereiche innerhalb eines svg-Elements überschreiben den style-Bereich im head.

SMIL funktioniert nicht im Firefox - in Chromium-Browsern ist alles ok

@Rolf-B
Copy link
Member Author

Rolf-B commented Oct 17, 2021

  1. Das ist Working As Designed, würde ich sagen. Welches Todo im Rahmen eines Frickl-Makeover siehst Du hier?
  2. Was hat die FF-Kompatibilität bei Frickl zu suchen?_

@Matthias-Scharwies
Copy link

Matthias-Scharwies commented Dec 26, 2021

1.Das ist Working As Designed, würde ich sagen. Welches Todo im Rahmen eines Frickl-Makeover siehst Du hier?

Ja. Dann muss es aber auch dokumentiert werden, dass wir eben nur minimal-Beispiele einstellen sollten.

Zu 2. Evtl. findet man den Fehler / Bug und kann ihn umgehen. Andererseits passt es jetzt mit dem Warnhinweis im Wiki. SMIL wird eh durch CSS-Animations verdrängt.

@Matthias-Scharwies
Copy link

2. Was hat die FF-Kompatibilität bei Frickl zu suchen?_

Problem ist ja nicht, dass SMIL im FF nicht geht, sondern dass das Frickl im FF SMIL nicht ausführt.

@Matthias-Scharwies
Copy link

SVG wird vom Frickl gerendert;

In der Code-Darstellung werden schließende Tags allerdings nicht in einer neuen Zeile, sondern direkt nach dem schließenden Kindelement notiert.

@Matthias-Scharwies Matthias-Scharwies moved this from Offen to In Entwicklung in SelfTodo May 17, 2023
@Matthias-Scharwies Matthias-Scharwies moved this from In Entwicklung to Offen in SelfTodo May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Initiative Größere Anregung, muss konkretisiert werden. Ggf. einzele Issues für Details Wer macht mit? Extra attention is needed
Projects
Development

No branches or pull requests

2 participants