Move the charge-announcement style picker to Notification behaviour (#112 follow-up)#151
Merged
Merged
Conversation
…112 follow-up) "Charge notification style" sat on the Alerts screen (per-event rule), but the word "style" reads as appearance, and that is where the maintainer went looking for it. Moved to Notification behaviour > Appearance, next to Sticky notifications. The Alerts screen's empty Charging category is removed; #123's slow-charge toggle will recreate it there when built. Keys and values unchanged - pure XML placement + header-summary text. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015ZMe8rrw7eQKz69R4Xw3t5
almothafar
force-pushed
the
claude/issues-112-113-z070tr
branch
from
62e972c to
e589777
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #112 (PR #140), maintainer decision after using the new menu.
What changed
"Charge notification style" sat on the Alerts screen under a Charging category (per the one-category-per-event rule), but the word "style" reads as appearance — so it now lives on Notification behaviour › Appearance, next to Sticky notifications.
Verification
Static checks (env can't build, #137): XML validity, values ↔ values-ar parity, no dangling references to the removed category string. CI is the compile gate.
🤖 Generated with Claude Code
https://claude.ai/code/session_015ZMe8rrw7eQKz69R4Xw3t5
Generated by Claude Code