Node Panels Polishing #112506

New Issue

Dalai Felinto · 2023-09-18T12:17:37+02:00

Dalai Felinto commented

2023-09-18 12:17:37 +02:00

Support Ctrl+click to isolate panel.
Support dragging all panels to collapse/expand them.
Improve horizontal padding (#112593)

Don't space out sockets when hiding the node
Bug, hide panel when all its sockets are hidden with Ctrl+H

- [ ] Support Ctrl+click to isolate panel. - [ ] Support dragging all panels to collapse/expand them. - [x] Improve horizontal padding (#112593) --- - [x] Don't space out sockets when hiding the node ![image](/attachments/d5fc1d47-d378-4b73-86fc-c1323cc05a8a) - [ ] Bug, hide panel when all its sockets are hidden with Ctrl+H ![image](/attachments/812ca882-0200-4599-bcd8-1e4627856284)

image.png

66 KiB

image.png

67 KiB

Screenshot_20230919_181456.png

31 KiB

image.png

22 KiB

Dalai Felinto added the

Type

To Do

label 2023-09-18 12:17:37 +02:00

Dalai Felinto added this to the Nodes & Physics project 2023-09-18 12:17:39 +02:00

Simon Thommes commented

2023-09-18 12:42:51 +02:00

Sounds good overall! I just have one concern with supporting A to toggle a panel when hovering over it. In the properties panel A doesn't serve any other purpose. In the node editor A is already used to select all nodes, which is independent of the cursor position.
Making this do something entirely different if you happen to hover your cursor over a panel would introduce a pretty annoying papercut imo.
I personally didn't even know this shortcut existed. It does seem useful, but I'm a bit surprised why it would be on a key like A that is already this loaded in most editors for selection. I guess the issue only appears now as we are mixing panel UI with selectable elements for the first time.

Sounds good overall! I just have one concern with supporting `A` to toggle a panel when hovering over it. In the properties panel `A` doesn't serve any other purpose. In the node editor `A` is already used to select all nodes, which is independent of the cursor position. Making this do something entirely different if you happen to hover your cursor over a panel would introduce a pretty annoying papercut imo. I personally didn't even know this shortcut existed. It does seem useful, but I'm a bit surprised why it would be on a key like `A` that is already this loaded in most editors for selection. I guess the issue only appears now as we are mixing panel UI with selectable elements for the first time.

👍 1

Dalai Felinto commented

2023-09-18 12:59:03 +02:00

Maybe we can skip that one (@pablovazquez ) ^^

👍 1

Nika Kutsniashvili commented

2023-09-18 17:05:04 +02:00

Little bit more space between first panel and regular sockets would be nice to distinguish visually where panel is starting and other sockets end. Especially when zoomed out

Ludvik Koutny commented

2023-09-18 18:06:14 +02:00

Yes, the A shortcut is not a good idea. It would mean the user needs to be aware if mouse cursor is over a node or not to anticipate if pressing A key will toggle all panels or select all nodes. That would probably just cause frustration.

Daybreak commented

2023-09-18 21:54:56 +02:00

I also agree that A changing functionality depending on if you're hovering over a node will get rather annoying.

Otherwise, I have nothing to add except that I agree with the proposals, especially the padding change. I've found that Principled BSDF with all panels expanded appears incredibly cluttered in its current state.

I also agree that `A` changing functionality depending on if you're hovering over a node will get rather annoying. Otherwise, I have nothing to add except that I agree with the proposals, especially the padding change. I've found that Principled BSDF with all panels expanded appears incredibly cluttered in its current state.

Simon Thommes referenced this issue

2023-09-19 19:02:43 +02:00

Hiding Unused Sockets does not remove panels when all sockets are hidden #112588

Pablo Vazquez referenced this issue

2023-09-19 20:09:40 +02:00

UI: improve vertical spacing in node panels #112593

Alaska referenced this issue

2023-09-25 04:25:37 +02:00

Nodes: Node Panels cannot be toggled by mouse dragging as usual panels #112811

dupoxy referenced this issue

2023-10-06 09:09:37 +02:00

Principle BSDF Collapsible Panels don't follow blenders current standard #113318

Iliya Katushenock referenced this issue

2023-10-06 10:46:43 +02:00

Principle BSDF Collapsible Panels don't follow blenders current standard #113318

Adam Earle commented

2023-10-11 06:11:46 +02:00

Sounds good overall! I just have one concern with supporting A to toggle a panel when hovering over it. In the properties panel A doesn't serve any other purpose. In the node editor A is already used to select all nodes, which is independent of the cursor position.
Making this do something entirely different if you happen to hover your cursor over a panel would introduce a pretty annoying papercut imo.
I personally didn't even know this shortcut existed. It does seem useful, but I'm a bit surprised why it would be on a key like A that is already this loaded in most editors for selection. I guess the issue only appears now as we are mixing panel UI with selectable elements for the first time.

Wouldn't it be Alt + H

> Sounds good overall! I just have one concern with supporting `A` to toggle a panel when hovering over it. In the properties panel `A` doesn't serve any other purpose. In the node editor `A` is already used to select all nodes, which is independent of the cursor position. > Making this do something entirely different if you happen to hover your cursor over a panel would introduce a pretty annoying papercut imo. > I personally didn't even know this shortcut existed. It does seem useful, but I'm a bit surprised why it would be on a key like `A` that is already this loaded in most editors for selection. I guess the issue only appears now as we are mixing panel UI with selectable elements for the first time. Wouldn't it be Alt + H

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

Node Panels Polishing #112506