-
-
Notifications
You must be signed in to change notification settings - Fork 374
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
chore(cpn): generic simulator #5692
Conversation
Should we consider adding buttons for the left/right roller movements to the generic UI? There are keyboard mappings; but these aren't obvious, so navigation can be challenging if your mouse does not have a scroll wheel. |
I do not see why not and there could even be a Simulator setting option as to whether they display or not. |
Also planning to move trims from around joysticks for surface radios to centre where extra trims now go |
Currently the simulator UI for B&W radios with OLED screens switches the LCD colors so the simulator is white text on a black background (to match the physical display). The generic UI should probably have some method to handle this as well. |
1af28aa
to
3858b30
Compare
0ace67e
to
99b157d
Compare
fyi I've updated the sample images in the description. Radio 'case' colour should I make it per radio profile rather than same for all? I'm leaning towards per radio profile as not much more effort. |
This is looking great! :)
Per profile would be nice, as if you have more than one radio and they are different colours you have the option of making the background colour match... |
Radio case colour changed to per radio profile |
1f1ad84
to
1e627d0
Compare
I have tried to make them intuitive but there are a couple of odd ones. Mac users please advise if not on standard keyboard especially Macbook. Latest mappings |
Just FYI, on Windows and Linux:
really is Any chance of getting Backspace back as another shortcut for Cursor keys on windows (left/right/up/down) appear to run afoul of UI element focus... i.e. with T-Lite I need to click the LCD display, and then the right arrow only works once... it seems like up only works every second press, and down only the first time. Clicking on the LCD between each key press works perfectly though 🤔 Other than that, and some work being needed for the NV14/EL18/PL18 (the "switchable hats" thing) which IMO can be resurrected in a separate PR so this can go in sooner than later, this is looking great. |
@philmoz has asked for unshifted to ',' and '.'
Yes
I don't think it is focus but they need another mouse or key event. But this is not required for other keys. Pressing a button with arrows assigned doesn't suffer from the same issue as far as I've found. The two methods do not follow the exact code path so its running in debug mode time. Thanks for testing. |
Excellent... To be honest, it was annoying, but consistent. The unshifted minus and shifted plus is more annoying. 🤪 re: focus... I am seeing the focus rectangle on the button when the shortcut is pressed, hence why I think the keyboard focus is being lost, but yeah, debug should reveal all. |
Not if you use the keypad |
MacOS:
On the TLITE the Up arrow works on every second press. The Down arrow works if I click on the LCD display; but only once. I have to keep clicking on the LCD display section between each press of the down arrow. |
The keypress is being lost/consumed by another object but strangely the keyrelease is being received. So that object is likely only intercepting keypress and not putting back on the event stack if it doesn't use it. The mouse click resets the arrow keys or as @pfeerick and yourself state is refocuses the key events back to the radio widget. So far all the other special keys don't suffer from this issue. |
GX12 also needs rotary encoder buttons and shortcuts. |
@pfeerick @philmoz you are on the right track as this post indicates So now to find the focus stealer. |
Threaten it with me coming stomping along with ma wellies... That should
share it into submission. 😂
…On Mon, 23 Dec 2024, 10:47 am Neil Horne, ***@***.***> wrote:
@pfeerick <https://github.com/pfeerick> @philmoz
<https://github.com/philmoz> you are on the right track as this post
indicates
https://forum.qt.io/topic/153217/key-press-arrow-keys-cause-loss-of-focus
So now to find the focus stealer and reprimand it sternly.
—
Reply to this email directly, view it on GitHub
<#5692 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABJ66KISHAVHC7Y2O4IYEZ32G5MSNAVCNFSM6AAAAABS2GCKJWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJYGY4TKNJZHE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
76e0a63
to
267baca
Compare
0a24ff4
to
6d1f4c9
Compare
Happy New Year to you too. Merge if you cannot break. I believe there are still some focus issues to resolve/reduce however let's not hold the change up for that. |
Nothing is jumping out at me... and I didn't need to click the LCD once when trying with T-Lite/TX16S/X9D+2019 etc... was able to use keyboard shortcuts in all cases, so looks like you beat the majority if not all of the focus issues into submission ;) Is something to resurrect the switchable hats (NV14/EL18/PL18) simulator functionally on the agenda soon? Before I ask about the customisable switches... and... just kidding! |
Summary of changes:
Benefits:
Radio profile new Radio Case Colour setting

Simulator new settings

Surface radio with trims in centre, keys default positions and scroll keys

TX16S with keys RHS and no scroll keys and change of case colour

TX16S with keys LHS, scroll keys and change of case colour

GX12
