Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

longstanding editor camera annoyances #1060

Closed
kfarr opened this issue Feb 7, 2025 · 4 comments
Closed

longstanding editor camera annoyances #1060

kfarr opened this issue Feb 7, 2025 · 4 comments

Comments

@kfarr
Copy link
Collaborator

kfarr commented Feb 7, 2025

things eating away at me w/r/t interface usability

  1. "close up need to zoom a lot to move" -- as a user when i zoom in to an object and wish to keep zooming in more, i want the camera to continue moving forward therefore the rotation point of the viewport moves forward

    • idea: when below certain level of zoom, the z just starts moving forward instead of asymptotically approaching a limit. i think this is the sketchfab method?
  2. click on initial load can, in rare cases, mistakingly move an entity, and for some reason undo doesn't work only in this case

    • perhaps remove the diamond transform gizmo of death as a starting point
  3. the first click to rotate the camera view usually results in a small "jump" as though the pivot point is suddenly changed -- why and how to fix?

    • should we make rotation pivot point clearer in the ui? compare with blender which is very smooth in this regard
@vincentfretin
Copy link
Collaborator

point 3 fixed in #1061

@kfarr
Copy link
Collaborator Author

kfarr commented Feb 7, 2025

Yes the fix for 3 looks good, just merged!

For 1, it looks like if the distance between the requested zoom-in distance (delta) is greater than the distance between the camera and the selected object (distance) the zooming action is canceled in line 140:
https://github.com/3DStreet/3dstreet/blob/main/src/editor/lib/EditorControls.js#L135

Removing line 140 sort of results in the action we want:

  • It will allow the user to keep zooming in beyond the selected objected and continue in the same direction

However, since we're multiplying distance * zoomSpeed, the movement is very slow when you're closest to the target object center and grows as you move further away. Perhaps instead we could use a constant speed that is not dependent upon distance when zooming "past" the target object?

@vincentfretin
Copy link
Collaborator

vincentfretin commented Feb 10, 2025

PR #1067 is my take for point 1, what do you think? This includes some of the things you said.

@kfarr
Copy link
Collaborator Author

kfarr commented Feb 12, 2025

This was/is great, thanks Vincent! I'll close this and use a posthog recording the next time I encounter point 2 and then make a new ticket

@kfarr kfarr closed this as completed Feb 12, 2025
@github-project-automation github-project-automation bot moved this from To Do - High Prio to Done in 3DStreet Dev Tracking Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

2 participants