Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

notification enhancements: icons, etc #1492

Closed
totaam opened this issue Apr 9, 2017 · 15 comments
Closed

notification enhancements: icons, etc #1492

totaam opened this issue Apr 9, 2017 · 15 comments

Comments

@totaam
Copy link
Collaborator

totaam commented Apr 9, 2017

Issue migrated from trac ticket # 1492

component: core | priority: major | resolution: fixed

2017-04-09 12:07:03: antoine created the issue


See desktop notification specification.

The icons could be png encoded.
The actions would require us to check that the client supports the feature before we expose the server-side capability.

@totaam
Copy link
Collaborator Author

totaam commented Nov 14, 2017

2017-11-14 14:09:07: antoine commented


See #1688, #1375

@totaam
Copy link
Collaborator Author

totaam commented Nov 26, 2017

2017-11-26 13:37:09: antoine changed status from new to assigned

@totaam
Copy link
Collaborator Author

totaam commented Jan 7, 2018

2018-01-07 05:56:17: antoine edited the issue description

@totaam
Copy link
Collaborator Author

totaam commented Jan 7, 2018

2018-01-07 05:56:17: antoine commented


new spec link (had gone 404)

@totaam
Copy link
Collaborator Author

totaam commented Jan 7, 2018

2018-01-07 09:02:08: antoine commented


Important bug fix in r17933: send icon data not path (which is often at best meaningless for a remote client).
Minimal backport in 17934.

@totaam
Copy link
Collaborator Author

totaam commented Jan 7, 2018

2018-01-07 12:45:56: antoine commented


More related changes:

To test:

notify-send 'Hello world!' 'This is an example notification.' \
    --icon=/usr/share/xpra/icons/xpra.png

Already tested with:

Note: some specific desktop environment may decide to block or hide the notifications..

Moving support for notification actions to #1735.

@totaam
Copy link
Collaborator Author

totaam commented Jan 8, 2018

2018-01-08 06:50:53: antoine changed status from assigned to new

@totaam
Copy link
Collaborator Author

totaam commented Jan 8, 2018

2018-01-08 06:50:53: antoine changed owner from antoine to maxmylyn

@totaam
Copy link
Collaborator Author

totaam commented Jan 8, 2018

2018-01-08 06:50:53: antoine commented


@maxmylyn: mostly a FYI, see comment:5.

@totaam
Copy link
Collaborator Author

totaam commented Jan 9, 2018

2018-01-09 17:05:20: maxmylyn changed status from new to closed

@totaam
Copy link
Collaborator Author

totaam commented Jan 9, 2018

2018-01-09 17:05:20: maxmylyn set resolution to fixed

@totaam
Copy link
Collaborator Author

totaam commented Jan 9, 2018

2018-01-09 17:05:20: maxmylyn commented


Noted and closing.

@totaam totaam closed this as completed Jan 9, 2018
@totaam
Copy link
Collaborator Author

totaam commented Jan 13, 2018

2018-01-13 05:37:15: antoine changed title from notification enhancements: icons, actions, etc to notification enhancements: icons, etc

@totaam
Copy link
Collaborator Author

totaam commented Jan 13, 2018

2018-01-13 05:37:15: antoine commented


r17973 (+r17974): the macos python3 client can now use the gtk notifier

(edit ticket title since "actions" have been moved to #1735)

@totaam
Copy link
Collaborator Author

totaam commented Jan 30, 2018

2018-01-30 06:15:37: antoine commented


r18195: when we need to show actions or when we don't have a native tray to bind to, fallback to using the gtk notifier.

This helps with #1688

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

No branches or pull requests

1 participant