Results 81 comments of kleonc
trafficstars

So @jarmush pretty much already pinpointed the issue. The problem is in here: https://github.com/godotengine/godot/blob/b29bb11a4cb5ffcca6ddcc4d495656f146bfd22a/scene/resources/navigation_polygon.cpp#L270-L272 When the segment `(r[0], outside_point)` being checked for intersection goes directly through an outline point (e.g....

@jarmush It's feels hackish to me. Even if it fixes your use case I doubt it really fixes the issue. Very likely it just makes a different specific use case...

> ### Minimal reproduction project > > [Scaling problem.zip](https://github.com/godotengine/godot/files/10147272/Scaling.problem.zip) Checked it out, not fully sure what's the issue. But I think you might be talking about the result of `gui/common/snap_controls_to_pixels`...

@MecoCL I'd say the main issue in the MRP is the animation itself. You need to be aware that for `Control` node both (position + size) and (anchors + margins)...

Is the same planned for `MIN`/`MAX`/`CLAMP`/`SIGN`? If no, why doing so for `ABS` but not for the others?

> Fixed #63502 @vonagam Please add this and other similar notes to the PR's description, it will link such issue to be auto closed when this PR would be merged....

If it'll be approved/merged, I think it could be cherry-picked / backported to 3.x as well. @akien-mga If you'd want to wait e.g. for reduz's approval on this but have...

> I guess you should update the OP to clarify the new state (re-fixes the reopened issues). Yeah, I was already on that. :slightly_smiling_face:

I'll try to rebase. :upside_down_face: *Edit: Rebased. I think I haven't changed anything besides the commit name.* Generally the PR looks ok, I'd say the docs need some more clarification....