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

"Add instance" and "Remove instance" hotkeys don't work when object is selected via de object browser #5350

Open
jaapp opened this issue Dec 1, 2020 · 6 comments

Comments

@jaapp
Copy link

jaapp commented Dec 1, 2020

Version

Prusaslicer 2.4.0-beta1

Operating system type + version

MacOs 10.14.6

Behavior

The + and - hotkeys don't work in the object-browser

  • Start Prusaslicer
  • Add STL to buildplate
  • Select object via object browser (click on object so it's highlighted blue)
    image
  • Press "add instance" or "remove instance" hotkey on keyboard (+ and - keys)
  • No new instance is made / instances can't be removed this way
  • Clicking on the corresponding buttons does work
    image
  • Also, when you select an object by selecting them on the buildplate the hotkeys do work.

Is this a new feature request?
No

@fe60
Copy link

fe60 commented Dec 7, 2020

Also, when you select an object by selecting them on the buildplate the hotkeys _do_ work.

To me it seems to be related to the location of the mouse pointer, not to the way an object has been selected.

  1. Select an object in the right panel/object list
  2. Keep mouse pointer over right panel
  3. use + / - to add/remove an instance
    => nothing happens
  4. Move mouse pointer back into 3D view (don't click, just move)
  5. use + / - to add/remove an instance
    => works

Tested with PrusaSlicer-2.3.0-beta2+win64-202012051115

@jaapp
Copy link
Author

jaapp commented Dec 7, 2020

I had to test @fe60 his remark and yes - it is related to the mouse pointer! So weird. None of the other hotkeys seem to have this limitation. It is very unintuitive. And the effect is that you can't use your keyboard to do some quick instance creation without involving the mouse.

bubnikv added a commit that referenced this issue Dec 8, 2020
…ject is selected via de object browser #5350

Captures the +- hotkeys in the object list to add / remove an instance.
@jaapp
Copy link
Author

jaapp commented Dec 8, 2020

Great to see this is getting addressed in the next release. This means that the most important hotkeys are now supported by the object list.

I've looked at the changes in the source code and this prompted me to test some other hotkeys. These are all the hotkeys that don't work when the object list is selected:

From the top toolbar:

  • A - Arrange all
  • Shift+A - Arrange all selected

From the toolbar on the left:

  • M - Move
  • S - Scale
  • R - Rotate
  • F - Place on face
  • C - Cut
  • L - Paint on support
  • P - Seam painting

The hotkeys for the left-toolbar all require the user to do some object manipulation with the mouse in the 3D workspace so I guess there is a reason for these not to work outside of this context - although its arguably not the most consistent user experience.

The same can't be said for the top toolbar. Hotkeys A and Shift+A don't require any manual object manipulation. It would be great to add these too. Plus that would also mean that the entire top-toolbar is now supported by the object list.

Thanks for taking the time to read my feedback!

@bubnikv
Copy link
Collaborator

bubnikv commented Dec 12, 2020

Implemented for Windows in PrusaSlicer 2.3.0-beta3.
On OSX the +- accelerators do not work reliably with a localized keyboard.

@jaapp
Copy link
Author

jaapp commented Dec 14, 2020

@bubnikv I've tried beta3 and the +- hotkeys don't work on a standard keyboard.

I have a mid-2014 13-inch Macbook Pro with a (US) English International Layout keyboard layout.

I wonder if the right keycodes to catch should be WXK_ADD and WXK_SUBTRACT

@bubnikv
Copy link
Collaborator

bubnikv commented Oct 4, 2021

We are limited on OSX with the wxWidgets UI wrapper library. We install an accelerator table, but for some reason it seems the support of accelerator tables in wxWidgets is not right. We would likely have to fix wxWidgets.

I wonder if the right keycodes to catch should be WXK_ADD and WXK_SUBTRACT

@YuSanka tried that. It works for +/- on numerical keyboard only, which the macs don't offer by default.

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

No branches or pull requests

3 participants