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

Move Add menu to root #433

Merged
merged 10 commits into from
Jul 15, 2023
Merged

Move Add menu to root #433

merged 10 commits into from
Jul 15, 2023

Conversation

skalarproduktraum
Copy link
Member

@skalarproduktraum skalarproduktraum commented Jan 20, 2022

This PR:

  • Moves the Add menu to the root, sorts the menu
  • Updates SlicingPlane
  • Fixes a bug in SwingMainWindow

@kephale kephale added this to the 1.0.0 milestone Jun 22, 2023
@kephale kephale changed the title UI cleanup Move Add menu to root Jun 23, 2023
@kephale kephale marked this pull request as ready for review June 23, 2023 20:24
@kephale
Copy link
Member

kephale commented Jun 23, 2023

@skalarproduktraum there have been changes to the addVolume API since the original refactor. I am not claiming this is the best API, my adjustments were about making this build because there ended up being a clash with the Java function signatures of addVolume's nullable arguments.

@kephale kephale merged commit 32407ec into master Jul 15, 2023
3 of 4 checks passed
@kephale kephale deleted the feature/ui-cleanup branch July 15, 2023 19:30
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.

None yet

2 participants