picker isn't working on cryptomatte matte id picker #99445

New Issue

michael campbell · 2022-07-04T21:51:35+02:00

michael campbell commented

2022-07-04 21:51:35 +02:00

System Information
Operating system: Windows-10-10.0.19044-SP0 64 Bits
Graphics card: NVIDIA GeForce GTX 1070/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 512.96

Blender Version
Broken: version: 3.3.0 Alpha, branch: master, commit date: 2022-06-27 22:46, hash: 7a44f62bdb
Worked: (newest version of Blender that worked as expected)

add a cryptomatte node (new or legacy), hover over matt id and press E . No picker will appear meaning user must click the + icon.

**System Information** Operating system: Windows-10-10.0.19044-SP0 64 Bits Graphics card: NVIDIA GeForce GTX 1070/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 512.96 **Blender Version** Broken: version: 3.3.0 Alpha, branch: master, commit date: 2022-06-27 22:46, hash: `7a44f62bdb` Worked: (newest version of Blender that worked as expected) add a cryptomatte node (new or legacy), hover over matt id and press E . No picker will appear meaning user must click the + icon. ![image.png](https://archive.blender.org/developer/F13258856/image.png)

michael campbell commented

2022-07-04 21:51:35 +02:00

Added subscriber: @3di

Pratik Borhade commented

2022-07-05 07:31:09 +02:00

Added subscriber: @PratikPB2123

Pratik Borhade commented

2022-07-05 07:31:09 +02:00

Changed status from 'Needs Triage' to: 'Confirmed'

Pratik Borhade commented

2022-07-05 07:31:09 +02:00

Thanks for the report. Shortcut only works when hovering over +/- icons

Germano Cavalcante commented

2023-02-15 17:18:18 +01:00

Different problem, but may be related to #101058

Sergey Sharybin added

and removed

labels 2023-02-15 17:53:30 +01:00

Sergey Sharybin commented

2023-02-15 17:55:49 +01:00

Does not seem to be related. This report is about eyedropper not starting at all and the #101058 is about eyedropper not properly picking up matte id.

As for this report, the way how the cryptomatte eyedropper is implementes is quite horrible and can not support text fields. We need to move it out of the generic eyedropper, which will remove some internal tricketry with invisible properties, as well as will allow using eyedropper from the field itself.

Does not seem to be related. This report is about eyedropper not starting at all and the #101058 is about eyedropper not properly picking up matte id. As for this report, the way how the cryptomatte eyedropper is implementes is quite horrible and can not support text fields. We need to move it out of the generic eyedropper, which will remove some internal tricketry with invisible properties, as well as will allow using eyedropper from the field itself.

Julian Eisel commented

2023-02-15 17:56:36 +01:00

I'd consider this a known limitation, using the eyedropper from the main text button isn't supported.

Problem is that the cryptomatte eyedropper reuses the color eyedropper operator, this is kind of a hack. It doesn't seem difficult to add a separate eyedropper operator. We also have data-block, driver and depth eyedropper operators for example. The button could then even display the eyedropper icon we show in other places (at least when there's no value set yet):

Design wise there is the question how this should work though: Does it always add? Or does it replace? Or can you for example hold Alt while clicking to toggle from add to remove? Sounds like straight forward design decisions though.

I'd consider this a known limitation, using the eyedropper from the main text button isn't supported. Problem is that the cryptomatte eyedropper reuses the color eyedropper operator, this is kind of a hack. It doesn't seem difficult to add a separate eyedropper operator. We also have data-block, driver and depth eyedropper operators for example. The button could then even display the eyedropper icon we show in other places (at least when there's no value set yet): ![](/attachments/1672b60b-2ff6-45b7-b1ff-9e56128dfe2b) Design wise there is the question how this should work though: Does it always add? Or does it replace? Or can you for example hold Alt while clicking to toggle from add to remove? Sounds like straight forward design decisions though.

Screenshot from 2023-02-15 17-54-13.png

4.4 KiB

Sergey Sharybin commented

2023-02-15 18:03:59 +01:00

@JulianEisel I think we should try to implement the feautre properly. It is kinda a bug from expectation point of view -- it doesn't work the same as, for example, bones selection where you can click "E" on the entire field.

I'd give it a try, and only move to known limitation if some big design limitation is encountered.

As for the behavior: doesn't seem like a complicated decision to be made either :)

@JulianEisel I think we should try to implement the feautre properly. It is kinda a bug from expectation point of view -- it doesn't work the same as, for example, bones selection where you can click "E" on the entire field. I'd give it a try, and only move to known limitation if some big design limitation is encountered. As for the behavior: doesn't seem like a complicated decision to be made either :)

👀 1

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

picker isn't working on cryptomatte matte id picker #99445