Jingga/jsOMS
1
0
Fork 0
mirror of https://github.com/Karaka-Management/jsOMS.git synced 2026-01-21 14:18:40 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Autoremove list elements

Browse Source
This commit is contained in:
Dennis Eichhorn 2018-04-24 22:50:58 +02:00
parent 49cbea14d4
commit e2555e7b0d
2 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
UI/ActionManager.js
View File

@ -91,11 +91,11 @@
const observeConfig = { childList: false, attributes: true, subtree: false };
if (hasSelector) {
this.app.eventManager.attach(e.id + 'childList', function(data) {
this.app.eventManager.attach(e.id + '-childList', function(data) {
const length = data.addedNodes.length;
for (let j = 0; j < length; ++j) {
self.bindListener(data.addedNodes[j].id, listeners[i]);
self.bindListener(data.addedNodes[j].id, listeners[i], true);
}
});
@ -103,7 +103,7 @@
observeConfig.subtree = true;
}
this.app.eventManager.attach(e.id + 'attributes', function(data) {});
this.app.eventManager.attach(e.id + '-attributes', function(data) {});
this.app.uiManager.getDOMObserver().observe(e, observeConfig);
}
};
@ -120,7 +120,7 @@
*
* @since 1.0.0
*/
jsOMS.UI.ActionManager.prototype.bindListener = function(id, listener)
jsOMS.UI.ActionManager.prototype.bindListener = function(id, listener, removable = false)
{
const self = this,
actionLength = listener.action.length;
@ -133,7 +133,7 @@
this.app.eventManager.attach(id + '-' + listener.key + '-' + listener.action[j - 1].key, function (data)
{
self.runAction(id, listener, listener.action[j], data);
}, false, true);
}, removable, true); // todo: make actions removable e.g. in case of child elements getting removed = tag list
}
// todo: the true here is a memory leak since it should be removed at some point?!
// todo: handle onload action right after registering everything. this will be used for onload api calls in order to get content such as lists or models. Maybe in the main application after registering a invoke('onload') should be called if the application wants to execute the onload elements

2
UI/UIManager.js
View File

@ -36,7 +36,7 @@
const length = mutations.length;
for(let i = 0; i < length; i++) {
self.app.eventManager.trigger(mutations[i].target.id + mutations[i].type, 0, mutations[i]);
self.app.eventManager.trigger(mutations[i].target.id + '-' + mutations[i].type, 0, mutations[i]);
}
});
};