Skip to content

Conversation

@tpikonen
Copy link
Contributor

@tpikonen tpikonen commented Nov 9, 2020

Add 'Always show New episodes' toggle to View menu

Fixes #468.

Adds a new boolean config item ui.gtk.episode_list.always_show_new, which can be toggled from menu with View->Always show New episodes.

If set, the Downloaded episodes view also shows new episodes.

@elelay
Copy link
Member

elelay commented Nov 9, 2020

Thanks for the patch. See #468 (comment) for proposal to add a checkbox "always show new episodes" in combination with this.
Could you add this checkbox?

@tpikonen tpikonen force-pushed the fix-view-downloaded branch from bf9fa8c to 16c65e7 Compare November 9, 2020 13:32
@tpikonen
Copy link
Contributor Author

tpikonen commented Nov 9, 2020

Ok, I implemented the checkbox. It works for me but please test.

Copy link
Member

@elelay elelay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't tested it yet but it looks fine!
Just change the one remark.

I want to push a new release this week and already polled translators for updates so I won't merge your PR just now.
It will be merged next week, though.

Thanks,

tooltip.append(_('New episode'))
view_show_downloaded = True
if self._config.ui.gtk.episode_list.always_show_new:
view_show_downloaded = True
Copy link
Member

@elelay elelay Nov 9, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replace by view_show_downloaded = self._config.ui.gtk.episode_list.always_show_new
Maybe store self._config.ui.gtk.episode_list.always_show_new in a variable at the start of the loop.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, I've pushed a fixed version.

@tpikonen tpikonen force-pushed the fix-view-downloaded branch from 16c65e7 to d9ada60 Compare November 9, 2020 15:47
@tpikonen tpikonen requested a review from elelay November 10, 2020 08:09
Copy link
Member

@elelay elelay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perfect, thanks.
Just make it checked by default for backward compatibility.

Adds a new boolean config item ui.gtk.episode_list.always_show_new,
which can be toggled from menu with View->Always show New episodes.

If set, the Downloaded episodes view also shows new episodes.
@tpikonen tpikonen force-pushed the fix-view-downloaded branch from d9ada60 to b73282b Compare November 10, 2020 20:55
@elelay
Copy link
Member

elelay commented Nov 11, 2020

Great, thanks!
Will be merge as is

@elelay elelay merged commit 3eac29f into gpodder:master Nov 24, 2020
@elelay
Copy link
Member

elelay commented Nov 24, 2020

Merged, thanks

@tpikonen tpikonen deleted the fix-view-downloaded branch December 12, 2020 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add option to hide New Episodes

2 participants