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

Enhance Key Preview on Press #1522

Merged
merged 3 commits into from
Dec 3, 2024

Conversation

WhiredPlanck
Copy link
Collaborator

Pull request

Issue tracker

Fixes will automatically close the related issues

Fixes # N/A

Feature

Describe features of this pull request

In this PR, key preview on press gets enhanced. And more code cleanup BTW.

Code of conduct

Code style

Build pass

  • make debug

Manually test

  • Done

Code Review

  1. No wildcards import
  2. Manual build and test pass
  3. GitHub Action CI pass
  4. At least one contributor review and approve
  5. Merged clean without conflicts
  6. PR will be merged by rebase upstream base

Daily build

Login and download artifact at https://github.com/osfans/trime/actions

Additional Info

@WhiredPlanck
Copy link
Collaborator Author

LGTM on emulator

Screen_recording_20241201_173612.mp4

@Bambooin
Copy link
Collaborator

Bambooin commented Dec 2, 2024

Weird F and N button with popup on key press, hard to reproduce, so I don't have any idea.

  • F has float preview
  • N is missing

weird

@WhiredPlanck
Copy link
Collaborator Author

@Bambooin I cannot reproduce too. Debug version is always a little laggy, we may no need to worry about it.

@WhiredPlanck WhiredPlanck merged commit 41623f5 into osfans:develop Dec 3, 2024
3 checks passed
@WhiredPlanck WhiredPlanck deleted the keyboard-20241201 branch December 3, 2024 10:37
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