evolution shouldn't use notifications with actions

Hi,

"This should be replaced by the message status menu."

It's a bit difficult to know what to do here without a specification for this.

Is there something we can do with evolution in Jaunty in the meantime? Is this something which
will land for jaunty? Is there something we can do that will work whether we have the message
status menu or not?

Thanks,

James

https://wiki.ubuntu.com/NotifyOSD is available now and has details

This proposed patch will check if actions are supported, and only display the action if it is.

thank you for your work there, some comments though:

- would it be possible to have a libnotify call to query capabilities rather than having to implement those can_support_actions() in each softwares?
- are urgency and timeout equivalent to actions in regard of the spec? ie you check for actions support but also stop using those is that right or should you check for other capabiliities?
- could you open an upstream bug?

> - would it be possible to have a libnotify call to query capabilities rather than having to implement those
> can_support_actions() in each softwares?

Well, that'd require changing libnotify, which we currently don't do.

timeout shouldn't be affected, since it just uses DEFAULT anyway.

> - could you open an upstream bug?

ACK.

Patch looks okay to me, although conditionalizing the timeout seems weird to me. I hope that notify-osd supports non-default, non-zero timeouts?

This bug was fixed in the package evolution - 2.25.92-0ubuntu1

---------------
evolution (2.25.92-0ubuntu1) jaunty; urgency=low

  * New upstream version:
    Bug Fixes:
    - #238346 : Set timezones from component to the calendar first, then add events
    - #333716 : Do not store incomplete or broken files in a cache
    - #404232 : Rebuild view immediately for an advanced search too
    - #404232 : search bar text is set, thus the text will not be empty, which is considered as "no search" these days.
    - #529037 : Disconnect signals on ESource-s too, not only on ECal-s
    - #548623 : No auto-proxy at the moment
    - #550114 : Replaced 'compute_mouse_over' with 'get_mouse_over', now works with cached values 'motion_row', 'motion_col'
    - #550114 : Show proper tooltip in table with grouping columns
    - #551470 : Do not force adding "--
    - #554458 : Make content type from a mime type first
    - #555888 : Do not overwrite user settings with system proxy settings
    - #558366 : Set default timezone to resolve floating DATE-TIME properly
    - #559027 : Do not set date for 'None' value
    - #559719 : Use it's own/unique name for the property on the 'epl' and free the widget only if it wasn't freed yet
    - #561465 : Expand 'Attendee' column instead of the last
    - #563212 : Ensure empty GError before filling it
    - #564229 : Do not use uninitialized variable
    - #564229 : Initialize the EPluginUI registry during class initialization, so that it's sure to be there when we need it
    - #567089 : Do not crash when no From set yet
    - #567949 : Do not free memory before done with it
    - #568176 : Make the migration dialog look better
    - #569700 : Stop reading from a network when operation was canceled
    - #569986 :
    - #570364 :
    - #571625 : Use black/white for foreground instead of active/inactive text color
    - #571721 :
    - #572268 : Do not duplicate "Loading..." node when adding to existing node
    - #572348 : Call gtk_status_icon_set_tooltip_text
    - #572399 : Do not read from invalid iterator after call of row changed
    - #572543 : Always look for alternative apps for application/octet-stream
    - #572903 :
    - #572950 : Fix ordering of -I compiler directives
    - #572962 : Pass e_shell_dbus_initialise
    - #572975 : Destroy file chooser dialog early enough to not have hidden any other windows below it
    - #573198 :
    $566556 : Make it use the proper action.
    Other contributions:
    Build a "GUI" executable.
    Do not use non-zero page size in an adjustment, it's deprecated now.
    Format signature/encryption information nicer.
    Function-declaration compiler warning.
    Migrate from deprecated gtk_window_set_policy to gtk_window_set_resizable
    Remove deprecated gtk_combo_set_case_sensitive
    Remove useless and deprecated gtk_layout_freeze and gtk_layout_thaw calls.
    Substitute some deprecated gtk functions.
  * debian/control:
    - updated evolution-data-server requirement
  * debian/patches/04_autoconf.patch:
    - new version update
  * debian/patches/62_no_upstream_email_notification_by_default.patch:
    - change by Ken VanDine to not enable the notification icon by ...

Reported in upstream bugzilla:

http://bugzilla.gnome.org/show_bug.cgi?id=573919