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

Mouse wheel shortcut to change the landmark's size #111

Open
csagonas opened this issue Oct 13, 2015 · 5 comments
Open

Mouse wheel shortcut to change the landmark's size #111

csagonas opened this issue Oct 13, 2015 · 5 comments

Comments

@csagonas
Copy link

Notes by @jabooth:
Landmark size is currently normalised by the total size of the asset. This is pretty much the most sensible thing we can do, but does mean that in the cases of image databases the perceptive size of the landmarks can change quite a lot between different assets (as the object being annotated is sometimes larger or smaller in the frame of the asset).

As a result, it's quite common that the user needs to make little adjustments to the landmark scale frequently. This currently requires mousing down to the bottom left of the UI.

Maybe we could have a keyboard modifier + wheel shortcut for landmark size? Say ctrl + wheel?

@lirsacc
Copy link
Contributor

lirsacc commented Oct 17, 2015

+ and - keyboard shortcuts already exists for this (maybe not documented in help)

@jabooth
Copy link
Member

jabooth commented Oct 18, 2015

Ah yes indeed they are - just missing from the help screen.

@lirsacc
Copy link
Contributor

lirsacc commented Oct 19, 2015

Just thinking, but maybe an hybrid of what you suggested without a keyboard modifier, but we could add scrolling over the range element to resize as well. I feel it's now a pretty standard behaviour that people would expect. What do you think ?

@jabooth
Copy link
Member

jabooth commented Oct 20, 2015

Ah that's a nice idea actually, I like it!

@lirsacc
Copy link
Contributor

lirsacc commented Oct 20, 2015

Cool, I'll try and get it working in the morning then.

EDIT: First version in https://github.com/menpo/landmarker.io/tree/scroll-landmark-size. Not as smooth as I'd like it to be, but functional.

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

No branches or pull requests

3 participants