Allow dragging extrusions by dragging the end faces. Add the face poi…#1600
Allow dragging extrusions by dragging the end faces. Add the face poi…#1600phkahler merged 1 commit intosolvespace:masterfrom
Conversation
|
We did a lot of face selection in #1258. I'll look through some of those areas... |
|
Looking at some other code for hover and selection faces are never added to the hoverlist. A variable called "hover" will be set near the end of the function but that is distinct from hoverList. Dragging grabs something from the hoverList, and faces won't be in there. So it looks like HitTestMakeSelection() uses a local "sel" to hold the one item that will be selected on click, but at the end it is transferred to "hover" which is part of the class. hoverList holds everything that is under the mouse pointer so slightly different logic can be used for dragging. This could be cleaned up a bit... Hmmm did I split out the function ChooseFromHoverToDrag() ?? I know I added the "CanBeDragged" function to get click-through working right. |
…cular surfaces create by Lathe or Revolve to drag the previous group line that swept out the surface - this was unexpected but cool. Dragging other faces will need investigation as indicated in the comments.
|
I finally got this! So easy too. The behavior on Lathe and Revolve was total bonus! I'd like to do faces for linked and other groups separately because I think they need more surgery. |
|
This boiled down to a really trivial change after all :-) But the (stupid) macOS CI is failing again :-( Not because of you... just Mac stuff :-( |
2 participants
…nt to the dragged list.
This does not work yet. It seems that ChooseFromHoverToSelect() and ChooseFromHoverToDrag() never recieve face entites. If they don't I'm not sure how faces get selected.