search

derf / travelynx

Traewelling-inspired journey logger with realtime data
https://travelynx.de
GNU Affero General Public License v3.0
117 stars 18 forks source link

Refactor JavaScript to get rid of jQuery #77

Open nachtjasmin opened 1 year ago

nachtjasmin commented 1 year ago

Wir haben gestern noch darüber gesprochen, jetzt ist's erledigt. upsi. :D

Ich hab den Code allerdings nicht testen können, travelynx wollte bei mir lokal nicht laufen. Da es sich aber fast um eine 1-zu-1 Übersetzung des Codes handelt, sollte alles soweit funktionieren.

Frage: Wie sieht's mit Browser-Support aus? Willst du den Internet Explorer noch unterstützen? Teile der Syntax dürften nicht IE11-kompatibel sein, sofern gewünscht, würd ich das dann noch anpassen.

derf commented 1 year ago

:ablobcatattention: (wann GitHub mit custom emoji? :pleading_face: )

Ne, IE11 brauch ich definitiv nicht, ich teste mal. sehr cool, danke :3

derf commented 1 year ago

Ich stelle erstmal fest, dass JS-dev in travelynx noch nicht so ganz ideal ist. Muss noch ein paar Randfälle jagen, work in progress :)

nachtjasmin commented 1 year ago

es reicht ja fürs Meiste, so ist's nicht ^^ und joa, ließe sich bestimmt optimieren, aber je nach tooling ist das auch wieder maintenance :D

nachtjasmin commented 1 month ago

Hab da nach Zeiten nochmal draufgeschaut. Also: da Materialize von jQuery abhängig ist, würde die komplette Ablösung von jQuery vorerst nicht so einfach möglich sein. Sofern sie weiterhin gewünscht ist, wäre mein Ansatz der Migration folgender:

  1. Ablösung von jQuery im travelynx-eigenen Code
  2. (ggf.) Finden/Schreiben einer Alternative für die Toasts und das Carousel (afaik die einzigen JS-basierten Dinge, die von Materialize verwendet werden)
  3. Ablösung vom JavaScript-Teil von Materialize, zusammen mit jQuery
  4. profit?

Wäre das in deinem Sinne?

derf commented 1 month ago

Die Autocompletion kommt auch noch von Materialize.

Rein technisch sehe ich unter diesen Voraussetzungen wenig Sinn darin, sich die Mühe zu machen, jQuery rauszuwerfen.

Davon ab hab ich durchaus Interesse mehr vanilla JS ohne jQuery zu lernen und baue neue Projekte inzwischen auch vermehrt so.

In dem Sinne: Wenn du Lust hast freue ich mich über Pull Requests die das jQuery aus einzelnen Teilen von travelynx rausziehen (bevorzugt nicht zu groß, damit das reviewen / testen auch handhabbar ist). Wenn du genug anderes um die Ohren hast lassen wir es sein und das Thema ploppt im Zweifelsfall nochmal auf wenn materialize so antiqiert ist dass es Zeit wird davon wegzumigrieren :)

nachtjasmin commented 1 month ago

alrighty, mighty feline! :3 dann schau ich mal, was sich machen lässt und wozu ich Lust habe ^^