V4.9: User dialog in getDoi.js without jQuery-ui#1414
V4.9: User dialog in getDoi.js without jQuery-ui#1414bfalkenstein wants to merge 2 commits intoOPUS4:v4.9from
Conversation
|
Dies soll ein erster Schritt sein, jquery-ui in OPUS komplett zu ersetzen. Statt der bisherigen modalen jQuery-UI-Dialoge in getDoi.js werden nun HTML-dialog verwendet (Fallback: alert()). Die angedachte Library micromodal.min.js wird auch nicht benötigt. |
|
Vielen Dank! Ich habe auf den ersten Blick keine Probleme gesehen, die mich davon abhalten würden die Änderungen zu übernehmen. Bei einem längeren Blick findet man immer etwas was noch verbessert werden könnte, aber ich sehe im Augenblick keinen Bedarf. Wenn wir so jQuery-UI loswerden können, ohne uns eine neue Abhängigkeit einzufangen, wäre das erst einmal sehr gut. Wenn am Ende immer noch Abhängigkeiten auf jQuery bestehen, können wir da auf die neueste Version wechseln. Das müsste man dann mal testen. Ansonsten verlasse ich mich darauf, dass alles nach den Änderungen weiterhin funktioniert. Wir haben ja leider keine automatischen Tests für unseren Javascript Funktionen. Manuell schaue ich mir das höchstens später noch mal an, wenn ich nicht gerade in anderen Issues drin stecke. Solange die Menge an Code überschaubar ist, können wir später immer noch mal drüber gehen bzw. über neue Abhängigkeit nachdenken, die von Vorteil wären. |
Changed component for user dialog in getDoi.js: HTML is now used. jquery-ui is no longer required.