Skip to content

Sensitivity Sliders #1166

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jun 26, 2025
Merged

Sensitivity Sliders #1166

merged 3 commits into from
Jun 26, 2025

Conversation

ariesninjadev
Copy link

@ariesninjadev ariesninjadev commented Jun 25, 2025 •
edited
Loading

This PR adds sensitivity sliders to general settings that allow the user to modify the DPI required to rotate the scene, and to rotate the scene using the Viewcube. The bounds on the sensitivity are higher on the Viewcube, because it's typically used for faster rotation (giving a "zoomed out" feel).

image

@ariesninjadev ariesninjadev requested review from a team as code owners June 25, 2025 17:49
AlexD717
AlexD717 approved these changes Jun 25, 2025
View reviewed changes
Copy link
Member

@AlexD717 AlexD717 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, only thing I would change is placing the camera settings below the preferences as I think this would be used less than the preferences.

Dhruv-0-Arora
Dhruv-0-Arora approved these changes Jun 26, 2025
View reviewed changes
Copy link
Collaborator

@Dhruv-0-Arora Dhruv-0-Arora left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good. One extra feature for the view cube that could be its own ticket that I wanted to note was that on fusion, when you drag the view cube, your cursor is stuck in the same spot & disappears. To be consistent, if we did that it would be good.

Also is branch on your fork?

@azaleacolburn
Copy link
Contributor

azaleacolburn commented Jun 26, 2025 •
edited
Loading

Also is branch on your fork?

It is on his fork :)

@BrandonPacewic
Merge dev
d4723e9 
* dev:
  2471 Robot Unit Tests Removed
  chore(doc): remove init scripts
  Point Snapping
  desync partial fix
  typo
  Game piece focus fix, partial drag fix
  Dependency fix
  Display toast when closed with button
  const req
  Improved dropoff, fixed styling, fixed post-drag snap
  Created drag mode
@BrandonPacewic BrandonPacewic mentioned this pull request Jun 26, 2025
Draft
@BrandonPacewic BrandonPacewic added the gameplay Relating to the playability of Synthesis label Jun 26, 2025
@BrandonPacewic BrandonPacewic merged commit 5551914 into Autodesk:dev Jun 26, 2025
15 checks passed
@ariesninjadev ariesninjadev deleted the aries/1944/sensitivity-sliders branch June 26, 2025 23:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Dhruv-0-Arora Dhruv-0-Arora Dhruv-0-Arora approved these changes

@AlexD717 AlexD717 AlexD717 approved these changes

@PepperLola PepperLola Awaiting requested review from PepperLola PepperLola is a code owner automatically assigned from Autodesk/synthesis-admin

@BrandonPacewic BrandonPacewic Awaiting requested review from BrandonPacewic BrandonPacewic is a code owner automatically assigned from Autodesk/synthesis-admin

Assignees

@ariesninjadev ariesninjadev

Labels
gameplay Relating to the playability of Synthesis
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ariesninjadev @azaleacolburn @Dhruv-0-Arora @AlexD717 @BrandonPacewic