Skip to content

Conversation

@tpikonen
Copy link
Contributor

@tpikonen tpikonen commented Sep 9, 2021

Split from #1129. Save the ui-file with Glade to regularize it, fix the collateral damage and remove deprecated GTK features.

Sets the required GTK version to max 3.16 everywhere and updates README.md to mention this.

Also sets 'transient-for' to parent in podcastdirectory, as that was broken in a previous commit.

@auouymous
Copy link
Member

Splitting would have been nice before I spent a week or two reviewing the original, now I must start over. But I will accept it as it is easier to squash smaller PRs into single commits.

Why is the podcastdirectory commit in this PR if something else broke it?

@tpikonen
Copy link
Contributor Author

Oh, sorry if this made things harder. The purpose of the splitting was of course to make the review easier. You can check that the changes to gpodderpreferences.ui are identical to #1129 and that the rest of the commits are trivial.

The podcastdirectory is here because submitting a one-line PR felt silly at the time. This can be split out if needed.

@auouymous auouymous merged commit 2ded357 into gpodder:master Sep 11, 2021
@auouymous
Copy link
Member

Thanks, merged.

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.

2 participants