LogicalError / realtime-CSG-for-unity

Realtime-CSG, CSG level editor for Unity
https://realtimecsg.com
MIT License
761 stars 80 forks source link

Remove a brush but the game object is not removed #253

Closed Hisashimaru closed 5 years ago

Hisashimaru commented 6 years ago

Remove a brush while selected the face, the game object is not removed. It's just removing CSGBrush component.

  1. Select a face.
  2. Hit a delete key.

You can get error logs in the different way. I think this is similar problem.

  1. Select a face.
  2. Move mouse cursor over the other face pivot point.
  3. Hit a delete key.
  4. Click the viewport.

gif

Error logs:

Undo objects may not be null.
UnityEditor.Undo:RecordObjects(Object[], String)
RealtimeCSG.EditModeMeshEdit:DoMoveControlPoints(Vector3) (at Assets/Plugins/RealtimeCSG/Editor/Scripts/View/GUI/EditModeGUI/EditModes/EditMode.MeshEdit.cs:646)
RealtimeCSG.EditModeMeshEdit:HandleEvents(Rect) (at Assets/Plugins/RealtimeCSG/Editor/Scripts/View/GUI/EditModeGUI/EditModes/EditMode.MeshEdit.cs:3192)
RealtimeCSG.EditModeManager:OnSceneGUI(SceneView) (at Assets/Plugins/RealtimeCSG/Editor/Scripts/View/GUI/EditModeGUI/EditModeManager.cs:517)
RealtimeCSG.SceneViewEventHandler:OnScene(SceneView) (at Assets/Plugins/RealtimeCSG/Editor/Scripts/Control/Managers/SceneViewEventHandler.cs:54)
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr)
LogicalError commented 6 years ago

Thanks for reporting this, I'll take a look

LogicalError commented 5 years ago

Fix available in 1.533, I will close this ticket once this version is available from the asset store

LogicalError commented 5 years ago

This is available from the asset store, closing this ticket