Skip to content

refactor: enhance processing of nullable theme parameters#1690

Merged
WhiredPlanck merged 1 commit into
osfans:developfrom
WhiredPlanck:theme-tweak-0712
Jul 13, 2025
Merged

refactor: enhance processing of nullable theme parameters#1690
WhiredPlanck merged 1 commit into
osfans:developfrom
WhiredPlanck:theme-tweak-0712

Conversation

@WhiredPlanck

Copy link
Copy Markdown
Collaborator

Pull request

Issue tracker

Fixes will automatically close the related issues

Fixes # N/A

Feature

Describe features of this pull request

  • Use native (de)serializer for style/enter_label, style/layout and preset_keys
  • Remove parameters: background_dim_amount, color_scheme, keyboards, locale and latin_lacale in style
  • Set default values for data classes

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

- Use native (de)serializer for `style/enter_label`, `style/layout` and `preset_keys`
- Remove parameters: `background_dim_amount`, `color_scheme`, `keyboards`, `locale` and `latin_lacale` in `style`
- Set default values for data classes
@WhiredPlanck
WhiredPlanck requested review from Bambooin and goofyz July 12, 2025 14:30
@WhiredPlanck
WhiredPlanck merged commit 8481dee into osfans:develop Jul 13, 2025
3 checks passed
@WhiredPlanck
WhiredPlanck deleted the theme-tweak-0712 branch July 13, 2025 09:19
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