Refactor ToolButton implementation and decouple from RimeDaemon

[WhiredPlanck]

Pull request

Issue tracker

Fixes will automatically close the related issues

Fixes #2062

Feature

Describe features of this pull request

Provides a better solution than #2064 and #2065 .

Code of conduct

• CONTRIBUTING

Code style

• make sytle-lint
• Conventional Commits

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

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Refactors toolbar buttons to be “pure views” that react to option updates pushed from InputBarDelegate, and improves toolbar background typing/decoding.

Changes:

• Refactored ToolButton to internalize style selection and expose option + updateStyle(option, enabled) for toggle buttons.
• Added initial syncing of toolbar toggle states from Rime options on startup, and routed subsequent option updates with (option, enabled).
• Changed toolbar background type from String to an enum with safer decoding.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
app/src/main/java/com/osfans/trime/ime/bar/ui/ToolButton.kt	Refactors content/styling logic; adds option-driven style switching; updates background building.
app/src/main/java/com/osfans/trime/ime/bar/ui/ButtonsBarUi.kt	Updates API to forward (option, enabled) to child ToolButtons.
app/src/main/java/com/osfans/trime/ime/bar/ui/AlwaysUi.kt	Updates button style propagation and adds collection of toggle options.
app/src/main/java/com/osfans/trime/ime/bar/InputBarDelegate.kt	Seeds toolbar toggle states from Rime and updates UI on option changes.
app/src/main/java/com/osfans/trime/data/theme/model/ToolBar.kt	Replaces background type string with enum + tolerant decoding.

Suppressed comments (1)

app/src/main/java/com/osfans/trime/data/theme/model/ToolBar.kt:1

• Changing Background.type from String to an enum changes the Parcelable wire format (Parcelize will now parcel an enum instead of a string). If any saved instance state / persisted parcels contain the previous String field layout, restoring after an app update can crash. If this model can be parceled across process recreation, consider keeping a stable serialized representation (e.g., store type: String and derive val typeEnum, or implement a custom Parceler/TypeParceler) to preserve forward-compatibility.

/*

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Bambooin]

Verified with LineageOS 23.

[WhiredPlanck]

Verified with LineageOS 23.

Great! My work isn't in vain at all.