Skip to content

cinnamonDBus.js: Port to newer class style and remove use of Lang.bind()#13580

Merged
mtwebster merged 1 commit intolinuxmint:masterfrom
JosephMcc:dbus-cleanup
Mar 4, 2026
Merged

cinnamonDBus.js: Port to newer class style and remove use of Lang.bind()#13580
mtwebster merged 1 commit intolinuxmint:masterfrom
JosephMcc:dbus-cleanup

Conversation

@JosephMcc
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread js/ui/cinnamonDBus.js Outdated
ScreenshotArea(include_cursor, x, y, width, height, flash, filename) {
let screenshot = new Cinnamon.Screenshot();
screenshot.screenshot_area(include_cursor, x, y, width, height, filename,
Lang.bind(this, this._onScreenshotComplete, flash));
Copy link
Copy Markdown
Member

@mtwebster mtwebster Feb 20, 2026

Choose a reason for hiding this comment

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

Mabye get rid of Lang.bind in this file also?

@JosephMcc JosephMcc changed the title cinnamonDBus.js: Port to newer class style cinnamonDBus.js: Port to newer class style and remove use of Lang.bind() Feb 20, 2026
@JosephMcc
Copy link
Copy Markdown
Contributor Author

@mtwebster Done!

@mtwebster
Copy link
Copy Markdown
Member

Need rebase, sorry

@JosephMcc
Copy link
Copy Markdown
Contributor Author

No need to apologize. Comes with the territory. Rebased and force-pushed.

@mtwebster mtwebster merged commit f3d821d into linuxmint:master Mar 4, 2026
4 of 5 checks passed
@JosephMcc JosephMcc deleted the dbus-cleanup branch March 28, 2026 23:35
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