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

Fix some bugs since v3.2.10 #899

Merged
merged 11 commits into from
Jan 10, 2023
Merged

Conversation

WhiredPlanck
Copy link
Collaborator

@WhiredPlanck WhiredPlanck commented Jan 3, 2023

Pull request

Issue tracker

Feature

update social media information:
update the join code and api link of QQ group 2, since the original group has been dismissed for some reasons.

Code of conduct

Gradle task

Tasks passed on every commit

  • ./gradlew spotlessCheck
  • ./gradlew assembleDebug

Manual test

  • Done

Code Review

  1. No wildcards import
  2. Manual build and test pass
  3. GitHub action ci pass
  4. At least one contributor reviews and votes
  5. Can be merged clean without conflicts
  6. PR will be merged by rebase upstream base

Daily build

Login to fetch artifact

https://github.com/osfans/trime/actions

Additional Info

v3.2.10 heavily refactors the parsing of the theme, replacing the Drawable type of plain-color-only graphics / rects with ColorDrawable instead of GradientDrawable, in order to reduce the performance loss.
However, that is just my wishful thinking, as there is no conclusive evidence show that that way can do so here.
ColorDrawable is a simple Drawable type, which is basically just for showing a plain color shape, but without round corner setting. So let us go back to GradientDrawable.
Update QQ group 2 join code and link.
@WhiredPlanck WhiredPlanck marked this pull request as draft January 6, 2023 12:54
…id keyboard (osfans#869)

- Also reimplement how to save symbol key history (easier).
... to help parse yaml on Java (Kotlin) side

version 0.49.0
@tao659
Copy link

tao659 commented Jan 8, 2023

这个版本,点击输入框,都会进入数字键盘模式,而不是字母键盘模式

@WhiredPlanck
Copy link
Collaborator Author

@tao659 我这里未能复现

@tao659
Copy link

tao659 commented Jan 9, 2023

最新更新的版本https://github.com/osfans/trime/actions/runs/3872195547
无法弹出键盘,部署没报错,实时日志没报错

@WhiredPlanck
Copy link
Collaborator Author

@tao659 可我这里什么事情都没有 …… 我甚至为了测试都卸载重装过一轮了 🤣

@WhiredPlanck WhiredPlanck marked this pull request as ready for review January 9, 2023 13:33
@WhiredPlanck
Copy link
Collaborator Author

@Bambooin I think here are all the bugs I can fix this time. Ready for review.

- Add preference to enable/disable to use custom sound theme (package)
- Utilize kaml library to parse sound theme configurations
- Ship stuffs of parsing the configurations to SoundManager
- Ship stuffs of playing the custom key effect to InputFeedbackManager
- Suppress unchecked casts
- Eliminate implicit casts
- Remove redundant parameters
@tao659
Copy link

tao659 commented Jan 10, 2023

@tao659 可我这里什么事情都没有 …… 我甚至为了测试都卸载重装过一轮了 🤣

最新的pr 已经解决了无法弹出键盘的问题👍🏻

@Bambooin Bambooin merged commit 6ce3631 into osfans:develop Jan 10, 2023
@WhiredPlanck WhiredPlanck deleted the 3.2.10-fix branch January 10, 2023 14:53
@WhiredPlanck
Copy link
Collaborator Author

@tao659 @Bambooin Many thanks to you ~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants