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

Add slider for free rotate #50

Open
wants to merge 1 commit into
base: mirador4-compat-clean
Choose a base branch
from

Conversation

gerdesque
Copy link
Collaborator

This pull request introduces the 'Free Rotate Enhancement' feature, which improves the user experience when rotating manifests within the application. The enhancement allows for smoother, more intuitive control over object rotation, addressing previously reported issues.

Related Issues

#27

Additional Notes

This pull request is based on mirador4-compat-clean branch to benefit from the Material UI 5 changes.

@lutzhelm
Copy link

I finally had a chance to look at this.

  • A slider is a good solution for this feature, but currently, for me, the slider reacts somewhat clunky.
  • People might still want to use rotation in 90° steps, so maybe instead of replacing the clockwise/counter clockwise rotation entirely, why not offer both options, with a configuration option to dis-/enable them separately?
  • It would be great to be able to reset the rotation - I'm having trouble hitting the exact point at 0°.
  • The style of the slider might need some additional work.

I'm not sure if we need more design for that or who would be able to do that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants