-
Improvement
-
Resolution: Unresolved
-
Minor
-
None
-
4.2
-
MOODLE_402_STABLE
Sadly the way in which these were implemented will make this complex. Hopefully not too challenging, just fiddly.
There is only one place in tool_lp which uses moode-core-notification, but that in turn is subclassed by other place, and has a range of weird and wonderful arguments.
I would suggest that we:
- create a new tool_lp/modal
- subclass it where required
- update all uses to use it one-by-one
- remove the dependence on the YUI moodle-core-notification module.