Much of the code for the existing Split-Edge tool has been merged into the new Refine-Mesh command. Hopefully I didn't break Split Edge.
I also fixed some comment typos in UndoState.h.
I was uncertain about a few things:
Should the new command be called "Refine Mesh"? Another option is "Split All Edges". After all, it does exactly the same thing as the Split-Edge tool, but to lots of edges at once.
I inserted the new command just after "Duplicate..." in the Shape menu. Is that the best place?
I wasn't sure what functions needed to be called at the end of OnRefineMesh after ApplyUndoState. Someone should check that.
Should the new command have a keyboard shortcut?
I've forgotten how to stop git from adding lots of spurious commits to the list of commits for a pull request. The only one that matters is commit 56255ea.
Much of the code for the existing Split-Edge tool has been merged into the new Refine-Mesh command. Hopefully I didn't break Split Edge.
I also fixed some comment typos in UndoState.h.
I was uncertain about a few things:
I've forgotten how to stop git from adding lots of spurious commits to the list of commits for a pull request. The only one that matters is commit 56255ea.
This should resolve issue #292 .