@ChrisLend The changes requested by @dr.sybren has been addressed and were reviewed in the conversation thread, so this PR is probably ready to land!
I have also added a "Commit message for the…
Thank you very much for the feedback!
Ideally, such cleanups would happen before this PR (first cleanup, then change functionality). In this case I don't think that would be worth the effort,…
Thank you very much for the review and the insights about mesh data 🙏
The query is incorrect as you have pointed out, and have probably lead to the tools not functioning properly in some…
@nathanvegdahl Thank you so much for pointing this out! This is probably related to the boolean check error @dr.sybren has pointed out, and should be fixed in the latest commit 🙏
@ChrisLend Do you think it would be beneficial to create a separate pull request for changing the behavior of clicking in an empty space during weight paint mode (row1 for LCS and RCS)? The…
@Mets I checked the behavior of B and C hotkeys in weight paint mode, and they aligned with what you mentioned 👍 (region selecting with hotkeys doesn't deselect bones while region selecting…
Thank you for spotting these mistakes! Will be careful with comments in future PRs 🙏
I applied the same change to the two functions above (change the question mark to a period) since I encountered them many times when coding for this PR, and were used as it says.
I forgot to mention in the description, but B and C hotkeys will be available with this patch! (the respective keymap from 3d View will be applied automatically now that the selection tools are…
Thank you for the feedback! Will check the RCS behavior and append to the PR description.
@ChrisLend Thank you so much for the review!
And the order of the items in the toolbar should be consistent between vert/face/bone select modes.
This should be addressed in the last…