-
Notifications
You must be signed in to change notification settings - Fork 18
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
Del key and add commands #492
Conversation
The cylinder looks a bit shitty because its missing lids. scenerygraphics/scenery#556 |
… to work like Box
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I leave the choice to @moreApi, but I favor adding a warning before object deletion (at least for using the delete
key).
src/main/kotlin/sc/iview/Controls.kt
Outdated
@@ -185,6 +184,9 @@ open class Controls(val sciview: SciView) { | |||
h.addKeyBinding("node: move selected one closer or further away", "ctrl scroll") | |||
h.addBehaviour("node: rotate selected one", NodeRotateControl(sciview)) | |||
h.addKeyBinding("node: rotate selected one", "ctrl shift button1") | |||
h.addBehaviour("node: delete selected one", ClickBehaviour { _, _ -> sciview.deleteActiveNode() }) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to warn the user? I think Blender throws up a dialog before deletion. This could be a call to the DeleteObject command and we could have that command contain the dialog warning?
FWIW, I would also be fine with merging this and creating an issue to track the need for deletion warning dialogs. |
I would prefer an undo option instead but that probatly requires it own hackerton^^ |
Accidentally deleting the node you fiddled with the last hour would be the most annoing though... Therefore I vote for havig a warning. |
Do you know by heart how to display such a warning? I am not familiar with that. |
Also I did not use the delete command in the key binding. Maybe that needs also to be fixed. |
Maybe try |
IMHO it is fine to not use the DeleteObject command as well, but then the dialog could be part of |
ok so now the user has to confirm a delete if it is triggerd by a key press. And the button descriptions would make my human interaction professor proud. @kephale if you are also happy you are free to merge. |
Controls: delete key to delete nodes
closes #473
AddCone,AddCylinder,MenuWeights: Add Cylinder and Cone menu entries
closes #487