There is no "Knife" tool in the menu #102187

New Issue

Damian · 2022-10-31T17:21:41+01:00

Damian commented

2022-10-31 17:21:41 +01:00

System Information
Operating system: Windows-10-10.0.19044-SP0 64 Bits
Graphics card: NVIDIA GeForce GTX 1080/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 517.40

Blender Version
Broken: version: 3.3.1, branch: master, commit date: 2022-10-04 18:35, hash: b292cfe5a9
Worked: (newest version of Blender that worked as expected)

Short description of error
[Please fill out a short description of the error here]

I think this is pretty much an old issue, but there is no "Knife" tool in any menu in edit mode.
As far as I understand, the toolbar on the left is just a "visual" convenient way to access some operators, tools, and commands.
But there shouldn't be tools hidden from the menu.

Exact steps for others to reproduce the error
[Please describe the exact steps needed to reproduce the issue]
[Based on the default startup or an attached .blend file (as simple as possible)]

**System Information** Operating system: Windows-10-10.0.19044-SP0 64 Bits Graphics card: NVIDIA GeForce GTX 1080/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 517.40 **Blender Version** Broken: version: 3.3.1, branch: master, commit date: 2022-10-04 18:35, hash: `b292cfe5a9` Worked: (newest version of Blender that worked as expected) **Short description of error** [Please fill out a short description of the error here] I think this is pretty much an old issue, but there is no "Knife" tool in any menu in edit mode. As far as I understand, the toolbar on the left is just a "visual" convenient way to access some operators, tools, and commands. But there shouldn't be tools hidden from the menu. ![image.png](https://archive.blender.org/developer/F13824590/image.png) **Exact steps for others to reproduce the error** [Please describe the exact steps needed to reproduce the issue] [Based on the default startup or an attached .blend file (as simple as possible)]

Damian commented

2022-10-31 17:21:41 +01:00

Added subscriber: @DamianWinnichenko

Pratik Borhade commented

2022-11-01 04:22:20 +01:00

Added subscribers: @ideasman42, @PratikPB2123

Pratik Borhade commented

2022-11-01 04:22:20 +01:00

Changed status from 'Needs Triage' to: 'Needs Developer To Reproduce'

Pratik Borhade commented

2022-11-01 04:22:21 +01:00

Hi, thanks for the report.
Adding operator in the panel requires only one line of code. But I'm not sure whether this is needed. The Thought behind Operator distribution between toolbar and mesh panel is not clear to me. So I'll ask developers for more info (@ideasman42 ?)

index 39684aaf161..d6492ea2d85 100644
    - a/release/scripts/startup/bl_ui/space_view3d.py
+++ b/release/scripts/startup/bl_ui/space_view3d.py
@@ -3849,6 +3849,7 @@ class VIEW3D_MT_edit_mesh(Menu):

   layout.operator("mesh.bisect")
   layout.operator("mesh.knife_project")

+        layout.operator("mesh.knife_tool")

   if with_bullet:
       layout.operator("mesh.convex_hull")```

Hi, thanks for the report. Adding operator in the panel requires only one line of code. But I'm not sure whether this is needed. The Thought behind Operator distribution between toolbar and mesh panel is not clear to me. So I'll ask developers for more info (@ideasman42 ?) - - - ```diff --git a/release/scripts/startup/bl_ui/space_view3d.py b/release/scripts/startup/bl_ui/space_view3d.py index 39684aaf161..d6492ea2d85 100644 - a/release/scripts/startup/bl_ui/space_view3d.py +++ b/release/scripts/startup/bl_ui/space_view3d.py @@ -3849,6 +3849,7 @@ class VIEW3D_MT_edit_mesh(Menu): ``` layout.operator("mesh.bisect") layout.operator("mesh.knife_project") ``` + layout.operator("mesh.knife_tool") ``` if with_bullet: layout.operator("mesh.convex_hull")```

Julien Kaspar commented

2022-11-01 11:46:17 +01:00

Added subscriber: @JulienKaspar

Julien Kaspar commented

2022-11-01 11:46:17 +01:00

@PratikPB2123 All operators should be present in the menus. Even if some are also available in the toolbar as a tool.
But I am also a bit puzzled why this wasn't done already for this operator.

@PratikPB2123 All operators should be present in the menus. Even if some are also available in the toolbar as a tool. But I am also a bit puzzled why this wasn't done already for this operator.

Pratik Borhade commented

2022-11-01 14:25:12 +01:00

@JulienKaspar ,thanks.
Knife operator is instead present in right click context menu (available only in edge select mode)
I think we should remove it from there and add in Mesh panel

@JulienKaspar ,thanks. Knife operator is instead present in right click context menu (available only in edge select mode) I think we should remove it from there and add in Mesh panel

Damian commented

2022-11-01 15:35:00 +01:00

In #102187#1439692, @PratikPB2123 wrote:
@JulienKaspar ,thanks.
Knife operator is instead present in right click context menu (available only in edge select mode)
I think we should remove it from there and add in Mesh panel

huh! My bad! I used to the old naming where we did have "Knife" and "Knife Project tool".
However! There is a bug that the hotkey for that tool is not drawn in the right-click menu

> In #102187#1439692, @PratikPB2123 wrote: > @JulienKaspar ,thanks. > Knife operator is instead present in right click context menu (available only in edge select mode) > I think we should remove it from there and add in Mesh panel huh! My bad! I used to the old naming where we did have "Knife" and "Knife Project tool". However! There is a bug that the hotkey for that tool is not drawn in the right-click menu ![image.png](https://archive.blender.org/developer/F13833012/image.png)

Julien Kaspar commented

2022-11-01 15:37:43 +01:00

Removing anything from the context menus is not necessary. It's good that common operators are exposed there.
But yes we should add it to a header menu.

Removing anything from the context menus is not necessary. It's good that common operators are exposed there. But yes we should add it to a header menu.

Pratik Borhade commented

2022-11-05 14:21:37 +01:00

In #102187#1439737, @JulienKaspar wrote:
Removing anything from the context menus is not necessary. It's good that common operators are exposed there.
But yes we should add it to a header menu.

Didn't see your comment before submitting a patch :p (I've removed it from context menu).
I'm fine to keep it at both places but why is it exposed only in Edge select mode?

> In #102187#1439737, @JulienKaspar wrote: > Removing anything from the context menus is not necessary. It's good that common operators are exposed there. > But yes we should add it to a header menu. Didn't see your comment before submitting a patch :p (I've removed it from context menu). I'm fine to keep it at both places but why is it exposed only in `Edge select mode`?

Julien Kaspar commented

2022-11-05 15:58:52 +01:00

@PratikPB2123 It's very arbitrary that the Knife operator is in the edge context menu. IMO it should be part of all context menus in edit mode, but it can for now stay in the Edge context menu.

I would even go further than that and make more operators accessible in more context menus, but that goes beyond this issue.

@PratikPB2123 It's very arbitrary that the Knife operator is in the edge context menu. IMO it should be part of all context menus in edit mode, but it can for now stay in the Edge context menu. I would even go further than that and make more operators accessible in more context menus, but that goes beyond this issue.

Damian commented

2022-11-08 23:13:35 +01:00

In #102187#1441187, @JulienKaspar wrote:
I would even go further than that and make more operators accessible in more context menus, but that goes beyond this issue.

Yes, please!)

> In #102187#1441187, @JulienKaspar wrote: > I would even go further than that and make more operators accessible in more context menus, but that goes beyond this issue. Yes, please!)

blender-admin commented

2022-11-13 18:18:24 +01:00

This issue was referenced by 6a9b0772fe

This issue was referenced by 6a9b0772fee918e408b6b3fbe7f92ab3de8c9ccb

blender-admin commented

2022-11-13 18:18:24 +01:00

This issue was referenced by 92680674d0

This issue was referenced by 92680674d0b78f477ec8dd68589289e45870791b

blender-admin commented

2022-11-13 18:18:24 +01:00

This issue was referenced by b927cc9ba6

This issue was referenced by b927cc9ba6c7567066f26fd82c95b0059a8cff75