Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-11-03 04:25:55 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: bug where action:settings.sorted-list.loaded fired early

Browse Source 
The hook was fired before the elements had been added to the DOM.

Also, I updated it to use the new hooks lib.
This commit is contained in:
Julian Lam
2021-01-31 16:57:28 -05:00
parent a8be6fb8fb
commit 1a04ec6419
1 changed files with 14 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
public/src/modules/settings/sorted-list.js
View File

@@ -2,8 +2,9 @@
define('settings/sorted-list', [
'benchpress',
'hooks',
'jquery-ui/widgets/sortable',
], function (benchpress) {
], function (benchpress, hooks) {
var SortedList;
var Settings;
@@ -57,13 +58,14 @@ define('settings/sorted-list', [
form.attr('data-sorted-list-object', key);
$('#content').append(form.hide());
parse($container, itemUUID, item);
parse($container, itemUUID, item).then(() => {
hooks.fire('action:settings.sorted-list.loaded', { element: $list.get(0) });
});
});
}
});
$list.sortable().addClass('pointer');
$(window).trigger('action:settings.sorted-list.loaded', { element: $list.get(0) });
},
};
@@ -113,13 +115,16 @@ define('settings/sorted-list', [
var $list = $container.find('[data-type="list"]');
var itemTpl = $container.attr('data-item-template');
app.parseAndTranslate(itemTpl, data, function (itemHtml) {
itemHtml = $(itemHtml);
$list.append(itemHtml);
itemHtml.attr('data-sorted-list-uuid', itemUUID);
return new Promise((resolve) => {
app.parseAndTranslate(itemTpl, data, function (itemHtml) {
itemHtml = $(itemHtml);
$list.append(itemHtml);
itemHtml.attr('data-sorted-list-uuid', itemUUID);
setupRemoveButton($container, itemUUID);
setupEditButton($container, itemUUID);
setupRemoveButton($container, itemUUID);
setupEditButton($container, itemUUID);
resolve();
});
});
}