When evolution window is active, notifications are still displayed for new mail

[Ken VanDine]

Binary package hint: evolution

Use case testing for the message indicator/notify-osd raised the issue of notifications appearing while evolution is the focused window.

Test Case 7: Evolution Window is active, email received

    * Confirm that a notification does not appear
          o Fail
    * Confirm that the new message appears in the main evolution window
          o Confirmed

More info at https://wiki.ubuntu.com/DesktopTeam/Meeting/JauntyIndicatorAppletTestCases

[Ted Gould (ted)]

Neil, is it possible to block notificaitons when the evolution window is in focus?

[Neil J. Patel (njpatel)]

Sure, this patch should fix it.

[David Barth (dbarth)]

Neil sent a patch for this one.

[Ted Gould (ted)]

I'm not sure why this didn't get updated. But the merge happened and everything. Perhaps because it was under "evolution" and not "evolution-indicator"

[Ken VanDine (ken-vandine)]

This patch does fix the behavior in the general case, but not if you click "Send & Receive". The dialog spawned doesn't appear to be a window from evo_shell and you do get the notification.

[Neil J. Patel (njpatel)]

Attached a patch that uses the _NET_ACTIVE_WINDOW_ hint on the root X window to grab the current active window, and then checks if it's resource_name is "evolution" and, if so, doesn't notify.

The patch adds two new files, but no new dependencies. Seems to work fine after a morning of testing. I've left the old code as a fallback if there is an issue getting the active window.

[Ken VanDine (ken-vandine)]

This patch fixes the problem, can someone create a new release?

[Launchpad Janitor (janitor)]

This bug was fixed in the package evolution-indicator - 0.1.12-0ubuntu1

---------------
evolution-indicator (0.1.12-0ubuntu1) jaunty; urgency=low

  * New upstream version.
    * Patch from Neil Patel to fix LP: #342429 that looks to X to figure
      out if an Evolution window is open before sending notifications.
    * Patch from Timo Jyrinki which fixes LP: #352657 where the plugin
      was using the wrong gettext domain.

 -- Ted Gould <email address hidden> Mon, 06 Apr 2009 12:09:23 -0500