[gpodder-devel] New tray features not in newest gpodder trunk.
- From: jerome.chabod at ifrance.com (Jérôme Chabod)
- Date: Tue, 05 Feb 2008 21:52:45 +0100
Hello Thomas,
I understand the reason of you patch, what I don't understand is what it
brakes Shane gpodder.
Only for the fun:
I would have solve this problem another way (see patch in attachment):
have a show_message method in gPodder class which override the base
method, and either send a systray notification or call the base class
method.
Shane could at least try this patch and tell us if it solves something.
Thomas Perl a ?crit :
> The reason for the patch is that it failed when showing a message from
> other dialogs than the main window (because show_message is in
> GladeWidget and only the subclass "gPodder" has a "tray_icon" and
> "minimized" attribute.
>
> So, when using show_message in the preferences dialog, you can't access
> the "tray_icon" and "minimized" attribute, because they are simply not
> there.
> This should pose no problem, because the only case when a non-main
> window window opens a dialog is when gPodder is not minimized (except
> for when you open the preferences dialog via the tray icon).
>
>
> Thomas
>
> _______________________________________________
> gpodder-devel mailing list
> gpodder-devel at lists.berlios.de
> https://lists.berlios.de/mailman/listinfo/gpodder-devel
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gpodder_r560_no_popup_alternate.patch
Type: text/x-diff
Size: 4513 bytes
Desc: not available
URL:
<https://lists.berlios.de/pipermail/gpodder-devel/attachments/20080205/568370c7/attachment.patch>
Other related posts:
