[Python API] Not working `alert' property of 'bpy.types.UILayout' that contains 'popover' #109778

New Issue

Alex Zhornyak · 2023-07-06T13:56:29+02:00

Alex Zhornyak commented

2023-07-06 13:56:29 +02:00

System Information
Operating system: Windows-10-10.0.22621-SP0 64 Bits
Graphics card: NVIDIA GeForce RTX 3060 Laptop GPU/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 526.86

Blender Version
Broken: version: 3.5.1, branch: blender-v3.5-release, commit date: 2023-04-24 18:11, hash: e1ccd9d4a1d3
Worked: seems to be never worked

'Alert' property of 'bpy.types.UILayout' that contains 'popover' is not displayed

Exact steps for others to reproduce the error

Open file space_outliner.py
SetOUTLINER_PT_filter row layout alert property to True and active property to False

Expected:

We expect row to be of red color because of alert=True and reduced opacity because of acitve=False

Result:

It is not working for row layout that contains popover method

**System Information** Operating system: Windows-10-10.0.22621-SP0 64 Bits Graphics card: NVIDIA GeForce RTX 3060 Laptop GPU/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 526.86 **Blender Version** Broken: version: 3.5.1, branch: blender-v3.5-release, commit date: 2023-04-24 18:11, hash: `e1ccd9d4a1d3` Worked: seems to be never worked **'Alert' property of 'bpy.types.UILayout' that contains 'popover' is not displayed** **Exact steps for others to reproduce the error** 1. Open file `space_outliner.py` 2. Set`OUTLINER_PT_filter` row layout `alert` property to `True` and `active` property to `False` ![image](/attachments/37cfa2c5-237c-4aaa-98ad-f63e0264efd2) **Expected:** We expect row to be of red color because of `alert=True` and reduced opacity because of `acitve=False` **Result:** ![image](/attachments/105f5041-baec-48c8-a653-4d01a24ab40b) It is not working for row layout that contains `popover` method

image.png

98 KiB

image.png

2.2 KiB

Alex Zhornyak added the

labels 2023-07-06 13:56:29 +02:00

Philipp Oeser commented

2023-07-07 11:17:22 +02:00

Can confirm.

Philipp Oeser added

and removed

labels 2023-07-07 11:17:55 +02:00

Philipp Oeser referenced a pull request that will close this issue

2023-07-07 16:47:29 +02:00

UI: experimental support for "alert" property in menus/popups #109828

Philipp Oeser commented

2023-07-07 16:50:45 +02:00

I have put up an experimental patch supporting this, see ##109828.

I assume the purpose is to hint at a problem that is inside those popus/menus, right?
To increase the chance of this being supported, could you elaborate a bit on the usecase you have in mind (why is this important to you?)

I have put up an experimental patch supporting this, see ##109828. I assume the purpose is to hint at a problem that is inside those popus/menus, right? To increase the chance of this being supported, could you elaborate a bit on the usecase you have in mind (why is this important to you?)

Alex Zhornyak commented

2023-07-07 17:29:04 +02:00

Yes, in my addon I have something similar to Outliner panel with objects and I would like to attract user attention when something is filtered. I use asterix for now but it would be more laconic to use the standard alert behaviour

![image](/attachments/1ab8cfd1-c321-47c6-b0ec-445e5b1579ce) Yes, in my addon I have something similar to Outliner panel with objects and I would like to attract user attention when something is filtered. I use asterix for now but it would be more laconic to use the standard alert behaviour

image.png

93 KiB

Pablo Vazquez commented

2023-07-07 17:56:53 +02:00

I use asterix for now but it would be more laconic to use the standard alert behaviour

Please do not use alert for simply showing if a list is filtered or not. That styling is called alert for a reason, to be used when there is action needed from the user or to warn about something destructive (like overriding data), or when things don't work as expected (missing modifier data).

> I use asterix for now but it would be more laconic to use the standard alert behaviour Please do not use `alert` for simply showing if a list is filtered or not. That styling is called `alert` for a reason, to be used when there is action needed from the user or to warn about something destructive (like overriding data), or when things don't work as expected (missing modifier data).

Alex Zhornyak commented

2023-07-07 18:07:37 +02:00

I specially initially didn't mention my case. If we are discussing about color user attraction, I would prefer UIlayout.info, UILayout.warning color schemes. But since they do not exist and we can not manage layout colors, I see only alert property that is available to affect layout.

Ok, but all this is not important to the question of why layout behaviour regarding alert property is different and why it is not mentioned in documentation.

I specially initially didn't mention my case. If we are discussing about color user attraction, I would prefer `UIlayout.info`, `UILayout.warning` color schemes. But since they do not exist and we can not manage layout colors, I see only `alert` property that is available to affect layout. Ok, but all this is not important to the question of why layout behaviour regarding `alert` property is different and why it is not mentioned in documentation.

Alex Zhornyak commented

2023-07-07 18:10:10 +02:00

Let's imagine the case when there will be regular expression inside the filter panel and it will be written with critical error in regular expression

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

[Python API] Not working `alert' property of 'bpy.types.UILayout' that contains 'popover' #109778