Page MenuHome
Differential D16395

Fix T102187: Add knife tool in mesh panel
ClosedPublic
Actions

Authored by Pratik Borhade (PratikPB2123) on Nov 5 2022, 2:11 PM.

Details

Reviewers
Campbell Barton (campbellbarton)
Julien Kaspar (JulienKaspar)
Maniphest Tasks
T102187: There is no "Knife" tool in the menu
Commits
rB6a9b0772fee9: Fix T102187: Add knife tool in mesh panel
rB92680674d0b7: Fix T102187: Add knife tool in mesh panel
rBb927cc9ba6c7: Fix T102187: Add knife tool in mesh panel
Summary

Add knife tool option in mesh panel

Diff Detail

Repository
rB Blender

Event Timeline

Pratik Borhade (PratikPB2123) requested review of this revision.Nov 5 2022, 2:11 PM
Pratik Borhade (PratikPB2123) created this revision.
Pratik Borhade (PratikPB2123) edited the summary of this revision. (Show Details)
Pratik Borhade (PratikPB2123) added reviewers: Campbell Barton (campbellbarton), Julien Kaspar (JulienKaspar).
Pratik Borhade (PratikPB2123) added projects: Modeling, User Interface.
Pratik Borhade (PratikPB2123) added a task: T102187: There is no "Knife" tool in the menu.
Julien Kaspar (JulienKaspar) requested changes to this revision.Nov 5 2022, 4:02 PM

Like mentioned in T102187 I don't think we should remove the operator from the context menu.
The vital part of this patch is that ALL operators should be accessible from a menu other than context menus.
Changing and improving the context menus can be a separate topic.

This revision now requires changes to proceed.Nov 5 2022, 4:02 PM
Pratik Borhade (PratikPB2123) updated this revision to Diff 57294.Nov 6 2022, 2:34 PM
  • revert the option removal from context menu
Pratik Borhade (PratikPB2123) edited the summary of this revision. (Show Details)Nov 6 2022, 2:34 PM
Julien Kaspar (JulienKaspar) accepted this revision.Nov 7 2022, 1:54 PM
This revision is now accepted and ready to land.Nov 7 2022, 1:54 PM
Pratik Borhade (PratikPB2123) added a comment.Nov 9 2022, 4:11 AM

Should I wait for Campbell? :)

Julien Kaspar (JulienKaspar) added a comment.Nov 9 2022, 9:46 AM

@Pratik Borhade (PratikPB2123) IMO this is such a minor addition that you can go ahead and commit it.

Campbell Barton (campbellbarton) accepted this revision.Nov 9 2022, 12:45 PM
Closed by commit rBb927cc9ba6c7: Fix T102187: Add knife tool in mesh panel (authored by Pratik Borhade (PratikPB2123)). · Explain WhyNov 13 2022, 6:22 PM
This revision was automatically updated to reflect the committed changes.
Pratik Borhade (PratikPB2123) added a commit: rBb927cc9ba6c7: Fix T102187: Add knife tool in mesh panel.
Pratik Borhade (PratikPB2123) added a comment.Nov 13 2022, 6:29 PM

Thanks :)
Committed the patch to both 3.4, 3.5

Philipp Oeser (lichtwerk) added a commit: rB92680674d0b7: Fix T102187: Add knife tool in mesh panel.Nov 28 2022, 2:55 PM
Philipp Oeser (lichtwerk) added a commit: rB6a9b0772fee9: Fix T102187: Add knife tool in mesh panel.Dec 14 2022, 2:43 PM

Revision Contents
Changeset List

PathSize
release/
scripts/
startup/
bl_ui/
2 lines

Diff 57272

View Options

release/scripts/startup/bl_ui/space_view3d.py

Loading...