Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

soundBox@scollins: Fix sliders for the latest Cinnamon #1218

Merged
merged 7 commits into from
Jul 20, 2024

Conversation

kawashiro
Copy link
Contributor

@kawashiro kawashiro commented Jul 3, 2024

To be fixed: #781

@rcalixte
Copy link
Member

rcalixte commented Jul 3, 2024

@kawashiro Have you tested this on older versions of Cinnamon? We can always create a 5.4 folder and copy new code there. It would then only apply to Cinnamon 5.4+ which I believe is the breaking point here.

@kawashiro
Copy link
Contributor Author

@rcalixte Could you please suggest me a distro with such an old Cinnamon provided? Even the last Debian has 5.6.8 installed. Hope, it would not be a Nix OS or Gentoo :)

Just want to check if it really backward compatible, because regarding the docs, this API was introduced a long time ago

@rcalixte
Copy link
Member

rcalixte commented Jul 3, 2024

@rcalixte Could you please suggest me a distro with such an old Cinnamon provided? Even the last Debian has 5.6.8 installed. Hope, it would not be a Nix OS or Gentoo :)

Linux Mint 20.3 is still supported and in heavy usage. That has Cinnamon 5.2. If it works there, that should be okay.

Just want to check if it really backward compatible, because regarding the docs, this API was introduced a long time ago

We can always merge it now and fix it later if there are issues.

@kawashiro
Copy link
Contributor Author

Yep, I really broke backward compatibility. It really does not work on 5.2. Give me some time to come up with a workaround of that

@rcalixte
Copy link
Member

rcalixte commented Jul 3, 2024

Yep, I really broke backward compatibility. It really does not work on 5.2. Give me some time to come up with a workaround of that

That already exists. We can create a 5.4 directory and put the new contents there.

@kawashiro
Copy link
Contributor Author

Make sense, will return with a fix, I hope, tomorrow

@rcalixte
Copy link
Member

rcalixte commented Jul 3, 2024

Make sense, will return with a fix, I hope, tomorrow

Shout if you need a hand and we can collaborate!

@rcalixte
Copy link
Member

@kawashiro This is phenomenal! I pushed a change to restore the 2.8 folder without symlinks (just in case) and then updated the translation files. This folder likely won't need to be touched going forward. I also renamed the 5.2 directory to 4.6 as that is the earliest version that it should support. (I should've been clearer here.) And the same for 5.6 to 5.4 as that is the boundary for the breaking changes.

Are you interested in taking over as the maintainer for this desklet? This means you would be notified for issues (but not alone).

@rcalixte rcalixte merged commit 92cbd76 into linuxmint:master Jul 20, 2024
1 check passed
@kawashiro
Copy link
Contributor Author

kawashiro commented Jul 20, 2024

@rcalixte thank you for your support

Are you interested in taking over as the maintainer for this desklet?

Let's have a try

rcalixte added a commit to rcalixte/cinnamon-spices-desklets that referenced this pull request Jul 20, 2024
rcalixte added a commit that referenced this pull request Jul 20, 2024
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.

[soundBox@scollins] Volume slider has stopped working
2 participants