reaflow icon indicating copy to clipboard operation
reaflow copied to clipboard

[BUG] Node padding doesn't seem to be respected.

Open C0DK opened this issue 4 years ago • 2 comments
trafficstars

I am submitting a bug.

Node padding doesn't seem to be respected. i was looking through the sourcecode trying to find the cause of https://github.com/reaviz/reaflow/issues/68

I found this line and realize that it should presumably mention nodePadding not NODE_PADDING.

I might be wrong.

https://github.com/reaviz/reaflow/blob/b0511aefdd78128aac9d47262149b1b71b6a9832/src/layout/utils.ts#L66

C0DK avatar Mar 12 '21 13:03 C0DK

The whole node padding + alignment situation is a little unpolished right now. I personally didn't have a huge need for anything outside of center alignment so didn't go down the path of doing all the different combinations until someone needed them.

If you want to take a stab at implementing it, I'd be happy to help guide you.

amcdnl avatar Mar 12 '21 14:03 amcdnl

I'm very busy at the moment sadly, but will think of it when i have time to help in the OSS space.

C0DK avatar Mar 25 '21 06:03 C0DK