[Bug 331571] Re: disable Mail Notification eplugin by default

2009-03-02 Thread Ken VanDine
Propsed patch will check if GDMSESSION=gnome-stracciatella, if that
isn't true it will disable the status icon from both the notification
area and from the config UI for the plugin.

** Attachment added: 331571.patch
   http://launchpadlibrarian.net/23300258/331571.patch

-- 
disable Mail Notification eplugin by default
https://bugs.launchpad.net/bugs/331571
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evolution in ubuntu.

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 331571] Re: disable Mail Notification eplugin by default

2009-03-02 Thread Sebastien Bacher
thank you for your work, why do you return enable directly in
enable_status()?

-- 
disable Mail Notification eplugin by default
https://bugs.launchpad.net/bugs/331571
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evolution in ubuntu.

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 331571] Re: disable Mail Notification eplugin by default

2009-03-02 Thread Ken VanDine
Not intentional, i had some debugging code in there and I missed a line
when deleting it.  I am attaching a new patch without it.

** Attachment added: 331571.patch
   http://launchpadlibrarian.net/23302750/331571.patch

-- 
disable Mail Notification eplugin by default
https://bugs.launchpad.net/bugs/331571
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evolution in ubuntu.

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 331571] Re: disable Mail Notification eplugin by default

2009-03-02 Thread Martin Pitt
Ken, thanks! I like that approach because it doesn't break
stracciatella, and doesn't need gconf schema modifications (which are
unreliable).

So with the default Ubuntu session, this will just hide the evo status
icon?

Great, thanks!

-- 
disable Mail Notification eplugin by default
https://bugs.launchpad.net/bugs/331571
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evolution in ubuntu.

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 331571] Re: disable Mail Notification eplugin by default

2009-03-02 Thread Ken VanDine
Yes, exactly.  It will also hide the status icon portion of the
configuration dialog.

-- 
disable Mail Notification eplugin by default
https://bugs.launchpad.net/bugs/331571
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evolution in ubuntu.

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 331571] Re: disable Mail Notification eplugin by default

2009-03-02 Thread Launchpad Bug Tracker
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 default since
  the message indicator is running (lp: #331571)
  * debian/patches/63_use_actions_only_if_supported.patch:
- change by Ken VanDine to use notification actions only when the running
  notification-daemon support those (lp: #328596)
  * debian/patches/70_1024x600_contact-editor.glade.patch,
debian/patches/71_1024x600_em-account-editor.c.patch
debian/patches/72_1024x600_e-shell-settings-dialog.c.patch
debian/patches/73_1024x600_e-shell-window.c.patch
debian/patches/74_1024x600_e-sidebar.c.patch
debian/patches/75_1024x600_e-task-bar.c.patch
debian/patches/76_1024x600_e-timezone-dialog-1024x600.glade.patch
debian/patches/77_1024x600_mail-config.glade.patch
debian/patches/78_1024x600_startup-wizard.c.patch:
- change by Ying-Chun Liu to work better on small screens (lp: #8629)

 -- Sebastien Bacher seb...@ubuntu.com   Mon, 02 Mar 2009 23:55:35
+0100

** Changed in: evolution (Ubuntu Jaunty)
   Status: Confirmed = Fix Released

-- 
disable Mail Notification eplugin by default
https://bugs.launchpad.net/bugs/331571
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evolution in ubuntu.

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 331571] Re: disable Mail Notification eplugin by default

2009-02-28 Thread Bowmore
 In this case I think it's entirely reasonable to just
 suppress the applet itself, but leave the plugin running.

I don't think this is a good enough solution.

Sure, creating a bridge for bug #1 users is one thing but to adopt to
it's annoying behaviour is not an option. This applet aka alert! makes
no sense nor do the OK and Cancel buttons. So, please make this event
informative, cf a printout job.

-- 
disable Mail Notification eplugin by default
https://bugs.launchpad.net/bugs/331571
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evolution in ubuntu.

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 331571] Re: disable Mail Notification eplugin by default

2009-02-26 Thread Ken VanDine
** Tags added: jaunty-uif

-- 
disable Mail Notification eplugin by default
https://bugs.launchpad.net/bugs/331571
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evolution in ubuntu.

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 331571] Re: disable Mail Notification eplugin by default

2009-02-25 Thread Ken VanDine
Suggested patch

** Attachment added: 331571.patch
   http://launchpadlibrarian.net/23119982/331571.patch

-- 
disable Mail Notification eplugin by default
https://bugs.launchpad.net/bugs/331571
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evolution in ubuntu.

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 331571] Re: disable Mail Notification eplugin by default

2009-02-25 Thread Martin Pitt
We can't integrate this into stracciatella if we change gconf defaults.
Also, changing the default key wouldn't work for people who have
explicitly enabled it in their personal gconf tree.

I see two alternatives:

 * Make the plugin disable/quit itself if the indicator plugin is active (would 
be suitable for upstreaming)
 * Make the plugin disable/quit itself if GNOME_SESSION!=gnome-stracciatella 
(ubuntu specific, can't upstream)

-- 
disable Mail Notification eplugin by default
https://bugs.launchpad.net/bugs/331571
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evolution in ubuntu.

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 331571] Re: disable Mail Notification eplugin by default

2009-02-25 Thread Ken VanDine
My worry with making the plugin disable itself is applications that
might be depending on that plugin being enabled to poll for new mail.
That plugin doesn't just provide the icon, it provides dbus alerts for
new mail.

-- 
disable Mail Notification eplugin by default
https://bugs.launchpad.net/bugs/331571
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evolution in ubuntu.

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


Re: [Bug 331571] Re: disable Mail Notification eplugin by default

2009-02-25 Thread Martin Pitt
Ken VanDine [2009-02-25 14:52 -]:
 My worry with making the plugin disable itself is applications that
 might be depending on that plugin being enabled to poll for new mail.
 That plugin doesn't just provide the icon, it provides dbus alerts for
 new mail.

Ah, I see. So the indicator plugin doesn't provide those, or at least
not the same? In this case I think it's entirely reasonable to just
suppress the applet itself, but leave the plugin running.

-- 
disable Mail Notification eplugin by default
https://bugs.launchpad.net/bugs/331571
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evolution in ubuntu.

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 331571] Re: disable Mail Notification eplugin by default

2009-02-24 Thread Sebastien Bacher
** Changed in: evolution (Ubuntu)
   Importance: Undecided = Low

-- 
disable Mail Notification eplugin by default
https://bugs.launchpad.net/bugs/331571
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evolution in ubuntu.

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 331571] Re: disable Mail Notification eplugin by default

2009-02-23 Thread Ken VanDine
** Tags added: notifications

-- 
disable Mail Notification eplugin by default
https://bugs.launchpad.net/bugs/331571
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evolution in ubuntu.

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 331571] Re: disable Mail Notification eplugin by default

2009-02-19 Thread syldeb35
Well noted for the duplicate.
However, the new plug-in didn't notice if the new mail is not in Inbox. (Auto 
sorted mails)
output in the .xsession-errors: 
** (evolution:9806): DEBUG: Not inbox
and no config options.

-- 
disable Mail Notification eplugin by default
https://bugs.launchpad.net/bugs/331571
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evolution in ubuntu.

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs