Resetting default value does not work in popovers or menus #69200

New Issue

Chingiz Jumagulov · 2019-08-27T08:53:11+02:00

Chingiz Jumagulov commented

2019-08-27 08:53:11 +02:00

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

**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. ![image.png](https://archive.blender.org/developer/F7705744/image.png) **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

Chingiz Jumagulov commented

2019-08-27 08:53:11 +02:00

Added subscriber: @MeshVoid

blender-admin commented

2019-08-27 08:53:11 +02:00

#67201 was marked as duplicate of this issue

Abid Maqbool commented

2019-08-27 09:11:49 +02:00

Added subscriber: @AbidMaqbool

Abid Maqbool commented

2019-08-27 09:11:49 +02:00

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

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** ![image.png](https://archive.blender.org/developer/F7705780/image.png)

Philipp Oeser commented

2019-08-27 10:29:48 +02:00

Added subscribers: @JulianEisel, @ideasman42, @lichtwerk

Philipp Oeser commented

2019-08-27 10:29:48 +02:00

Can confirm this is not working from the context menu.
This is similar to #58341 (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 @JulianEisel, @ideasman42 what the recommended way to go here is?

Can confirm this is not working from the context menu. This is similar to #58341 (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 @JulianEisel, @ideasman42 what the recommended way to go here is?

Julian Eisel commented

2019-08-27 12:54:04 +02:00

Reset to Default Value is utterly broken, see #47618. 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 #47618. But it seems unrelated to the issue reported here, as it seems to work from regular panel regions.

Philipp Oeser commented

2019-08-27 13:09:51 +02:00

In #69200#763553, @JulianEisel wrote:
Reset to Default Value is utterly broken, see #47618.

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?

> In #69200#763553, @JulianEisel wrote: > *Reset to Default Value* is utterly broken, see #47618. 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?

Julian Eisel commented

2019-08-27 13:44:09 +02:00

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.

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](https://archive.blender.org/developer/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.

Philipp Oeser commented

2019-08-27 14:13:08 +02:00

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...

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...

Campbell Barton changed title from ~~Resetting default value does not work in context menu in sculpt mode~~ to Resetting default value does not work in popovers or menus

2020-02-20 06:47:01 +01:00

Campbell Barton commented

2020-02-21 02:37:26 +01:00

Added subscriber: @Blendify

Fabrício Luis commented

2020-11-23 03:05:59 +01:00

Added subscriber: @ce3po

Aaron Carlisle commented

2021-04-21 06:58:30 +02:00

This is unrelated to the specifics of painting

Philipp Oeser removed the

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Resetting default value does not work in popovers or menus #69200