Sculpt: issues with new menu operators #105557

New Issue

Closed

opened 2023-03-08 08:12:48 +01:00 by Sun Kim · 8 comments

Sun Kim commented

2023-03-08 08:12:48 +01:00

Contributor

System Information
Operating system: Windows-10-10.0.19045-SP0 64 Bits
Graphics card: Radeon RX 570 Series ATI Technologies Inc. 4.5.0 Core Profile Context 23.2.2.230217

Blender Version
Broken: version: 3.6.0 Alpha, branch: main, commit date: 2023-03-08 01:06, hash: 809a336a3d67
Worked: n/a

Operators were added to the menu with da65b21e2e

Short description of error
Exact steps for others to reproduce the error
With da65b21e2e some of the sculpt tools are added as operators under Sculpt menu at the header. I tried the operators and found these issues.

Rotate and Scale operators show shortcuts (R and S) that are not used for the operators in Sculpt mode. Move operator should not show drag-LeftMouse as a shortcut too imo.
Lasso Add executes Lasso with Difference mode instead of Join mode
Fair Positions and Fair Tangency do not execute as menu operators

**System Information** Operating system: Windows-10-10.0.19045-SP0 64 Bits Graphics card: Radeon RX 570 Series ATI Technologies Inc. 4.5.0 Core Profile Context 23.2.2.230217 **Blender Version** Broken: version: 3.6.0 Alpha, branch: main, commit date: 2023-03-08 01:06, hash: `809a336a3d67` Worked: n/a Operators were added to the menu with da65b21e2e **Short description of error** **Exact steps for others to reproduce the error** With da65b21e2e some of the sculpt tools are added as operators under `Sculpt` menu at the header. I tried the operators and found these issues. - Rotate and Scale operators show shortcuts (`R` and `S`) that are not used for the operators in Sculpt mode. Move operator should not show `drag-LeftMouse` as a shortcut too imo. - Lasso Add executes Lasso with Difference mode instead of Join mode - Fair Positions and Fair Tangency do not execute as menu operators

Sun Kim added the

labels 2023-03-08 08:12:49 +01:00

Sun Kim commented

2023-03-08 08:30:05 +01:00

Author

Contributor

Edited out visibility related issue. This seems like a different issue and related operators were not part of the patch.

Edited out visibility related issue. This seems like a different issue and related operators were not part of the patch.

Iliya Katushenock added the

Sculpt, Paint & Texture

labels 2023-03-08 09:34:31 +01:00

Pratik Borhade commented

2023-03-08 10:20:25 +01:00

Member

Hi, thanks for the report. I can confirm this to some extent.

Lasso Add executes Lasso with Difference mode instead of Join mode

Unable to reproduce this as said. Instead difference mode itself is not working for both box/lasso method (selecting box/lasso trim from toolbar also result triggers this issue). Maybe a separate report is needed for it? (I'll verify the culprit and will open a new ticket if needed)

Hi, thanks for the report. I can confirm this to some extent. > Lasso Add executes Lasso with Difference mode instead of Join mode Unable to reproduce this as said. Instead `difference` mode itself is not working for both box/lasso method (selecting box/lasso trim from toolbar also result triggers this issue). Maybe a separate report is needed for it? (I'll verify the culprit and will open a new ticket if needed)

Pratik Borhade added

Sculpt, Paint & Texture

and removed

Sculpt, Paint & Texture

labels 2023-03-08 10:20:58 +01:00

Pratik Borhade commented

2023-03-08 10:21:35 +01:00

Member

cc @Tarek-Yasser
cc @JosephEagar

cc @Tarek-Yasser cc @JosephEagar

Tarek-Yasser referenced this issue

2023-03-08 17:06:15 +01:00

Sculpt: Add Transform, Trim, and Mesh Filter operators to Sculpt menu #104718

Tarek-Yasser commented

2023-03-08 17:12:23 +01:00

Contributor

I think I already encountered and reported the box/lasso trim/add misbehavior in in a comment on #104718.

IIRC, most of my changes regarding this operator were UI related (Python code mainly) so I wasn't sure if I should go down the rabbit hole of debugging it. Anyway, I'll try investigating/fixing stuff I no one is already working on it.

I think I already encountered and reported the box/lasso trim/add misbehavior in [in a comment on #104718](https://projects.blender.org/blender/blender/pulls/104718#issuecomment-883218). IIRC, most of my changes regarding this operator were UI related (Python code mainly) so I wasn't sure if I should go down the rabbit hole of debugging it. Anyway, I'll try investigating/fixing stuff I no one is already working on it.

Joseph Eagar referenced this issue from a commit

2023-03-10 00:44:28 +01:00

Sculpt: Fix #105557 Bugs in new sculpt menu entries

Joseph Eagar closed this issue

2023-03-10 00:44:29 +01:00

Blender Bot added

and removed

labels 2023-03-10 00:44:30 +01:00

Joseph Eagar commented

2023-03-10 00:45:00 +01:00

Member

I wasn't able to fix the incorrect hotkeys.

I wasn't able to fix the incorrect hotkeys.

Sun Kim commented

2023-03-12 05:07:00 +01:00

Author

Contributor

I wasn't able to fix the incorrect hotkeys.

@PratikPB2123
@JosephEagar

Should I open a new issue for this? In my opinion, even if they are not feasible to fix right now, it should be listed as a known issue on this board at least.

> I wasn't able to fix the incorrect hotkeys. @PratikPB2123 @JosephEagar Should I open a new issue for this? In my opinion, even if they are not feasible to fix right now, it should be listed as a known issue on this board at least.

Tarek-Yasser commented

2023-03-12 09:29:28 +01:00

Contributor

Should I open a new issue for this? In my opinion, even if they are not feasible to fix right now, it should be listed as a known issue on this board at least.

Definitely agree. it's better to know it exists than discover it again later on.

>Should I open a new issue for this? In my opinion, even if they are not feasible to fix right now, it should be listed as a known issue on this board at least. Definitely agree. it's better to know it exists than discover it again later on.

Sun Kim commented

2023-03-16 15:16:27 +01:00

Author

Contributor

I wasn't able to fix the incorrect hotkeys.

@PratikPB2123
@JosephEagar

Should I open a new issue for this? In my opinion, even if they are not feasible to fix right now, it should be listed as a known issue on this board at least.

I went ahead and posted one. #105831

> > I wasn't able to fix the incorrect hotkeys. > > @PratikPB2123 > @JosephEagar > > Should I open a new issue for this? In my opinion, even if they are not feasible to fix right now, it should be listed as a known issue on this board at least. I went ahead and posted one. #105831

Sun Kim referenced this issue

2023-03-16 15:17:11 +01:00

Transform operators in Sculpt menu show unavailable shortcuts #105831

Sign in to join this conversation.

No Label

Animation & Rigging

Asset Browser Project Overview

Automated Testing

Blender Asset Bundle

Dependency Graph

Development Management

EEVEE & Viewport

Images & Movies

Motion Tracking

Nodes & Physics

Pipeline, Assets & IO

Platforms, Builds & Tests

Render & Cycles

Render Pipeline

Sculpt, Paint & Texture

Video Sequencer

Virtual Reality

Blender 2.8 Project

Milestone 1: Basic, Local Asset Browser

Good First Issue

Animation & Rigging

Development Management

EEVEE & Viewport

Nodes & Physics

Pipeline, Assets & IO

Platforms, Builds & Tests

Render & Cycles

Sculpt, Paint & Texture

Needs Info from Developers

Needs Information from User

No Milestone

No project

No Assignees

4 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender#105557

No description provided.