Skip to content

Keys API for C++/JS/Python#11098

Merged
LeonMatthes merged 5 commits intoslint-ui:masterfrom
LeonMatthes:lm/102-keys-language-bindings
Apr 2, 2026
Merged

Keys API for C++/JS/Python#11098
LeonMatthes merged 5 commits intoslint-ui:masterfrom
LeonMatthes:lm/102-keys-language-bindings

Conversation

@LeonMatthes
Copy link
Copy Markdown
Member

@LeonMatthes LeonMatthes commented Mar 24, 2026

  • Keys: Add Phython bindings
  • Fix clippy lints on Python value conversion
  • Keys: Add NodeJS bindings
  • Keys: Encapsulate as opaque C++ type
  • Keys: Expose in Rust API

Comment thread .github/workflows/autofix.yaml
Comment thread api/cpp/include/private/slint_keys.h Outdated
Comment thread api/cpp/include/private/slint_keys.h
Comment thread internal/core/items/input_items.rs
Comment thread internal/core/input.rs
Comment thread internal/interpreter/api.rs
@LeonMatthes LeonMatthes force-pushed the lm/102-keys-language-bindings branch from 333f071 to c1529b1 Compare March 25, 2026 15:09
@tronical
Copy link
Copy Markdown
Member

Python and JS side looks good to me. Thanks a lot ❤️

@LeonMatthes LeonMatthes force-pushed the lm/102-keys-language-bindings branch from c1529b1 to b01bd10 Compare March 25, 2026 15:56
@LeonMatthes LeonMatthes requested a review from ogoffart March 25, 2026 16:05
@LeonMatthes LeonMatthes force-pushed the lm/102-keys-language-bindings branch 2 times, most recently from f7aebbd to 0715cb0 Compare March 26, 2026 12:30
@LeonMatthes LeonMatthes force-pushed the lm/102-keys-language-bindings branch from 0715cb0 to ccb1718 Compare April 1, 2026 08:31
@LeonMatthes LeonMatthes force-pushed the lm/102-keys-language-bindings branch from ccb1718 to 547bdc5 Compare April 1, 2026 14:33
@LeonMatthes LeonMatthes enabled auto-merge (rebase) April 2, 2026 08:12
Copy link
Copy Markdown
Member

@ogoffart ogoffart left a comment

Choose a reason for hiding this comment

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

I'm still a bit concerned to see this "empty" type in the public API at this point, but let's merge this so we can progress

@LeonMatthes LeonMatthes force-pushed the lm/102-keys-language-bindings branch from 547bdc5 to f376db0 Compare April 2, 2026 08:26
@LeonMatthes LeonMatthes merged commit be3222e into slint-ui:master Apr 2, 2026
45 checks passed
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.

3 participants