Skip to content

Allow symbol:slider, make single sliders not macro in this syntax - #1398

Open
SainoNamkho wants to merge 2 commits into
Julow:masterfrom
SainoNamkho:fix-slider-with-symbol
Open

Allow symbol:slider, make single sliders not macro in this syntax#1398
SainoNamkho wants to merge 2 commits into
Julow:masterfrom
SainoNamkho:fix-slider-with-symbol

Conversation

@SainoNamkho

Copy link
Copy Markdown
Contributor

Currently keys like cursor_up does not work if configured with the long syntax.

Currently keys like `cursor_up` does not work if configured with the long syntax.
@SainoNamkho
SainoNamkho force-pushed the fix-slider-with-symbol branch from 5fbbca2 to e8d4956 Compare August 19, 2026 08:48
@SainoNamkho
SainoNamkho force-pushed the fix-slider-with-symbol branch from 7fb05f2 to a9c17cc Compare August 19, 2026 12:30

@Julow Julow left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel like this is a lot of code for an edge case. What do you think of leaving the bug in place with a mention in the doc ?

@SainoNamkho

SainoNamkho commented Aug 31, 2026

Copy link
Copy Markdown
Contributor Author

I feel like this is a lot of code for an edge case. What do you think of leaving the bug in place with a mention in the doc ?

Maybe I mixed up too much problems in a single patch because one cannot be fixed without the others.😂

I have a specific use case, what do you think of it?
I make every swipe in cardinal directions, which have no function yet, a cursor move. So I can swipe to move cursor from any position. And I hide the icons, because otherwise there will be arrows everywhere on the keyboard. Sounds like a well-founded need?

In favor of that, I need:

  • cursor moves with customized symbols, which are macros now, work as sliders.
  • the symbols persist while modified.
  • in order to implement the previous item, calls to withSymbol might pass key font symbols. I choose to check if the first character is in PUA. This is inconsistent with other typed based overload solutions, but there isn't a good way to do this.

I'm pleased if there would be other ways for this need.😊

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

Successfully merging this pull request may close these issues.

2 participants