Skip to content

Refine battery charge limit card layout - #517

Merged
onehoon merged 2 commits into
mainfrom
refactor/battery-card-layout
Sep 12, 2026
Merged

onehoon merged 2 commits into
mainfrom
refactor/battery-card-layout

Conversation

@onehoon

@onehoon onehoon commented Sep 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • Move the dynamic current/desired battery status line into the Battery Charge Limit card description area, directly below the title.
  • Remove the static Set the maximum charge level... description.
  • Keep the percentage value on the left and the enable toggle on the right of the slider.
  • Explicitly center the slider and toggle vertically in the control row.
  • Keep the existing snapshot-driven percentage and enabled-state synchronization unchanged.

Validation

  • Focused battery-card layout test: passed.
  • Full test suite: 2736/2736 passed.
  • Release build: 0 warnings, 0 errors.
  • git diff --check passed.
  • Physical MSI Claw visual/input validation remains manual.

Scope

This PR changes only the production Battery Charge Limit card presentation. Battery protocol, persistence, lifecycle, and mutation behavior are unchanged.

@onehoon
onehoon marked this pull request as ready for review September 12, 2026 10:52
@onehoon
onehoon merged commit 96067ab into main Sep 12, 2026
1 check passed
@onehoon
onehoon deleted the refactor/battery-card-layout branch September 13, 2026 09:01
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.

1 participant