Grease Pencil. Draw tool pressure - curve widget button missing? #88787

New Issue

Tomasz Kaye · 2021-06-03T12:00:18+02:00

Tomasz Kaye commented

2021-06-03 12:00:18 +02:00

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

Blender Version
Broken: version: 3.0.0 Alpha, branch: master, commit date: 2021-05-16 05:49, hash: 0e581f6196

Short description of error
The Blender manual for the Grease Pencil draw tool says:

Use Pressure (pressure sensitivity icon)
Uses stylus pressure to control how strong the effect is. The gradient of the pressure can be customized using the curve widget.

But I see no button (or any other way) to access a curve widget for mapping the draw tool's pressure. Should there be one?

**System Information** Operating system: Windows-10-10.0.19041-SP0 64 Bits Graphics card: GeForce RTX 2060/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 457.30 **Blender Version** Broken: version: 3.0.0 Alpha, branch: master, commit date: 2021-05-16 05:49, hash: `0e581f6196` **Short description of error** The Blender manual for the Grease Pencil draw tool says: > Use Pressure (pressure sensitivity icon) > Uses stylus pressure to control how strong the effect is. The gradient of the pressure can be customized using the curve widget. But I see no button (or any other way) to access a curve widget for mapping the draw tool's pressure. Should there be one?

Tomasz Kaye commented

2021-06-03 12:00:18 +02:00

Added subscriber: @info-27

Philipp Oeser commented

2021-06-03 13:58:39 +02:00

Added subscribers: @antoniov, @lichtwerk

Philipp Oeser commented

2021-06-03 13:58:39 +02:00

Changed status from 'Needs Triage' to: 'Needs User Info'

Philipp Oeser commented

2021-06-03 13:58:39 +02:00

For some reason these were re-organized to only show in the Properties Editor in b571516237

@antoniov: wouldnt it make much more sense to have these in the sidebar as well? What was the reasoning to exclude the curves from there?

For some reason these were re-organized to only show in the Properties Editor in b571516237 ![image.png](https://archive.blender.org/developer/F10155806/image.png) @antoniov: wouldnt it make much more sense to have these in the sidebar as well? What was the reasoning to exclude the curves from there?

Philipp Oeser commented

2021-06-03 13:58:56 +02:00

Changed status from 'Needs User Info' to: 'Needs Developer To Reproduce'

Philipp Oeser commented

2021-06-03 14:09:46 +02:00

This would also show the curve widget in the sidebar:

P2153: T88787_snippet



diff --git a/release/scripts/startup/bl_ui/properties_paint_common.py b/release/scripts/startup/bl_ui/properties_paint_common.py
index f3462dfb35d..f96df8843d9 100644
--- a/release/scripts/startup/bl_ui/properties_paint_common.py
+++ b/release/scripts/startup/bl_ui/properties_paint_common.py
@@ -1235,7 +1235,7 @@ def brush_basic_gpencil_paint_settings(layout, context, brush, *, compact=False)
         row.prop(brush, "size", text="Radius")
         row.prop(gp_settings, "use_pressure", text="", icon='STYLUS_PRESSURE')
 
-        if gp_settings.use_pressure and context.area.type == 'PROPERTIES':
+        if gp_settings.use_pressure:
             col = layout.column()
             col.template_curve_mapping(gp_settings, "curve_sensitivity", brush=True,
                                        use_negative_slope=True)
@@ -1244,7 +1244,7 @@ def brush_basic_gpencil_paint_settings(layout, context, brush, *, compact=False)
         row.prop(gp_settings, "pen_strength", slider=True)
         row.prop(gp_settings, "use_strength_pressure", text="", icon='STYLUS_PRESSURE')
 
-        if gp_settings.use_strength_pressure and context.area.type == 'PROPERTIES':
+        if gp_settings.use_strength_pressure:
             col = layout.column()
             col.template_curve_mapping(gp_settings, "curve_strength", brush=True,
                                        use_negative_slope=True)

This would also show the curve widget in the sidebar: [P2153: T88787_snippet](https://archive.blender.org/developer/P2153.txt) ``` diff --git a/release/scripts/startup/bl_ui/properties_paint_common.py b/release/scripts/startup/bl_ui/properties_paint_common.py index f3462dfb35d..f96df8843d9 100644 --- a/release/scripts/startup/bl_ui/properties_paint_common.py +++ b/release/scripts/startup/bl_ui/properties_paint_common.py @@ -1235,7 +1235,7 @@ def brush_basic_gpencil_paint_settings(layout, context, brush, *, compact=False) row.prop(brush, "size", text="Radius") row.prop(gp_settings, "use_pressure", text="", icon='STYLUS_PRESSURE') - if gp_settings.use_pressure and context.area.type == 'PROPERTIES': + if gp_settings.use_pressure: col = layout.column() col.template_curve_mapping(gp_settings, "curve_sensitivity", brush=True, use_negative_slope=True) @@ -1244,7 +1244,7 @@ def brush_basic_gpencil_paint_settings(layout, context, brush, *, compact=False) row.prop(gp_settings, "pen_strength", slider=True) row.prop(gp_settings, "use_strength_pressure", text="", icon='STYLUS_PRESSURE') - if gp_settings.use_strength_pressure and context.area.type == 'PROPERTIES': + if gp_settings.use_strength_pressure: col = layout.column() col.template_curve_mapping(gp_settings, "curve_strength", brush=True, use_negative_slope=True) ```

Matias Mendiola commented

2021-06-03 14:46:52 +02:00

Added subscriber: @mendio

Matias Mendiola commented

2021-06-03 14:46:52 +02:00

Grease pencil brushes have a lot of curve widgets, not just Radius and Strength (see random settings for example), the decision was to keep as clean as possible the topbar only with the toggle buttons to activate the use of the tablet preassure and put all the curve widget in properties. Those settings are for advanced user and is not something that need tweaking very often.

Tomasz Kaye commented

2021-06-03 15:01:28 +02:00

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

Tomasz Kaye closed this issue

2021-06-03 15:01:28 +02:00

Tomasz Kaye self-assigned this 2021-06-03 15:01:28 +02:00

Tomasz Kaye commented

2021-06-03 15:01:28 +02:00

Thanks for the clarification! i'll close the ticket if everyone's ok with that

Philipp Oeser commented

2021-06-03 15:14:52 +02:00

Was not talking about the top bar, I was refering to the discrepancy between the Properties Editor and the sidebar (they are available in the Properties Editor but not the sidebar -- which I find confusing):

Was not talking about the top bar, I was refering to the discrepancy between the Properties Editor and the sidebar (they are available in the Properties Editor but not the sidebar -- which I find confusing): ![image.png](https://archive.blender.org/developer/F10155913/image.png)

Matias Mendiola commented

2021-06-03 15:27:40 +02:00

I see, in that case it seems to be an omission when all the widgets were moved to properties. @antoniov could you take a look?

Antonio Vazquez commented

2021-06-03 16:56:22 +02:00

@lichtwerk This looks that we missed to add these properties to the panel or that we decided to avoid too many things, don't remember now, but .if @mendio agrees in the UI side, your patch is ok.

Matias Mendiola commented

2021-06-03 18:13:03 +02:00

In #88787#1170426, @antoniov wrote:
@lichtwerk This looks that we missed to add these properties to the panel or that we decided to avoid too many things, don't remember now, but .if @mendio agrees in the UI side, your patch is ok.

Sure, properties and side panel should work the same

> In #88787#1170426, @antoniov wrote: > @lichtwerk This looks that we missed to add these properties to the panel or that we decided to avoid too many things, don't remember now, but .if @mendio agrees in the UI side, your patch is ok. Sure, properties and side panel should work the same

Antonio Vazquez commented

2021-06-03 20:04:18 +02:00

@lichtwerk do you commit the change or I can do it if you can't?

Philipp Oeser commented

2021-06-03 20:24:27 +02:00

will do tomorrow

blender-admin commented

2021-06-04 12:17:02 +02:00

This issue was referenced by 9ba6b64efa

This issue was referenced by 9ba6b64efa3acb941dee58207489bc593449f133

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

Grease Pencil. Draw tool pressure - curve widget button missing? #88787