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

Refactors 20241007 #1481

Merged
merged 4 commits into from
Oct 8, 2024
Merged

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

Optimize timing background sync and so on.

  • Remove confusing "sync in background" settings
  • Enhance the process of theme switching
  • Try to reduce "uninitialized lateinit property 'activeTheme'" problem

Next PR may focus on background sync with WorkManager.

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

... and related managers, therefore the stuffs in the scope can always get the initialized and unique Theme instance.
refactors: add id for some view for the convenience of debugging
@Bambooin
Copy link
Collaborator

Bambooin commented Oct 7, 2024

Found one regression in recent change, the input view is truncated such as wm look like wl.
~

@WhiredPlanck
Copy link
Collaborator Author

@Bambooin Similar to / same with #1479 ?

@Bambooin
Copy link
Collaborator

Bambooin commented Oct 8, 2024

Yes, seems duplicated with it.

@WhiredPlanck
Copy link
Collaborator Author

Yes, seems duplicated with it.

OK, I'll fix it in other recent PR soon, don't worry.

@WhiredPlanck WhiredPlanck merged commit 7925458 into osfans:develop Oct 8, 2024
4 checks passed
@WhiredPlanck WhiredPlanck deleted the refactors-20241007 branch October 8, 2024 15:55
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