Skip to content

Feat: Optional actions in the screenshot notification#3931

Open
faetalize wants to merge 9 commits into
niri-wm:mainfrom
faetalize:wip/screenshot-config
Open

Feat: Optional actions in the screenshot notification#3931
faetalize wants to merge 9 commits into
niri-wm:mainfrom
faetalize:wip/screenshot-config

Conversation

@faetalize
Copy link
Copy Markdown

@faetalize faetalize commented Apr 28, 2026

EDIT: This PR has now shifted into changing the user facing configuration and providing a new screenshot namespace.

This new feature lives in screenshot { notification {} } now.

screenshot-path has been moved to screenshot { path ""; }, while retaining backwards compat for now.


Adds the ability to configure the screenshot notification with actions.

Supports {path} substitution for the saved screenshot path.

For now, actions are only shown for screenshots saved to disk, since {path} is only available in that case - we could optionally also use clipboard content for cases that involve no persistence (maybe)

Preview:

image

Comment thread src/utils/mod.rs Outdated
Copy link
Copy Markdown
Member

@bluelinden bluelinden left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread docs/wiki/Configuration:-Miscellaneous.md Outdated
Comment thread src/utils/mod.rs Outdated
Comment thread src/utils/mod.rs Outdated
Comment thread src/utils/mod.rs
Comment thread src/protocols/foreign_toplevel.rs
Comment thread src/utils/mod.rs
@Sempyos Sempyos added area:config Config parsing, default config, new settings pr kind:feature New features and functionality labels Apr 28, 2026
@faetalize faetalize changed the title Wip/screenshot config Feat: Optional actions in the screenshot notification Apr 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:config Config parsing, default config, new settings pr kind:feature New features and functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants