1. Shane Caraveo
  2. weave-ext
  3. Issues
Issue #2 resolved

Notifications not displayed

Martin Kreichgauer
created an issue

Notifications don't show up in the statusbar after they have been added with Weave.Notifications.add(). The reason for this is a patch in the WeaveWindow constructor that causes the observers not to be registered:

{{{ if (!this.isPrimaryWindow()) { obs.push(["weave:notification:added", "onNotificationAdded"], ["weave:notification:removed", "onNotificationRemoved"]); } }}}

whereas in the original lib that part is

{{{ if (window.location.href == getBrowserURL()) { obs.push(["weave:notification:added", "onNotificationAdded"], ["weave:notification:removed", "onNotificationRemoved"]); } }}}

Comments (1)

  1. Log in to comment