Skip to content

feat: visible X close button on overlay notifications#464

Merged
garysheng merged 1 commit intoPeonPing:mainfrom
smurfy92:feat/close-button
Apr 14, 2026
Merged

feat: visible X close button on overlay notifications#464
garysheng merged 1 commit intoPeonPing:mainfrom
smurfy92:feat/close-button

Conversation

@smurfy92
Copy link
Copy Markdown
Contributor

Summary

Adds a discoverable `×` button in the top-right corner of overlay notifications (glass, sakura, jarvis themes). Helps users who don't realize the whole overlay is clickable to dismiss.

New config

  • `notification_close_button` (bool, default `true`)

Test plan

  • Verify X appears on each theme
  • Click X → verify dismiss
  • Set `notification_close_button: false` → verify X is hidden but overlay still dismissible via body click

🤖 Generated with Claude Code

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 14, 2026

@smurfy92 is attempting to deploy a commit to the Gary Sheng's projects Team on Vercel.

A member of the Team first needs to authorize it.

Adds a discoverable × button in the top-right corner of overlay
notifications. Clicking the X or anywhere else on the overlay dismisses
it the same way (reusing the existing dismiss handler).

New config key: notification_close_button (default: true)

Useful for users who want an obvious way to dismiss notifications
without having to guess that clicking the body works.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@garysheng garysheng merged commit 8fc5348 into PeonPing:main Apr 14, 2026
2 checks passed
garysheng added a commit that referenced this pull request Apr 14, 2026
- peon setup wizard (#465, closes #283)
- session-based notification stacking (#463)
- X close button on overlays (#464)
- notification_title_marker config (#457)
- expanded peon status (#461)
- NSPanel overlay focus fix (#462)
- overlay label + suppression fix (#460)
- pack sync robustness (#453)
- linux notify-send dismiss time (#455)
- nix zsh module (#456)
- Warcraft III Orc peon mapping fix in README EN/zh/ja (closes #333)
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