Skip to content

feat(notifications): option added to disable permanent notifications#3065

Open
Nivalux wants to merge 1 commit into
noctalia-dev:mainfrom
Nivalux:notification_permanent
Open

feat(notifications): option added to disable permanent notifications#3065
Nivalux wants to merge 1 commit into
noctalia-dev:mainfrom
Nivalux:notification_permanent

Conversation

@Nivalux

@Nivalux Nivalux commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary & Motivation

My webbrowser sends notifications as permanent notifications (timeout = 0). This sometimes results in a big list of notifications, that I have to click away manually on browser startup.
This PR adds a setting, which allows disabling permanent notifications and have them timeout normally.

Edit: Also added the option to the filter. It allows enabling or disabling permanent notifications per application.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactoring
  • Build / packaging

Related Issue

Testing

Running notify-send -t 0 Test before and after setting the allow_permanent option.

Manual Coverage

  • Tested on Niri
  • Tested on Hyprland
  • Tested on Sway
  • Tested on another compositor:
  • Tested with different bar positions and density settings
  • Tested at different interface scaling values
  • Tested with multiple monitors

Screenshots / Videos

Checklist

  • This PR is ready for review, or it is marked as Draft.
  • I read and followed the relevant guidance in CONTRIBUTING.md.
  • I ran just format with clang-format v22+ installed, or this PR has no code changes.
  • I ran the relevant build or test commands, or explained why they were not run.
  • I self-reviewed the changes.
  • I checked for new warnings or errors.
  • I will update end-user documentation after merge, or this PR does not change user-facing configuration or behavior.
  • I added or updated assets/translations/en.json, or this PR adds no new user-facing strings.
  • I did not edit non-English translation files unless this PR is explicitly for translation tooling, an import/export sync, or a maintainer-requested locale change.
  • I used the existing canonical names for config keys, IPC names, paths, and identifiers.

Additional Notes

@Nivalux Nivalux marked this pull request as draft June 18, 2026 10:32
@Nivalux Nivalux force-pushed the notification_permanent branch from c42d0f7 to 559f56d Compare June 18, 2026 12:21
@Nivalux Nivalux marked this pull request as ready for review June 18, 2026 12:23
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