search

mnater / Hyphenopoly

Hyphenation for node and Polyfill for client-side hyphenation.
http://mnater.github.io/Hyphenopoly/
MIT License
696 stars 44 forks source link

Allow to configure custom loader #197

Closed StephanHoyer closed 2 years ago

mnater commented 2 years ago

Zuerst einmal: Vielen Dank für den PR. Die Stil-Sachen kann ich auch noch selber klären - die sind egal.

Vom Konzept her habe ich leider noch folgenden Einwand:

Mir gefällt nicht, dass der Loader mal mittels eines Strings (fs oder https) und mal mittels einer Funktion konfiguriert werden soll.

Falls es für deinen Use-Case reicht, dass die .wasm-Dateien mittels fetch geladen werden (vgl. dein Beispiel in der Dokumentation), würde ich eine zusätliche Option fetch (neben fs und https) bevorzugen. Das könnte zur gegebener Zeit auch noch durch import ergänzt werden. Klar geben wir damit etwas an Flexibilität preis, würden aber alles einfacher und konsistenter halten.

Was meinst du?

mnater commented 2 years ago

Oder, oooder Wir verlangen immer eine loader-Funktion (gemäss deinem Design) und verzichten ganz auf vorimplementierte IO-Routinen. Das wäre auch konsistent und wir hätten die gewünschte Flexibilität. Das verlangt den Usern des Skripts ein bisschen mehr ab, könnte aber mit ein paar Code-Beispielen in der Dokumentation abgefangen werden...