https://bugs.gpodder.org/show_bug.cgi?id=1050

--- Comment #5 from Thomas Perl <[email protected]> 2010-07-18 20:46:16 BST ---
(In reply to comment #4)
> the patch is not yet addressing those issue:
> * sync with gpodder.net for the disable flag.
> * not bitmap overlap on the field image to display the disable status
> * not tested on every platform (just linux desktop), so for now, I didn't 
> touch
> at the meamo and other stuff...

The name "skip_for_sync" sounds a bit strange. "skip_feed_updates" sounds more
clear (as "sync" sounds more like sync-to-an-MP3-player to me). Then again, we
could make the flag sound more positive: What about "feed_update_enabled"? It
would have a default value of 1/True, and the check in the feed update loop
could look like this:

    if channel.feed_update_enabled:
        # ..update here..

The wording of the checkbox in the dialog might also need some revision, but I
can do that after merging the patch if you want.

Thanks for the patch so far. If you could include the above changes and create
a new patch, that'd be really helpful :)

-- 
Configure bugmail: https://bugs.gpodder.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.
_______________________________________________
gPodder-Bugs mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/gpodder-bugs

Reply via email to