View Issue Details

IDProjectCategoryView StatusLast Update
0001799OpenMPTAccessibilitypublic2024-07-26 23:14
Reportera11cf0 Assigned ToSaga Musix  
PrioritynormalSeverityminorReproducibilityhave not tried
Status resolvedResolutionfixed 
Platformx64OSWindowsOS Version11
Target VersionOpenMPT 1.32 / libopenmpt 0.8 (goals)Fixed in VersionOpenMPT 1.32 / libopenmpt 0.8 (goals) 
Summary0001799: More keyboard focus weirdness in OPL editor
Description
  1. When switching from OpenMPT's OPL editor to another program and going back the focus gets lost in a strange container control. This looks very similar to 0001795.
  2. Keyboard focus is not restored to the last used control when going to the upper half of sample view and then returning to the OPL editor. Currently it always goes back to the first control. The General tab is a good example of expected behavior.
TagsNo tags attached.
Has the bug occurred in previous versions?
Tested code revision (in case you know it)

Relationships

related to 0001800 resolvedSaga Musix Restore last focus position when navigating to upper view halves 

Activities

Saga Musix

Saga Musix

2024-07-22 20:53

administrator   ~0006000

Please check out r21256, it should hopefully resolve all remaining focus issues with the editor windows.

a11cf0

a11cf0

2024-07-23 15:59

reporter   ~0006004

The above issues are fixed, but there's a new and quite strange one.
When switching from an OPL instrument to a normal sample the focus doesn't go to the sample view but instead gets stuck. Afterwards it remains stuck even when switching back to the previous OPL instrument. The only way to make it behave well again is to switch tabs. Also, when the sample editor is in this inconsistent state, it's not possible to return to it from the instrument library by pressing Alt+S, the shortcut simply does nothing.

Saga Musix

Saga Musix

2024-07-23 19:28

administrator   ~0006006

Please check r21267.

a11cf0

a11cf0

2024-07-24 12:48

reporter   ~0006009

Unfortunately r21267 makes no difference in this case. After switching from an OPL instrument to a normal sample via prev/next shortcuts the focus becomes trapped in the already dead OPL control that was focused before the switch. Also, selecting a normal sample from the instrument library doesn't move the focus from the treeview at al.

Saga Musix

Saga Musix

2024-07-24 20:55

administrator   ~0006012

This should now be fixed in r21281.

a11cf0

a11cf0

2024-07-25 17:24

reporter   ~0006014

Switching samples with the treeview is now working properly, but the focus is still getting stuck in the OPL editor when using prev/next sample shortcuts. The following steps can be used to see this surprising thing in action:

  1. Load or create a normal sample.
  2. Initialize a new OPL instrument.
  3. Focus the Modulation Feedback slider and leave it at 0%.
  4. Press the Previous Sample shortcut, and then End.
  5. Go back to the OPLL instrument. Modulation feedback will be at 100%.
Saga Musix

Saga Musix

2024-07-25 17:55

administrator   ~0006015

So I guess the fix in 0001795 didn't work at all.

a11cf0

a11cf0

2024-07-25 18:26

reporter   ~0006016

Yes, it seems so, although it did prevent tab navigation between hidden OPL controls.

Saga Musix

Saga Musix

2024-07-25 18:57

administrator   ~0006018

One more try with r21285, please.

a11cf0

a11cf0

2024-07-25 20:08

reporter   ~0006020

Now switching from an OPL instrument totally breaks keyboard input in the sample editor. All key presses just produce the default Windows dummy sound and do nothing else. Going to a different tab and back again helps to get out of this state.

Saga Musix

Saga Musix

2024-07-25 20:27

administrator   ~0006021

r21288 should hopefully work better.

a11cf0

a11cf0

2024-07-26 09:16

reporter   ~0006025

Wow! r21288 seems to have actually fixed this. Thank you!

Issue History

Date Modified Username Field Change
2024-07-01 15:23 a11cf0 New Issue
2024-07-01 20:14 Saga Musix Assigned To => Saga Musix
2024-07-01 20:14 Saga Musix Status new => assigned
2024-07-21 21:15 Saga Musix Relationship added related to 0001800
2024-07-22 20:53 Saga Musix Status assigned => feedback
2024-07-22 20:53 Saga Musix Note Added: 0006000
2024-07-22 21:03 Saga Musix Target Version => OpenMPT 1.32 / libopenmpt 0.8 (goals)
2024-07-23 15:59 a11cf0 Note Added: 0006004
2024-07-23 15:59 a11cf0 Status feedback => assigned
2024-07-23 19:28 Saga Musix Status assigned => feedback
2024-07-23 19:28 Saga Musix Note Added: 0006006
2024-07-24 12:48 a11cf0 Note Added: 0006009
2024-07-24 12:48 a11cf0 Status feedback => assigned
2024-07-24 20:55 Saga Musix Status assigned => feedback
2024-07-24 20:55 Saga Musix Note Added: 0006012
2024-07-25 17:24 a11cf0 Note Added: 0006014
2024-07-25 17:24 a11cf0 Status feedback => assigned
2024-07-25 17:55 Saga Musix Note Added: 0006015
2024-07-25 18:26 a11cf0 Note Added: 0006016
2024-07-25 18:57 Saga Musix Status assigned => feedback
2024-07-25 18:57 Saga Musix Note Added: 0006018
2024-07-25 20:08 a11cf0 Note Added: 0006020
2024-07-25 20:08 a11cf0 Status feedback => assigned
2024-07-25 20:27 Saga Musix Status assigned => feedback
2024-07-25 20:27 Saga Musix Note Added: 0006021
2024-07-26 09:16 a11cf0 Note Added: 0006025
2024-07-26 09:16 a11cf0 Status feedback => assigned
2024-07-26 23:14 Saga Musix Status assigned => resolved
2024-07-26 23:14 Saga Musix Resolution open => fixed
2024-07-26 23:14 Saga Musix Fixed in Version => OpenMPT 1.32 / libopenmpt 0.8 (goals)