Dyntopo Relative Detail Size not in keymap #83828

New Issue

Ovionis · 2020-12-16T02:35:03+01:00

Ovionis commented

2020-12-16 02:35:03 +01:00

System Information
Operating system: Windows-10-10.0.19041-SP0 64 Bits
Graphics card: GeForce GTX 770/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 457.30

Blender Version
Broken: version: 2.92.0 Alpha, branch: master, commit date: 2020-12-14 13:57, hash: e3068f38c8

Short description of error
Shift + D is the hot key to adjust the LoD of dyntopo's polycount [bpy.ops.sculpt.set_detail_size / SCULPT_OT_set_detail_size]
This hot key was removed and replaced with [sculpt.dyntopo_detail_size_edit/ SCULPT_OT_dyntopo_detail_size_edit] which only works (and polls) for Constant Detail.

**System Information** Operating system: Windows-10-10.0.19041-SP0 64 Bits Graphics card: GeForce GTX 770/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 457.30 **Blender Version** Broken: version: 2.92.0 Alpha, branch: master, commit date: 2020-12-14 13:57, hash: `e3068f38c8` **Short description of error** `Shift + D` is the hot key to adjust the LoD of dyntopo's polycount [`bpy.ops.sculpt.set_detail_size` / `SCULPT_OT_set_detail_size`] This hot key was removed and replaced with [`sculpt.dyntopo_detail_size_edit`/ `SCULPT_OT_dyntopo_detail_size_edit`] which only works *(and polls)* for Constant Detail.

Ovionis commented

2020-12-16 02:35:03 +01:00

Added subscriber: @Phigon

Philipp Oeser commented

2020-12-16 12:56:19 +01:00

Added subscriber: @lichtwerk

Philipp Oeser self-assigned this 2020-12-16 12:56:19 +01:00

Philipp Oeser commented

2020-12-16 12:56:19 +01:00

Changed status from 'Needs Triage' to: 'Confirmed'

Philipp Oeser commented

2020-12-16 12:56:19 +01:00

I think both methods still have its usecases (set_detail_size for relative as @Phigon pointed out)?

Interestingly 99a7c917ea only changed this for the Blender keymap, the Industy Compatible keymap still has the "old" entry.
Will change both keymaps to have both entries.

I think both methods still have its usecases (`set_detail_size` for relative as @Phigon pointed out)? Interestingly 99a7c917ea only changed this for the Blender keymap, the Industy Compatible keymap still has the "old" entry. Will change both keymaps to have both entries.

blender-admin commented

2020-12-16 13:13:13 +01:00

This issue was referenced by 7f9b6b1dc9

This issue was referenced by 7f9b6b1dc920a6da6b028a5e8c14d7c14aab4555

Ovionis commented

2020-12-16 20:25:26 +01:00

There are 4 Dyntopo detail modes. sculpt.dyntopo_detail_size_edit, explicitly only runs in 1 of them (constant).

If you place the operator back into the keymap Shift + D, and just place it under the constant operator, they will both work as intended.
Constant will run on constant, and when the user isn't using constant, it will fail the poll and fallback to the relative operator, which works with all 4 modes.

If you separate them into 2 keys, the only benefit would be the user always has access to both operators. The thing is, Shift D will be empty/useless in the 3/4 cases.

There are 4 Dyntopo detail modes. `sculpt.dyntopo_detail_size_edit`, explicitly only runs in 1 of them *(constant)*. If you place the operator back into the keymap `Shift + D`, and just place it under the constant operator, they will both work as intended. Constant will run on constant, and when the user isn't using constant, it will fail the poll and fallback to the relative operator, which works with all 4 modes. If you separate them into 2 keys, the only benefit would be the user always has access to both operators. The thing is, `Shift D` will be empty/useless in the 3/4 cases.

Philipp Oeser commented

2020-12-17 16:23:14 +01:00

Dont think overloading one shortcut on purpose is allowed / what we want here?
If we want both behaviors on a single shortcut, this would probably have to be solved on the operator level.

But generall: +1 to have it on one shortcut

Dont think overloading one shortcut on purpose is allowed / what we want here? If we want both behaviors on a single shortcut, this would probably have to be solved on the operator level. But generall: +1 to have it on one shortcut

Philipp Oeser commented

2020-12-17 17:41:30 +01:00

I have updated D9871: Sculpt: 'Unifiy' Dyntopo Detail Size operators to have them both under one shortcut now.
(dyntopo_detail_size_edit now just calls set_detail_size internally if not working on constant detail size)

I have updated [D9871: Sculpt: 'Unifiy' Dyntopo Detail Size operators](https://archive.blender.org/developer/D9871) to have them both under one shortcut now. (`dyntopo_detail_size_edit` now just calls `set_detail_size` internally if not working on constant detail size)

slowburn commented

2021-02-03 19:51:13 +01:00

Added subscriber: @slowburn

Philipp Oeser commented

2021-03-09 13:57:49 +01:00

Changed status from 'Confirmed' to: 'Resolved'

Philipp Oeser closed this issue

2021-03-09 13:57:49 +01:00

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Dyntopo Relative Detail Size not in keymap #83828