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 v2 specs: Allow passing GFileIcon, sound and add additional properties #147

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

jsparber
Copy link

@jsparber jsparber commented Mar 8, 2024

Implement the new portal frontend in libportal flatpak/xdg-desktop-portal#1298

@jsparber jsparber force-pushed the implement_v2_notification_spec branch 2 times, most recently from bbe8883 to 57d6202 Compare April 11, 2024 14:38
@jsparber jsparber changed the title Implement v2 notification spec notification v2 specs: Allow passing GFileIcon, sound and add additional properties Apr 11, 2024
@jsparber jsparber marked this pull request as ready for review April 16, 2024 13:31
libportal/notification.c Outdated Show resolved Hide resolved
libportal/notification.c Outdated Show resolved Hide resolved
libportal/notification.c Show resolved Hide resolved
libportal/notification.c Outdated Show resolved Hide resolved
libportal/notification.c Outdated Show resolved Hide resolved
@jsparber jsparber force-pushed the implement_v2_notification_spec branch 3 times, most recently from 050fe36 to 03dd58c Compare April 23, 2024 14:41
libportal/notification.c Outdated Show resolved Hide resolved
@jsparber jsparber force-pushed the implement_v2_notification_spec branch 2 times, most recently from 595a948 to ff872a1 Compare April 23, 2024 15:48
@swick
Copy link

swick commented Apr 23, 2024

Might want to put the autofd stuff into a new commit but otherwise this LGTM.

Notification portal v2 allows passing a file-descriptors, so implement
support for GFileIcon and convert it to a file-descriptor icon.
Notification portal v2 has a sound property allow users to set a sound
file and get file-descriptor to the file that can be passed to the
portal.
The notification specs v2 have some optional properties. This adds a
method for querying those.
@jsparber jsparber force-pushed the implement_v2_notification_spec branch from ff872a1 to 7c39eb4 Compare April 25, 2024 11:10
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.

None yet

2 participants