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

NSWindowShouldDragOnGesture on macOS is not working with Signal #7128

Open
2 tasks done
marcolaux opened this issue Jan 9, 2025 · 5 comments
Open
2 tasks done

NSWindowShouldDragOnGesture on macOS is not working with Signal #7128

marcolaux opened this issue Jan 9, 2025 · 5 comments

Comments

@marcolaux
Copy link

Using a supported version?

  • I have searched searched open and closed issues for duplicates.
  • I am using Signal-Desktop as provided by the Signal team, not a 3rd-party package.

Overall summary

With the setting NSWindowShouldDragOnGesture enabled the user is able to drag any window at any cursor position via CTRL+ALT+Mouse1.

This doesn't seem to work with current Signal versions.

Steps to reproduce

  1. defaults write -g NSWindowShouldDragOnGesture -bool true
  2. restart
  3. See that you can drag any window with CTRL+ALT+Mouse1 at any cursor position
  4. See that this doesn't work with Signal

Expected result

it works like other macOS windows

Actual result

the mouse click just gets registered as a normal mouse click

Screenshots

No response

Signal version

Signal Desktop 7.37.0 ⁨production⁩ (Apple silicon)

Operating system

macOS 15.2 (24C101)

Version of Signal on your phone

No response

Link to debug log

No response

@trevor-signal
Copy link
Contributor

@marcolaux appears to be an issue with Electron. Did this work in previous Signal versions?

@marcolaux
Copy link
Author

marcolaux commented Jan 9, 2025

@marcolaux appears to be an issue with Electron. Did this work in previous Signal versions?

It looks like it - thanks for the hint.

It indeed worked in previous versions and I only noticed it recently. I use the gesture regularly.

Other apps have the same issue with newer Electron versions I have been told.

@indutny-signal
Copy link
Contributor

You seem to have most context on the issue, could you report it upstream, please? Thank you!

https://github.com/electron/electron/issues/new?template=bug_report.yml

@marcolaux
Copy link
Author

alright, I leave a link here:

electron/electron#45183

@indutny-signal
Copy link
Contributor

Thank you so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants