Skip to content
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

Toggle on space key #262

Merged
merged 3 commits into from
Jul 27, 2022
Merged

Toggle on space key #262

merged 3 commits into from
Jul 27, 2022

Conversation

eymar
Copy link
Collaborator

@eymar eymar commented Jul 26, 2022

This PR is a replacement for #126 (the #126 was not possible to rebase on top of jb-main, because that PR was based on a different branch - release/1.0)

This PR adds some changes to the original one:

  • adds KeyEvent.isToggle actual in jsNativeMain source set
  • adds a test for Modifier.toggleable to toggle by Space

Copy link
Collaborator

@igordmn igordmn left a comment

Choose a reason for hiding this comment

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

Thanks!

@eymar eymar merged commit 56ec185 into jb-main Jul 27, 2022
@eymar eymar deleted the toggle_on_space_key branch July 27, 2022 09:09
igordmn added a commit that referenced this pull request Aug 18, 2022
* Perform toggle on Space key

Similar to https://android-review.googlesource.com/c/platform/frameworks/support/+/1858797

* add a test for toggleable to toggle by Space button

* add Toggleable.jsNative.kt with `isToggle` actual

Co-authored-by: Igor Demin <igordmn@gmail.com>
Co-authored-by: Oleksandr Karpovich <oleksandr.karpovich@jetbrains.com>
igordmn added a commit that referenced this pull request Aug 18, 2022
* Perform toggle on Space key

Similar to https://android-review.googlesource.com/c/platform/frameworks/support/+/1858797

* add a test for toggleable to toggle by Space button

* add Toggleable.jsNative.kt with `isToggle` actual

Co-authored-by: Igor Demin <igordmn@gmail.com>
Co-authored-by: Oleksandr Karpovich <oleksandr.karpovich@jetbrains.com>
eymar pushed a commit that referenced this pull request Oct 26, 2022
* Perform toggle on Space key

Similar to https://android-review.googlesource.com/c/platform/frameworks/support/+/1858797

* add a test for toggleable to toggle by Space button

* add Toggleable.jsNative.kt with `isToggle` actual

Co-authored-by: Igor Demin <igordmn@gmail.com>
Co-authored-by: Oleksandr Karpovich <oleksandr.karpovich@jetbrains.com>
eymar pushed a commit that referenced this pull request Nov 16, 2022
* Perform toggle on Space key

Similar to https://android-review.googlesource.com/c/platform/frameworks/support/+/1858797

* add a test for toggleable to toggle by Space button

* add Toggleable.jsNative.kt with `isToggle` actual

Co-authored-by: Igor Demin <igordmn@gmail.com>
Co-authored-by: Oleksandr Karpovich <oleksandr.karpovich@jetbrains.com>
eymar pushed a commit that referenced this pull request Jan 13, 2023
* Perform toggle on Space key

Similar to https://android-review.googlesource.com/c/platform/frameworks/support/+/1858797

* add a test for toggleable to toggle by Space button

* add Toggleable.jsNative.kt with `isToggle` actual

Co-authored-by: Igor Demin <igordmn@gmail.com>
Co-authored-by: Oleksandr Karpovich <oleksandr.karpovich@jetbrains.com>
igordmn added a commit that referenced this pull request Nov 15, 2023
* Perform toggle on Space key

Similar to https://android-review.googlesource.com/c/platform/frameworks/support/+/1858797

* add a test for toggleable to toggle by Space button

* add Toggleable.jsNative.kt with `isToggle` actual

Co-authored-by: Igor Demin <igordmn@gmail.com>
Co-authored-by: Oleksandr Karpovich <oleksandr.karpovich@jetbrains.com>
igordmn added a commit that referenced this pull request Nov 16, 2023
* Perform toggle on Space key

Similar to https://android-review.googlesource.com/c/platform/frameworks/support/+/1858797

* add a test for toggleable to toggle by Space button

* add Toggleable.jsNative.kt with `isToggle` actual

Co-authored-by: Igor Demin <igordmn@gmail.com>
Co-authored-by: Oleksandr Karpovich <oleksandr.karpovich@jetbrains.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants