fix: taskbar icon not pushed via composer/persona

@antosik, partially reverted your PR:
342503e07a

just specifically this file: 342503e07a/public/src/modules/taskbar.js

I'm not 100% sure, but it might be because this hook is called sooner than the hook being registered? https://github.com/NodeBB/nodebb-theme-persona/blob/master/public/persona.js#L100-L101  @julianlam
v1.18.x
psychobunny 4 years ago
parent ec103ce8dd
commit 3a81c8fd42

@ -1,7 +1,7 @@
'use strict'; 'use strict';
define('taskbar', ['benchpress', 'translator', 'hooks'], function (Benchpress, translator, hooks) { define('taskbar', ['benchpress', 'translator'], function (Benchpress, translator) {
var taskbar = {}; var taskbar = {};
taskbar.init = function () { taskbar.init = function () {
@ -89,7 +89,7 @@ define('taskbar', ['benchpress', 'translator', 'hooks'], function (Benchpress, t
element: element, element: element,
}; };
hooks.fire('filter:taskbar.push', data); $(window).trigger('filter:taskbar.push', data);
if (!element.length && data.module) { if (!element.length && data.module) {
createTaskbarItem(data, callback); createTaskbarItem(data, callback);
@ -116,7 +116,7 @@ define('taskbar', ['benchpress', 'translator', 'hooks'], function (Benchpress, t
btnEl.toggleClass('new', state); btnEl.toggleClass('new', state);
if (!silent) { if (!silent) {
hooks.fire('action:taskbar.toggleNew', uuid); $(window).trigger('action:taskbar.toggleNew', uuid);
} }
}; };
@ -175,7 +175,7 @@ define('taskbar', ['benchpress', 'translator', 'hooks'], function (Benchpress, t
data.element = taskbarEl; data.element = taskbarEl;
taskbarEl.data(data); taskbarEl.data(data);
hooks.fire('action:taskbar.pushed', data); $(window).trigger('action:taskbar.pushed', data);
callback(taskbarEl); callback(taskbarEl);
}); });
} }

Loading…
Cancel
Save