refactor(geo): delegate the close responsability to the widget itself #1751
Conversation
There was a problem hiding this comment.
Quelles sont nos Widgets pour igo2-lib?
Mon souci est qu'on enlève une gestion globale de la fermeture de widget, qui était au niveau applicatif de l'assemblage igo2, pour le mettre spécifiquement sur un seul Widget OgcFilterComponent. L'idée en soit ne me dérange pas mais il faut s'assurer que tous les widgets vont implémenter cette fonctionnalité de fermeture et qu'on s'assure que la fonctionnalité ne soit pas oublier si on créé de nouveau Widget.
|
La méthode d'origine dans l'assemblage ne fermait meme pas le widget, il ne faisait que le cacher... encore pire. Je peux mettre un protected sur close sur la classe widget, mais ca ne change pas le fait que ce n'était pas fonctionnel. |
|
@alecarn on merge? La méthode on close existe déjà et emit deja un cancel event. Le seul widget de la librairie a été ajusté. |
|
N'est plus nécessaire selon les changements fait dans ce PR. infra-geo-ouverte/igo2#1142 |
2 participants
delegate the close responsability to the widget itself
to test with infra-geo-ouverte/igo2#1142