search

KittyCAD / modeling-app

The KittyCAD modeling app.
https://kittycad.io/modeling-app/download
MIT License
411 stars 35 forks source link

Small line segments should keep arrow heads at least. #3966

Closed lf94 closed 1 month ago

lf94 commented 1 month ago

See video:

https://github.com/user-attachments/assets/b000f64f-bc43-4efd-ba63-a22562b28304

It makes it not obvious these are individual line segments.

lf94 commented 1 month ago

Know what I'm already in this area of code, I'll quickly fix it

lf94 commented 1 month ago

Fixed out

Irev-Dev commented 1 month ago

Arrow heads act as the handle te the segment the intention was to hide the handle when the line get's too small, since once they're too small the user starts being more likely to click the wrong thing and get frustrated, the same way that it adding an extra segment when you didn't intend it is frustrating.

Same comment as https://github.com/KittyCAD/modeling-app/issues/3974

In the extreme case where there's 100 segments in a 5x5 pixel space, showing everything would be cluttered and there's no way the user would be able to click on what they actually intended to interact with.

Irev-Dev commented 1 month ago

I'm just want to be cautious, if the current UI is bad, I wouldn't want to fix it with a series of haphazard band aid fixs ya know. and just because this is "fixing" something that was intentional, I worry about it.

lf94 commented 1 month ago

It feels very wrong from a user point of view to just see my lines suddenly merge, I cannot tell if it is 1 line (maybe a curve) or many... When they are too small to accurately move, why would they not zoom in?

lf94 commented 1 month ago

You know what, I'll close. My proposal here was arrowheads but evidently it is not a valid solution. I'll revert the change locally.