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

Add "transient" hint for notification #18

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Chimstaz
Copy link

@Chimstaz Chimstaz commented Feb 8, 2018

I think that adding this flag make sense. For example when using xfce4-notifyd and enabled logging, every change of volume is stored in history, but if notification is marked as transient, then it's skipped in log.

I think that adding this flag make sense. For example when using xfce4-notifyd and enabled logging, every change of volume is stored in history, but if notification is marked as transient, then it's skipped in log.
Riesi pushed a commit to Riesi/pa-applet-revamp that referenced this pull request Aug 6, 2018
… notify text

Changed the right click display to sinks instead of options of the default sink. This allows for quickly selecting a new default sink. Maybe a bit dirty for now, but it works quite well. 
Fixed Notification only displaying pa-applet 
Changed deprecated function notify_notification_set_hint_int32 to notify_notification_set_hint 
Changed Notification info output 
Removed textual description of volume intensity 
added transient flag from Pull-request: 
fernandotcl#18 
Added some TODO markers for finding the deprecated funktions for later modification maybe
Write some additional infos into the README
@fernandotcl
Copy link
Owner

Is this addressed by #18? If so, I'd rather merge that.

@Chimstaz
Copy link
Author

Yes, this commit contains changes from this PR.

@Maelan
Copy link

Maelan commented Jun 14, 2020

Hi, what prevents this PR from being merged?

@fernandotcl
Copy link
Owner

Lack of time and interest in the project, sadly.

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.

3 participants