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

"Main menu" should be customizable #53

Open
enterprisey opened this Issue Feb 27, 2015 · 3 comments

Comments

Projects
None yet
2 participants
@enterprisey
Member

enterprisey commented Feb 27, 2015

At the moment, the "main menu" of the script has 3 buttons: Accept, Decline, and Comment. I can click on the arrow to the right to show Clean and Mark as Reviewing.

However, I only use three of those buttons on a regular basis: Accept, Decline, and Clean.

There should be an option in the preferences so I can select which buttons appear in the "main menu", as the buttons in the "secondary" menu (i.e. the one hidden by the arrow on the right) should be truly secondary.

@theopolisme

This comment has been minimized.

Show comment
Hide comment
@theopolisme

theopolisme Feb 27, 2015

Contributor

Of course the other option is to reduce the size of the buttons and enable the all to be displayed without the need for a slide out menu..

Contributor

theopolisme commented Feb 27, 2015

Of course the other option is to reduce the size of the buttons and enable the all to be displayed without the need for a slide out menu..

@enterprisey

This comment has been minimized.

Show comment
Hide comment
@enterprisey

enterprisey Feb 28, 2015

Member

I feel like the main reason there's a slide out menu in the first place is so that

  • the buttons present larger targets (Fitts's law!)
  • there's less complexity open to the user by default

i.e. I'd rather not see options that I never use by default.

Member

enterprisey commented Feb 28, 2015

I feel like the main reason there's a slide out menu in the first place is so that

  • the buttons present larger targets (Fitts's law!)
  • there's less complexity open to the user by default

i.e. I'd rather not see options that I never use by default.

@enterprisey

This comment has been minimized.

Show comment
Hide comment
@enterprisey

enterprisey Mar 1, 2015

Member

A few more thoughts on this:

  • We could use something like jqxDragDrop.
  • Or, we could have a table with two columns: name of action and a two-state slider indicating whether it's on the primary or secondary menu.
  • The Ozone Web Framework solution looks pretty cool, but complicated.
  • HTML Goodies seems to have a good solution, but the demo link gives a 404.
  • The solution HTML Goodies uses is a combination of the Draggable and Droppable jQuery UI elements.

At the moment, I'm leaning towards the last one on the list (jQuery UI) just because it looks nicer and seems easier to use.

Member

enterprisey commented Mar 1, 2015

A few more thoughts on this:

  • We could use something like jqxDragDrop.
  • Or, we could have a table with two columns: name of action and a two-state slider indicating whether it's on the primary or secondary menu.
  • The Ozone Web Framework solution looks pretty cool, but complicated.
  • HTML Goodies seems to have a good solution, but the demo link gives a 404.
  • The solution HTML Goodies uses is a combination of the Draggable and Droppable jQuery UI elements.

At the moment, I'm leaning towards the last one on the list (jQuery UI) just because it looks nicer and seems easier to use.

@enterprisey enterprisey added the epic label Feb 27, 2016

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