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

UX,UI : Key mapping with menu item #59

Closed
amitguptagwl opened this Issue Jul 2, 2018 · 10 comments

Comments

Projects
None yet
3 participants
@amitguptagwl
Member

amitguptagwl commented Jul 2, 2018

https://naturalintelligence.github.io/imglab

User can see key mappings when he clicks on keyboard icon. In addition, it'll be good if we display key mappings with each menu item.

Watch for changes, or Bookmark for easy discovery.
Fund this project for new features and maintenance.
[Showcase] your project with us by raising an issue

@DheenodaraRao

This comment has been minimized.

DheenodaraRao commented Jul 7, 2018

Can you be more specific? I am interested to contribute.

@amitguptagwl

This comment has been minimized.

Member

amitguptagwl commented Jul 7, 2018

If you'll visit mentioned link, you'll notice a 3 line icon on top left corner. When you'll hover on that you'll see list of menu items.

We want to show the key mapping with each item floating to the right.

Open - Ctrl + i
Save - Ctrl + e

@Alvin-Voo

This comment has been minimized.

Collaborator

Alvin-Voo commented Aug 16, 2018

Hi @amitguptagwl , are you referring to the tooltip when mouse hover over the Open and Save?

I saw that you had jquery-ui installed. I did a small jsfiddle using jquery-ui:
imglab_ 59

Please tell me if this is what you are looking for. Thanks.

@amitguptagwl

This comment has been minimized.

Member

amitguptagwl commented Aug 16, 2018

Yes that can be a way. However I was aiming to show them all the time in the right side of each menu item. So the user can press relevant key instead of hovering the mouse to know.

@Alvin-Voo

This comment has been minimized.

Collaborator

Alvin-Voo commented Aug 17, 2018

@amitguptagwl Like this?

screenshot from 2018-08-17 11-52-48

@amitguptagwl

This comment has been minimized.

Member

amitguptagwl commented Aug 17, 2018

Yes but without background color.

@Alvin-Voo

This comment has been minimized.

Collaborator

Alvin-Voo commented Aug 22, 2018

@amitguptagwl How bout this. The simpler the better?

imglab_ 59

@amitguptagwl

This comment has been minimized.

Member

amitguptagwl commented Aug 22, 2018

This is good.

@Alvin-Voo

This comment has been minimized.

Collaborator

Alvin-Voo commented Aug 22, 2018

@amitguptagwl PR created. Thanks!

amitguptagwl added a commit that referenced this issue Aug 22, 2018

Merge pull request #87 from Alvin-Voo/feat/#59
feat: Added shortcut Key Mapping for menu items. For issue #59
@amitguptagwl

This comment has been minimized.

Member

amitguptagwl commented Aug 22, 2018

Thanks @Alvin-Voo . merged

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