Skip to content

Conversation

@19wintersp
Copy link
Contributor

@19wintersp 19wintersp commented May 7, 2024

Makes the close button on the titlebar close the wake calculator.

I can't build this, so I have no idea if it works; sorry. Functionality works, tests should too

@19wintersp
Copy link
Contributor Author

Totally forgot about this, @AndyTWF would you mind reviewing please?

Still trying to get cross-compilation working, which EuroScope isn't particularly helping with

@AxonC
Copy link
Member

AxonC commented Mar 24, 2025

@19wintersp did you manage to resolve this?

@19wintersp
Copy link
Contributor Author

Forgot about this PR again 😅

@19wintersp did you manage to resolve this?

I never did manage to get it to build on my system, either through Wine or via clang-cl, but that was some time ago. I may give it another go if I find the time or a Windows system—really ought to be quite simple in the latter case, one would expect

@kristiankunc
Copy link
Contributor

kristiankunc commented Oct 27, 2025

@19wintersp please rebase so we get the build check here

@19wintersp
Copy link
Contributor Author

Merge instead of rebase due on mobile and no time

@kristiankunc
Copy link
Contributor

Looks like it just needs clang format and it's good to go

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.

3 participants