notification summary doesn't change for synchronous messages

It has nothing at all to do with libnotify, and I have no idea what the supposed behavior for this should be. The "synchronous" hint given is entirely private to Canonical's Notify-OSD. The standard way to update a notification (notify_notification_update() then resend the notification) isn't supported by notify-send.

None of the synchronous notifications use text, so how Notify OSD handles text of synchronous notifications is undefined. It shouldn't crash or hang, but beyond that it doesn't need to behave in any particular way.

Patch included,
the patch provide a fix to provide the "x-canonical-private-synchronous" to work with notifications different with LAYOUT_ICON_INDICATOR layout. Currently the "x-canonical-private-synchronous" interface only provide this layout to support brightness and volume control.

This patch make following command behave correctly.
#notify-send "RF-status" "enabled" -h string:x-canonical-private-synchronous:1 -h string:image_path:/tmp/test.jpg

Canonical OEM projects need this function to

* enable OSD for enable/disable WiFi or bluetooth with a hotkey
* enable OSD for display switching with a hotkey

For more details, see:

https://bugs.launchpad.net/bugs/1213507
https://bugs.launchpad.net/bugs/1213508

The attachment "notify-osd-lp404658.patch" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

I commented on larsu's patch.

The correct way to address this - staying within notify-osd's design-guidelines - splits up in two alternatives...

1.) If it needs to be a synchonous notification:
 * use either the sync-icon-only or icon-value variant of a synchonous notification
 * ask Design for a dedicated symbolic bluetooth- and wifi-icon that's within the style of notify-osd-icons for enabled and disabled states
 * add these icons to lp:ubuntu/notify-osd-icons
 * see the sync-icon-only or icon-value examples of notify-osd trunk

2.) If it needs to have summary- and body-text:
 * use the icon-summary variant
 * to make it skip ahead in the queue set the urgency to "critical"
 * see the icon-smmary example of notify-osd trunk

I can provide screencasts of both alternatives later today.

Here's the promised screencast...

mpt's comment (https://bugs.launchpad.net/ubuntu/+source/notify-osd/+bug/404658/comments/2) says that "None of the synchronous notifications use text", which is true for current notifications, but looking at the specification (https://wiki.ubuntu.com/NotifyOSD) I haven't found a requirements that Synchronous notification should never have text. In which case, we need to define a behaviour for sync notifications that do.

Switch display

Radio enable/disable

I'd quibble with the wording and spelling (I'd prefer "Wi-Fi is off" rather than "WiFi disabled", etc), but I like the general approach here. It just so happened that none of the original use cases for synchronous notifications involved text. But since text is possible, it should be updateable in the same way as it is for asynchronous notifications.

Fix committed into lp:notify-osd at revision 475, scheduled for release in notify-osd, milestone Unknown

This bug was fixed in the package notify-osd - 0.9.35+14.04.20131209-0ubuntu1

---------------
notify-osd (0.9.35+14.04.20131209-0ubuntu1) trusty; urgency=low

  [ Iain Lane ]
  * Clear the reference when we g_source_remove the timers so we don't
    attempt to remove them again in the dispose function.

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 478
 -- Ubuntu daily release <email address hidden> Mon, 09 Dec 2013 17:23:36 +0000

Hello Iain, or anyone else affected,

Accepted notify-osd into precise-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/notify-osd/0.9.34-0ubuntu2.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

works fine to me with notify-osd/0.9.34-0ubuntu2.1 on Precise.

The verification of the Stable Release Update for notify-osd has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regresssions.

This bug was fixed in the package notify-osd - 0.9.34-0ubuntu2.1

---------------
notify-osd (0.9.34-0ubuntu2.1) precise; urgency=medium

  * src/stack.c: Update contents of a synchronous notification when
    replacing it, thanks Lars! (lp: #404658)
 -- Sebastien Bacher <email address hidden> Wed, 15 Jan 2014 15:17:59 +0100