Skip to content

Option to persist notifications#157

Open
joshuamegnauth54 wants to merge 1 commit into
pop-os:masterfrom
joshuamegnauth54:persist-until-dismiss
Open

Option to persist notifications#157
joshuamegnauth54 wants to merge 1 commit into
pop-os:masterfrom
joshuamegnauth54:persist-until-dismiss

Conversation

@joshuamegnauth54

@joshuamegnauth54 joshuamegnauth54 commented Jun 1, 2026

Copy link
Copy Markdown

macOS supports persisting notifications until dismissed. This is useful to keep important notifications around instead of having them time out. Currently, this applies to all apps but it composes well with per-app notification settings (which is a future feature, most likely).

See: pop-os/cosmic-settings#1878


This feature was requested in the linked PR. I'll likely implement per-app settings then move this setting over to that to match macOS. However, this works as intended for now. The easiest way to test it is to use send-notify.

To enable persistence, write true to the persistence settings file.

echo true > ~/.config/cosmic/com.system76.CosmicNotifications/v1/persistence

  • I have disclosed use of any AI generated code in my commit messages.
    • If you are using an LLM, and do not fully understand the changes it is making to the code base, do not create a PR.
    • In our experience, AI generated code often results in overly complex code that lacks enough context for a proper fix or feature inclusion. This results in considerably longer code reviews. Due to this, AI authored or partially authored PRs may be closed without comment.
  • I understand these changes in full and will be able to respond to review comments.
  • My change is accurately described in the commit message.
  • My contribution is tested and working as described.
  • I have read the Developer Certificate of Origin and certify my contribution under its conditions.

macOS supports persisting notifications until dismissed. This is useful
to keep important notifications around instead of having them time out.
Currently, this applies to all apps but it composes well with per-app
notification settings (which is a future feature, most likely).

See: pop-os/cosmic-settings#1878
@wash2 wash2 requested a review from a team June 1, 2026 19:12
@jklgrasso jklgrasso self-assigned this Jun 2, 2026
@hojjatabdollahi

Copy link
Copy Markdown

I suggest this config to be similar to the one I created in #158 . Instead of a boolean, the user can choose None/All/Urgent only. I don't want All my notifications to persist, that's annoying, but I do like to keep the urgent ones around until I dismiss them.

@wash2 wash2 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest this config to be similar to the one I created in #158 . Instead of a boolean, the user can choose None/All/Urgent only. I don't want All my notifications to persist, that's annoying, but I do like to keep the urgent ones around until I dismiss them.

That's a great point!

@joshuamegnauth54

Copy link
Copy Markdown
Author

Sure! 😁 That's a great idea. I'll wait till your patch is merged, and then I'll rebase mine so I could reuse/rename your FullscreenBehavior type.

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.

4 participants