Open
Description
As a follow up to #7408 and related PR #7609, there are further improvements to be made:
PR feedback from @pierremtb
- Big nit that's because I'm deep in it atm: this could be moved to queryAst with a little unit test
- same here
- I think same comment to the other one here, we could have an Error returned on error and check with err on it
- Would also move to modifyAst with a unit test
From the slack conversation:
- I think we're still missing pre-select in scene -> Start sketch but that can be another pr no big deal at all
- I guess at some point we could make Start sketch on selection work with faces too
UX improvements:
- Currently right clicking on feature tree and then right clicking on canvas doesn't update the context menu items immediately, need to "left click" first (and vica versa)
- Would be nice to synchronize hover / selection of planes in feature tree <-> engine rendering with highlight on canvas, maybe only during "Sketch no face" mode
Metadata
Metadata
Assignees
Labels
No labels