Page MenuHome

Resetting default value does not work in context menu in sculpt mode
Open, Confirmed, MediumPublic

Description

System Information
Operating system: Windows 10, 64 bits,
Graphics card: Nvidia GTX 1080 8 gb, Nvidia GTX1070 8 gb

Blender Version
Broken:
blender-2.81.0-git.ce2368cf69f5-windows64

Worked: (optional)

Short description of error
Resetting options in the context menu (right click) of things like brush stroke strength, Autosmooth, plane offset and other parameters does not work at all, but does work through active Tool side panes and top bar. Probably it should work everywhere? If I'm working full screen I would want to be able to reset it in the context menu with my stylus as I sculpt.

Exact steps for others to reproduce the error
1 Create any mesh
2 Enable Sculpt mode
3 Select any brush
4 Right click to invoke context menu
5 Try resetting any of the parameter to default values

Details

Type
Bug

Event Timeline

right! & in other places (properties/top-bar/numeric-pane) Reset to Default Value sets to 0 rather then default
However, In my case most options are disabled

Can confirm this is not working from the context menu.
This is similar to T58341 (in that ui_context_rna_button_active within poll functions invoked for the popover UI elements can't find the actual UI control in the popover)

Regarding default values being not defined well [resetting to zero instead of the "default" value we actually see first]:
I think we had a task somewhere going into detail about our defaults being spread across RNA-defaults, (Tool-)specific init functions (possibly others...DNA?)
Before going in and adding RNA defaults here, I'd like feedback from @Julian Eisel (Severin), @Campbell Barton (campbellbarton) what the recommended way to go here is?

Philipp Oeser (lichtwerk) lowered the priority of this task from Needs Triage by Developer to Confirmed, Medium.Aug 27 2019, 10:30 AM

Reset to Default Value is utterly broken, see T47618. But it seems unrelated to the issue reported here, as it seems to work from regular panel regions.

Reset to Default Value is utterly broken, see T47618.

thx, that was the task I meant.

But it seems unrelated to the issue reported here, as it seems to work from regular panel regions.

Still we have stuff that just has no defaults defined, try resetting any brush radius [will default to 0px ... not 50px]
Question was: do we want to go in and define RNA defines based on what the brush is initialzed with?

There are hundreds of RNA properties without defaults so I'd say it's not worth bothering (even though it's an easy change for each individual one). Then there is D4500 which would completely change the way we handle defaults and IMHO that's the way to go. So no, I wouldn't suggest doing it.

OKi, lets settle the discussion about "bad" defaults (since this seems to be a well known issue with potential solutions even).

Keeping this open for the issue about ui_context_rna_button_active though...