Moon-0xff / gnome-mpris-label

A music related GNOME extension.
GNU General Public License v3.0
50 stars 10 forks source link

More cleanup for future compatibility #83

Closed Batwam closed 1 year ago

Batwam commented 1 year ago

Hello, I noticed a couple more minor changes I'd like to make which apply to the gnome 43/44 version too. It's not 100% necessary to include them in main as they can always be part of the gnome45 diff but I though it might be cleaner to merge those first to minimise the differences between the two versions.

The gnome45 version is now ready to go by the way, I'll publish once this is confirmed.

Moon-0xff commented 1 year ago

I'll wait a bit in case there's something else to add.

Also, please check your email inbox.

Batwam commented 1 year ago

Invitation accepted 👍

The gnome45 port is ready to go so I'm not expecting further changes. The commits in this PR were actually already in the previous gnome45 merge request. This is mostly aimed at keeping the gnome 45 PR to the strict minimum but considering the location and nature of the changes, we can certainly keep them within the gnome45 branch/commit/diff.

Moon-0xff commented 1 year ago

Invitation accepted 👍

Thanks!

The gnome45 port is ready to go so I'm not expecting further changes

Alright, then I'll merge this now. I agree it's better to keep the differences between main and the 45 version small.

If the 45 diff is small enough then I think a patch file would be more manageable, though it will require a script and/or some user guidance when installing manually.
I think it's better than keeping a branch that will inevitably drift from main.

Batwam commented 1 year ago

@Moon-0xff Sincere apologies as I just noticed a mistake in the last commit due to copy-paste. Could you please change 'Switch Automatically' with 'Settings' here: https://github.com/Moon-0xff/gnome-mpris-label/pull/83/files#diff-19a9a3efb94bf1b84229bda71ab0ef5badcdbf34ba3a0c9e4152a82c569a21e6R390

Moon-0xff commented 1 year ago

Done