Unclear Particle Texture Influence Setions #62055

New Issue

Rombout Versluijs · 2019-02-28T17:47:14+01:00

Rombout Versluijs commented

2019-02-28 17:47:14 +01:00

System Information
OSX 10.11.6
Intel Iris Pro 1536

Blender Version
2.80 (sub 45), 779860d34e, 2019-02-22 22:26

The different sections for hair particle influence texture is not very clear in this new layout.

Exact steps for others to reproduce the error

Add object (Mesh)
Add Hair psys
Add texture & switch to texture panel
Locate influence panel
See images below, The header of each section are replaced by adding only to the first option of each section. New users will never understand this.

PS why does it show 2 animation properties?

bl 2.80

bl 2.79

**System Information** OSX 10.11.6 Intel Iris Pro 1536 **Blender Version** 2.80 (sub 45), 779860d34e26, 2019-02-22 22:26 The different sections for hair particle influence texture is not very clear in this new layout. **Exact steps for others to reproduce the error** 1. Add object (Mesh) 2. Add Hair psys 3. Add texture & switch to texture panel 4. Locate influence panel 5. See images below, The header of each section are replaced by adding only to the first option of each section. New users will never understand this. PS why does it show 2 animation properties? bl 2.80 ![Screen Shot 2019-02-28 at 12.40.14.png](https://archive.blender.org/developer/F6731100/Screen_Shot_2019-02-28_at_12.40.14.png) bl 2.79 ![Screen Shot 2019-02-28 at 12.40.06.png](https://archive.blender.org/developer/F6731099/Screen_Shot_2019-02-28_at_12.40.06.png)

Rombout Versluijs commented

2019-02-28 17:47:14 +01:00

Added subscriber: @RomboutVersluijs

Rombout Versluijs commented

2019-02-28 17:48:26 +01:00

According to new design method, it should have sub-panels like Preferences.

Sebastian Parborg commented

2019-03-02 14:11:19 +01:00

Added subscribers: @pablovazquez, @ZedDB

Pablo Vazquez was assigned by Sebastian Parborg

2019-03-02 14:11:19 +01:00

Sebastian Parborg commented

2019-03-02 14:11:19 +01:00

@pablovazquez is it ok as it is now or do you think som changes are in order?

Pablo Vazquez commented

2019-03-02 16:18:37 +01:00

Added subscriber: @WilliamReynish

Pablo Vazquez commented

2019-03-02 16:18:37 +01:00

It is confusing indeed. It needs a mix of subpanels for splitting into categories and fixing the list.

The 2 decorators is one per property (toggle and factor). I think we can get rid of the toggle decorator, it would be hiding functionality but it's less confusing and we already do it in other parts (object transform panel).

I'm a bit short on time at the moment but here's a quick proposal for handling the toggles. It still needs subpanels. @WilliamReynish do you have plans to work on this area sometime soon? I can look into it but only next month.
{F6746097, size=full}

diff --git a/release/scripts/startup/bl_ui/properties_texture.py b/release/scripts/startup/bl_ui/properties_texture.py
index 1761cec8106..bfce390e6b4 100644
--- a/release/scripts/startup/bl_ui/properties_texture.py
+++ b/release/scripts/startup/bl_ui/properties_texture.py
@@ -793,12 +793,19 @@ class TEXTURE_PT_influence(TextureSlotPanel, Panel):
         tex = context.texture_slot
 
         def factor_but(layout, toggle, factor, name):
+            layout.use_property_decorate = False
+
             row = layout.row(align=True)
-            row.active = getattr(tex, toggle)
 
-            row.prop(tex, factor, text=name, slider=True)
+            row.alignment = 'RIGHT'
+            row.label(text=name)
+            row.prop(tex, toggle, text="")
+
             sub = row.row(align=True)
-            sub.prop(tex, toggle, text="")
+            sub.active = getattr(tex, toggle)
+            sub.use_property_decorate = True
+            sub.prop(tex, factor, text="", slider=True)
+
             return sub  # XXX, temp. use_map_normal needs to override.
 
         if isinstance(idblock, ParticleSettings):

It is confusing indeed. It needs a mix of subpanels for splitting into categories and fixing the list. The 2 decorators is one per property (toggle and factor). I think we can get rid of the toggle decorator, it would be hiding functionality but it's less confusing and we already do it in other parts (object transform panel). I'm a bit short on time at the moment but here's a quick proposal for handling the toggles. It still needs subpanels. @WilliamReynish do you have plans to work on this area sometime soon? I can look into it but only next month. {[F6746097](https://archive.blender.org/developer/F6746097/image.png), size=full} ``` diff --git a/release/scripts/startup/bl_ui/properties_texture.py b/release/scripts/startup/bl_ui/properties_texture.py index 1761cec8106..bfce390e6b4 100644 --- a/release/scripts/startup/bl_ui/properties_texture.py +++ b/release/scripts/startup/bl_ui/properties_texture.py @@ -793,12 +793,19 @@ class TEXTURE_PT_influence(TextureSlotPanel, Panel): tex = context.texture_slot def factor_but(layout, toggle, factor, name): + layout.use_property_decorate = False + row = layout.row(align=True) - row.active = getattr(tex, toggle) - row.prop(tex, factor, text=name, slider=True) + row.alignment = 'RIGHT' + row.label(text=name) + row.prop(tex, toggle, text="") + sub = row.row(align=True) - sub.prop(tex, toggle, text="") + sub.active = getattr(tex, toggle) + sub.use_property_decorate = True + sub.prop(tex, factor, text="", slider=True) + return sub # XXX, temp. use_map_normal needs to override. if isinstance(idblock, ParticleSettings): ```

William Reynish commented

2019-03-02 16:34:59 +01:00

This seems nicer indeed, although in the latest pic, everything seems pushed too far to the right?

I will take a look at it, yes.

This seems nicer indeed, although in the latest pic, everything seems pushed too far to the right? I will take a look at it, yes.

William Reynish commented

2019-03-02 17:22:44 +01:00

I had a look at swapping the checkbox over to the left hand side, and got this result:

I could fiddle with it more, but it seems as if the current layout system just doesn't really support this kind of layout well. It would be great to have a built-in way to handle this kind of layout - it's used quite a lot of places throughout Blender.

The way it's currently coded also makes it difficult to use sub-panels, which is annoying. Sub-panels are, internally, completely separate panels, which causes some issues in cases like this.

I had a look at swapping the checkbox over to the left hand side, and got this result: ![Screenshot 2019-03-02 at 17.18.14.png](https://archive.blender.org/developer/F6746584/Screenshot_2019-03-02_at_17.18.14.png) I could fiddle with it more, but it seems as if the current layout system just doesn't really support this kind of layout well. It would be great to have a built-in way to handle this kind of layout - it's used quite a lot of places throughout Blender. The way it's currently coded also makes it difficult to use sub-panels, which is annoying. Sub-panels are, internally, completely separate panels, which causes some issues in cases like this.

Pablo Vazquez commented

2019-03-04 14:28:19 +01:00

Maybe using my approach but making each row actually two columns? So the label is 50%, and slider+checkbox is the other 50%.

William Reynish commented

2019-03-04 14:52:39 +01:00

@pablovazquez Sure, that might work. Will take a look

Pablo Vazquez removed their assignment 2019-11-07 16:07:01 +01:00

Pablo Vazquez commented

2019-11-07 16:07:01 +01:00

Changed status from 'Open' to: 'Archived'

Pablo Vazquez closed this issue

2019-11-07 16:07:01 +01:00

Pablo Vazquez commented

2019-11-07 16:07:01 +01:00

Archiving this since it won't be fixed for 2.81, particles are being deprecated and 2.82 will hopefully be having a different approach for them anyway.

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

Unclear Particle Texture Influence Setions #62055