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

Ui #17

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Ui #17

wants to merge 3 commits into from

Conversation

julianko
Copy link

@julianko julianko commented Oct 23, 2016

this PR improves a couple of things about the UI:

  • If you move your mouse one pixel between mouse down and mouse up when clicking on an animation cell, it counts as dragging the cell. This happens very often on accident (at least to me). I've added a threshold of 10 pixels that you need to move at least to make it count as dragging.
  • Change the mouse wheel zoom behaviour to zoom towards the cursor instead of the origin of the image.
  • Change the way the checkerboard background is drawn. The new method only requires a tiny 2x2 BufferedImage that is used with appropriate tiling to draw the entire checkerboard in one simple fill command. This is requires less code and memory, and should also be more efficient because it's cache friendlier.

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.

1 participant