-
Notifications
You must be signed in to change notification settings - Fork 23
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
SmartSet crash on specific configuration #11
Comments
Thanks for the report. It's presumably a single bad token somewhere in that list. We will investigate and work on fix. Let us know if you get any specifics. |
I'm also getting this error with my Kinesis Advantage 360 on macOS SmartSet v1.0.2. The macro shown in the diff below causes this error. If I remove this line the error goes away and I can open the SmartSet app again. Nothing in the layout here was edited manually. This was all generated by the SmartSet app. Also, the F13 macro does work on the keyboard. Currently I have to comment this line out, make edits with the SmartSet app, and then put this line back in by editing the layout.txt file directly. <base>
[eql]>[esc]
[hk1]>[obrk]
[hk3]>[cbrk]
[esc]>[bspc]
[hk2]>[F15]
[hk4]>[F16]
[rshf]>[del]
[lfn]>[lctr]
[grav]>[null]
[caps]>[fn1s]
[left]>[F13]
[rght]>[F14]
[up]>[left]
[obrk]>[up]
[cbrk]>[rght]
[rfn]>[eql]
[lctr]>[F17]
[lalt]>[vol+]
[rwin]>[null]
[rctr]>[null]
[bspc]>[lwin]
[del]>[lalt]
[home]>[vol-]
[pgup]>[null]
[end]>[lctr]
[pgdn]>[null]
-{F13}>{s9}{x1}{-lshf}{9}{+lshf}
<keypad>
<function1>
[1]>[F1]
[2]>[F2]
[3]>[F3]
[4]>[F4]
[5]>[F5]
[6]>[F6]
[7]>[F7]
[8]>[F8]
[9]>[F9]
[0]>[F10]
[hyph]>[F11]
[bsls]>[F12]
[caps]>[defs]
<function2>
<function3> |
Looks like its actually that the App can't handle macros assigned to F-keys. We're working on fix now. {F8}>{x} doesn't work. |
Hi, I encounter similar issue when setup the macro like this:
After removing the macro, the application can start normally. |
Using modifier keys as the sole macro trigger is technically not supported/recommended so that is why the App is crashing. We can see if there is a way to allow the App to load normally and just ignore such a macro without overwriting it. |
same issue assigning macro to fn
|
assigning to fn keys is probably going to be a really popular use-case because the late function keys (> fn12) are essentially how you can assign a key a unique identifier in the above case i want to assign a macro to the key corresponding to the default layer's obviously this is still pretty unergonomic from a programming perspective. the real fix would be to have unique tokens corresponding to each physical key |
We just uploaded a Windows version 1.0.4 that should resolve this issue for you (macOS in progress). Please test and confirm if you are able. |
The SmartSet app v1.0.2 crashes on both Windows and macOS with the following in
layouts/layout1.txt
:The error is Access Violation and then the program either has to be stopped, or it freezes.
The text was updated successfully, but these errors were encountered: