fix: update badge visibility and daemon restart#135
Merged
CalebisGross merged 2 commits intomainfrom Mar 14, 2026
Merged
Conversation
White text on green background instead of --bg-primary (near-black), slightly larger font and padding, nowrap to prevent text wrapping. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Fix badge text invisible due to nav-brand's -webkit-text-fill-color: transparent cascading to children. Reset text-fill-color and background-clip on the badge element. - Fix badge font-size overridden by .nav-brand span rule (1.3rem). Add specific .nav-brand .update-badge override. - Fix daemon self-restart: Stop()+Start() killed the process before Start() could run. Add Restart() to ServiceManager interface that uses non-blocking exec (systemctl restart / launchctl stop+start / net stop+start) so the command outlives the dying process. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Summary
Follow-up fixes for the self-update feature (#132):
.nav-branduses-webkit-text-fill-color: transparentfor its gradient effect, which cascaded to the badge. Reset text-fill-color and background-clip on the badge..nav-brand spanforced1.3remon all child spans. Add specific.update-badgeoverride.Stop()+Start()sent SIGTERM to the running process, soStart()never executed. AddedRestart()toServiceManagerinterface using non-blockingexec.Command.Start()(systemctl --user restart/launchctl stop+start/net stop+start) so the command outlives the dying process.Test plan
make build && make checkpassgolangci-lint runclean on all changed packages🤖 Generated with Claude Code