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 issue with nsmenu items blocking the GUI #315

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

johnathan-becker
Copy link
Contributor

This MR fixes an issue where when you click item an NSMenuView item you have a frozen UI until you double click a second time, ending the loop.

Copy link
Member

@fredkiefer fredkiefer left a comment

Choose a reason for hiding this comment

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

Looks interesting, I will need to test it myself.

Source/NSMenuView.m Outdated Show resolved Hide resolved
Source/NSMenuView.m Show resolved Hide resolved
@johnathan-becker
Copy link
Contributor Author

@fredkiefer I am not sure why the pipelines are failing I am getting the message It looks like GNUstep backend is not yet installed

@fredkiefer
Copy link
Member

Skipping most of the tests is normal in the pipeline. Maybe we should add a step to download and compile back here?
But your error is another one:

gui/NSBezierPath/basic.m:
Failed file:  basic.m aborted without running all tests!

But then I have no idea, how your change could affect that.

@fredkiefer
Copy link
Member

I found and fixed the reason why the NSBezierPath test failed. If you rebase it should be fixed for you as well.

@johnathan-becker
Copy link
Contributor Author

@fredkiefer I have rebased and all checks have passed now! Thank you.

@fredkiefer fredkiefer merged commit 82717ee into gnustep:master Nov 18, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants