Right now we need to extend the OpModalComponent class in order to get a modal. This leads to modal components that have nothing to do with the modal but passing some configuration to it (closeOnEscape, Begin of the insertioncloseOnOutsideClick). This generatesEnd of the insertion Begin of the deletion closeOnOutsideClick), generatingEnd of the deletion tight coupling, code repetition, Begin of the deletion orEnd of the deletion lack of Begin of the insertionisolation or mixed concerns (UI behaviour (modal), functionality(ie: workpackage button macro)).

End of the insertion Begin of the deletion isolation.

End of the deletion Ideally, the modal shouldn't know anything about the component that it hosts and should provide a clear API to configure, open, close... it through a service. Either way, the components (the one who calls the modal or the one that is rendered on it) shouldn't know anything about the modal and just inject its service to access the functionality. Begin of the insertion

The way it is implemented right now makes it difficult to refactor it. Add a library component (e.g. material) could be a better solution.

How it works:

* A directive (OpModalComponent) containsEnd of the insertion Begin of the deletion

ExtendEnd of the deletion the Begin of the insertionfunctionality
* The component to be rendered in theEnd of the insertion modal Begin of the insertionextends the directive
* A service (OpModalService) creates dynamically the component and injects it into the portal. It also keeps a reference to the instantiated component class that is used to pass data and functionality around.End of the insertion Begin of the deletion so it:End of the deletion

Begin of the insertionTasks:

End of the insertion * Begin of the insertion[ ]End of the insertion Begin of the deletion [x]End of the deletion Accepts a configuration object so there is no need to extend it to customize actions.
* Begin of the insertion[x]End of the insertion Begin of the deletion [ ]End of the deletion Is able to return data when it is closed.