ILIAS icon indicating copy to clipboard operation
ILIAS copied to clipboard

NPM: Add `jQuery`

Open mjansenDatabay opened this issue 1 year ago • 1 comments

This PR adds jQuery as NPM dependency.

Usage:

  • In many components, searching for $( results in 2414 occurrences

Wrapped By:

  • Not applicable

Reasoning:

  • jQuery has been introduced many years ago as a DOM abstraction and utility (XMLHttpRequest, Event Handling etc.) library. Nowadays, many of this functions could be replaced by the "Vanilla JS" API. Nevertheless, I doubt that we could tackle the migration in a timely manner for ILIAS 10. It is unlikely to remove this dependency for all components outside of a big project with according funding and organisational resources.

Maintenance:

  • jQuery is actively maintained and has an existing security policy (https://github.com/jquery/jquery/blob/e06ff08849057cd099365bf43598c8952fe9956d/SECURITY.md).

Important: It might be necessary to also include "jquery-migrate": "^3.0.0",, maybe the developer who introduced this can explain this further.

Links:

  • NPM: https://www.npmjs.com/package/jquery
  • GitHub: https://github.com/jquery/jquery
  • Documentation: https://api.jquery.com/

mjansenDatabay avatar Jan 12 '24 09:01 mjansenDatabay

Jour Fixe, 22 JAN 2024: Dependency accepted for ILIAS 10.

matthiaskunkel avatar Jan 22 '24 12:01 matthiaskunkel

This should have become part of ILIAS 10 via this PR. As always, something might have gone wrong. If you are still missing that package, please ping me.

Kind regards!

klees avatar Jul 09 '24 11:07 klees