View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0001382 | OpenMPT | General | public | 2020-10-18 21:13 | 2021-02-23 22:32 |
Reporter | fwl | Assigned To | Saga Musix | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Platform | x64 | OS | Windows | OS Version | 10 |
Product Version | OpenMPT 1.29.04.00 / libopenmpt 0.5.2 (upgrade first) | ||||
Target Version | OpenMPT 1.29.08.00 / libopenmpt 0.5.6/0.5.7 (upgrade first) | Fixed in Version | OpenMPT 1.29.08.00 / libopenmpt 0.5.6/0.5.7 (upgrade first) | ||
Summary | 0001382: OPL Instrument view breaks shortcuts for general, pattern, instrument and comment view | ||||
Description | When you press Alt-S to go to Sample View and then press Ctrl-up or down to scroll through the samples whenever you reach an OPL instrument the shortcuts ALt-G, ALt-P Alt-N and Alt-C don't work anymore. It seems after a closer look it has to do with the focus on the checkbox 'additive synthesis'. When I press Tab to loose the focus on this checkbox the shortcuts work again. But still when I go back to the OPL-view this checkbox is in focus again. | ||||
Steps To Reproduce | When you press Alt-S to go to Sample View and then press Ctrl-up or down to scroll through the samples whenever you reach an OPL instrument the shortcuts ALt-G, ALt-P Alt-N and Alt-C don't work anymore. It seems after a closer look it has to do with the focus on the checkbox 'additive synthesis'. When I press Tab to loose the focus on this checkbox the shortcuts work again. But still when I go back to the OPL-view this checkbox is in focus again. | ||||
Tags | No tags attached. | ||||
Has the bug occurred in previous versions? | |||||
Tested code revision (in case you know it) | |||||
Even with these steps I cannot reproduce the issue - maybe because here the checkbox is never focussed automatically? I would actually want it to be focussed for accessibility reasons, but it definitely isn't focussed here after switching to an OPL instrument. I have to actively click on it to give it focus, and even then the shortcuts keep working for me. Strange! |
|
In addition to the report, OPENMPT loses focus as a program when selecting a OPL-instrument with shortcut Alt-S. Pressing Tab or arrow keys brings the focus back. |
|
Okay, THAT definitely sounds like some third-party software must be interfering with OpenMPT. |
|
Ok I'll see if I can figure out which software. But it is strange that it only affects Openmpt when an OPL instrument is selected... it works fine if it's a sample that gets selected with the shortcut Alt-S |
|
If this bug still happens to you, can you please check if this test build works better? https://builds.openmpt.org/builds/auto/openmpt/bin.win64/1.29/openmpt-win64-1.29.07.01-r14199.7z |
|
Hello Saga, I've checked if the bug was still there in version 1.29.07.00 and it isn't. So it is allready solved in the latest official version if it was a problem with Openmpt to begin with and not caused by variables on my side. In the 1.29.07.01 the issue is also solved, except here you can't enter the parameters of the OPL-instrument for editing by pressing Tab. This is still possible in the 1.29.07.00 version. So in that respect the .00 version works better. |
|
I don't think there was any change since you reported the original issue that would fix it, but if you have the time maybe you could double-check if it still happens with OpenMPT 1.29.04.00 (link leads to portable version for easy testing). It's strange that the Tab key doesn't work for you with the new version, as there shouldn't be a difference in that regard; it also works just fine here so I think that the original problem might still not be solved just is exposed in a different way now. By the way, another user had a similar problem , which is why I came back to this issue. They noticed that when the OPL editor is focussed, it would seem like the main OpenMPT window loses focus and its title bar becomes greyed out as a result. Can you remember if you experienced something similar, or do you maybe even still see it with the latest version? If so, it would perfectly explain why the shortcuts stopped working for you. |
|
I did some further testing and the issue with the TAB not working only occurs when the first sampleslot created is an OPL-instrument. When you first add samples then TAB does work and the Additive Synthesis checkbox has first focus. And yes if you pressed Alt-S and the shortcut lead you to an OPL-instrument, Openmpt would lose focus as a program. I will test some things with the 1.29.04.00 as well |
|
Hi Saga, I'm sorry, I was a little too hasty with testing. I had to recall/refresh what the issue details were. The test results are: 1.29.04.00 : when selecting an OPL instrument with Alt-S the shortcuts Alt-P,-G, -C, -N don't work anymore. So you can't return to pattern view for example. Alt-I does work. This goes for every sample slot. So when you scroll down to a sample slot below or above the OPL instrument the shortcuts still don't work. It's because Openmpt loses focus as a program and the main window gets greyed out like you said. While testing I also ran into another shortcut issue: when you are in the comment tab and switch to tree view with Alt-I you can't return to the comment tab with Alt-C |
|
Okay, that's great news. I'm sure we can fix the remaining issue with the tab key as well. I'll try some things and give you a new build to test soon. |
|
Does the Tab key work for you after clicking on one of the controls, or does just never work at all? |
|
It works when you click one of the parameter controls and then the dashed border appears to indicate that its is in focus. After that you can scroll through the controls using TAb and Shift Tab. But when you leave the sample editor and then go back to the sample editor you have to use the mouse again for the first control selection |
|
Okay, so we're on the same page then. This is how it worked for me before and after the fix, and it's how it worked on all other machines I could try so far. I'm not fully sure why it happens (and I tried fixing it before so I'm kinda out of ideas on that topic), so I would propose to close this issue (as the original problem is fixed), but I will continue to figure out why the keyboard focus doesn't work properly. |
|
Yes that's fine by me. The keyboard focus in the OPL instrument isn't that big of an issue. That you've got the other issue fixed is great and makes the switching between sample view and the other views with OPL-instruments in the sample-pool a lot smoother. So, Thanks! |
|
I just dug a bit more into this code and I think I have finally found the root cause for the keyboard focus problem. r14203 should be available for testing in about 1-2 hours from here: |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2020-10-18 21:13 | fwl | New Issue | |
2020-10-18 21:21 | Saga Musix | Note Added: 0004479 | |
2020-10-18 21:25 | fwl | Note Added: 0004480 | |
2020-10-18 21:26 | Saga Musix | Note Added: 0004481 | |
2020-10-18 21:41 | fwl | Note Added: 0004482 | |
2021-02-21 17:11 | Saga Musix | Note Added: 0004648 | |
2021-02-23 10:15 | fwl | Note Added: 0004650 | |
2021-02-23 10:22 | Saga Musix | Note Added: 0004651 | |
2021-02-23 10:42 | fwl | Note Added: 0004652 | |
2021-02-23 11:16 | fwl | Note Added: 0004653 | |
2021-02-23 17:25 | Saga Musix | Note Added: 0004654 | |
2021-02-23 17:25 | Saga Musix | Assigned To | => Saga Musix |
2021-02-23 17:25 | Saga Musix | Status | new => assigned |
2021-02-23 17:53 | Saga Musix | Note Added: 0004655 | |
2021-02-23 19:32 | fwl | Note Added: 0004656 | |
2021-02-23 20:02 | Saga Musix | Note Added: 0004657 | |
2021-02-23 20:47 | fwl | Note Added: 0004658 | |
2021-02-23 22:01 | Saga Musix | Status | assigned => resolved |
2021-02-23 22:01 | Saga Musix | Resolution | open => fixed |
2021-02-23 22:01 | Saga Musix | Fixed in Version | => OpenMPT 1.29.08.00 / libopenmpt 0.5.6/0.5.7 (upgrade first) |
2021-02-23 22:01 | Saga Musix | Target Version | => OpenMPT 1.29.08.00 / libopenmpt 0.5.6/0.5.7 (upgrade first) |
2021-02-23 22:32 | Saga Musix | Note Added: 0004659 |