Fix rectangle-line overlap

Nelarius / imnodes

A small, dependency-free node editor for dear imgui

MIT License

2.01k stars 244 forks source link

Closed potrepka closed 3 years ago

potrepka commented 3 years ago

Previous implementation failed for cases where both endpoints are outside of rectangle. Hard bug to notice when line segments are small.

sonoro1234 commented 3 years ago

Could you please provide image of case failing. I am not able to reproduce.

potrepka commented 3 years ago

Here is one example:

one_example

sonoro1234 commented 3 years ago

Thanks, I see now.

Nelarius commented 3 years ago

Nice work @nspotrepka , and massive props for finding this bug in the first place 👌 I'll take it.