Skip to content

Fix and Improve trackball interactivity #227

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

Merged
merged 1 commit into from
Apr 26, 2021

Conversation

9prady9
Copy link
Member

@9prady9 9prady9 commented Apr 26, 2021

  • A very basic wire-ball of two loops is shown when rotation is being done to better hint the user of the direction of rotation.
  • Fixed bug in trackball computation that was incorrectly using angle
    in degrees instead of radians
  • Add custom Cursors for zoom and rotation modes to visually aid the
    user
    • Hand cursor appears when rotation is activated(Mouse Right button
      drag)
    • Arrow with top and bottom heads is activated when zoom in/out mode
      is active(Ctrl + Left Right button) to hint the required cursor
      movement for desired zoom in or zoom out

SDL window toolkit additionally has custom move cursor.

Change(s)
Trackball rotation has a bug in computing the angle.

Current Behavior
Incorrectly uses degrees unit instead of radians

New Behavior
Fixed angle units and improved interactivity hints

Breaking Change(s)
None

- Fixed bug in trackball computation that was incorrectly using angle
  in degrees instead of radians
- Add custom Cursors for zoom and rotation modes to visually aid the
  user
  - Hand cursor appears when rotation is activated(Mouse Right button
    drag)
  - Arrow with top and bottom heads is activated when zoom in/out mode
    is active(Ctrl + Left Right button) to hint the required cursor
    movement for desired zoom in or zoom out

SDL window toolkit additionally has custom move cursor.
@9prady9 9prady9 added this to the 1.0.7 milestone Apr 26, 2021
@9prady9 9prady9 merged commit dc54597 into arrayfire:master Apr 26, 2021
@9prady9 9prady9 deleted the virtual_trackball branch April 26, 2021 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant