Snaterlicious added a comment. Widgets that aggregate other widgets should encapsulate their aggregated widgets' events using `event.stopPropagation()`. If parent components/widgets are to receive notification, these parent components should rely only on the events of their direct children--anything else implies a logical error. There is no need for events to bubble up any further than their parent component/widget.
Encapsulation is, for example, done in `fingerprintview`, where the propagation of the events of `labelview`, `descriptionview` and `aliasesview` is stopped. Encapsulation should be a general principle to avoid wasting performance for event bubbling that should not take place anyway. TASK DETAIL https://phabricator.wikimedia.org/T78149 REPLY HANDLER ACTIONS Reply to comment or attach files, or !close, !claim, !unsubscribe or !assign <username>. EMAIL PREFERENCES https://phabricator.wikimedia.org/settings/panel/emailpreferences/ To: Snaterlicious Cc: Aklapper, Tobi_WMDE_SW, Snaterlicious, adrianlang, thiemowmde, Wikidata-bugs, aude, GWicke _______________________________________________ Wikidata-bugs mailing list Wikidata-bugs@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs
