Skip to content

[cinnamon-screenshot@hilyxx] V1.3.3 - Ready for LM23 - #1138

Merged
rcalixte merged 4 commits into
linuxmint:masterfrom
Hilyxx:cinnamon-screenshot
Sep 3, 2026
Merged

[cinnamon-screenshot@hilyxx] V1.3.3 - Ready for LM23#1138
rcalixte merged 4 commits into
linuxmint:masterfrom
Hilyxx:cinnamon-screenshot

Conversation

@Hilyxx

@Hilyxx Hilyxx commented Aug 20, 2026

Copy link
Copy Markdown
Contributor
  • Improve compatibility with cinnamon 6.8 (Remove deprecated require() and module.exports)
  • Improve compatibility with wayland and the new native cinnamon-screenshot application.

Hilyxx added 2 commits August 20, 2026 16:53
* Improve compatibility with cinnamon 6.8 (Remove deprecated require() and module.exports)
revert locale path to align with Cinnamon Spices
@Hilyxx
Hilyxx marked this pull request as draft August 20, 2026 16:49
@Hilyxx

Hilyxx commented Aug 20, 2026

Copy link
Copy Markdown
Contributor Author

I relied on a comment stating that using GLib.get_user_data_dir() could cause a silent failure if the XDG_DATA_HOME variable was modified, so I changed it accordingly to follow Spices.py, which hard-coded the path as locale_inst = f'{home}/.local/share/locale'.
However, I went back and checked, and the code has since been completely refactored and moved to harvester.py, which does use GLib.get_user_data_dir().
So I’m reverting my change and switching it back to the way it was originally. Sorry for these unnecessary changes.

The pull request can now be merged.

@Hilyxx
Hilyxx marked this pull request as ready for review August 20, 2026 17:04
Improve compatibility with wayland and the new native cinnamon-screenshot application.
@rcalixte
rcalixte merged commit 5f3e9cf into linuxmint:master Sep 3, 2026
4 checks passed
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