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

Fix for right click menu in preview on Mac. #7814

Open
wants to merge 1 commit into
base: sg2/main
Choose a base branch
from

Conversation

iTris666
Copy link
Contributor

@iTris666 iTris666 commented Feb 10, 2023


Purpose of this PR

Fix a bug on Mac where if you display the right click menu on the preview, the preview then keeps scrolling when you move the mouse over it without a button pressed.

Also made sure the preview was resilient when given Nan size ( which currently happens with the overlays ).


Testing status

Describe what manual/automated tests were performed for this PR


Comments to reviewers

Notes for the reviewers you have assigned.

@github-actions
Copy link

It appears that you made a non-draft PR!
Please convert your PR to draft (button on the right side of the page).
See the PR template for more information.
Thank you!

@iTris666 iTris666 requested a review from Nightmask3 February 10, 2023 14:39
@iTris666 iTris666 force-pushed the sg2/fix/preview-menu-mac branch from 255e4e6 to e7c4708 Compare February 10, 2023 16:02
Base automatically changed from sg2/upcoming-staging to sg2/main February 13, 2023 18:57
Copy link
Contributor

@Nightmask3 Nightmask3 left a comment

Choose a reason for hiding this comment

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

LGTM

@iTris666 iTris666 requested a review from stummy February 20, 2023 11:22
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