-
-
Notifications
You must be signed in to change notification settings - Fork 442
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
Added stream titles to windows toast notifications #2044
Conversation
Co-authored-by: 23rd <23rd@vivaldi.net>
Co-authored-by: 23rd <23rd@vivaldi.net>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
@@ -66,11 +66,12 @@ QString Toasts::findStringFromReaction( | |||
return Toasts::findStringFromReaction(static_cast<ToastReaction>(i)); | |||
} | |||
|
|||
void Toasts::sendChannelNotification(const QString &channelName, Platform p) | |||
void Toasts::sendChannelNotification(const QString &channelName, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
warning: method 'sendChannelNotification' can be made static [readability-convert-member-functions-to-static]
src/singletons/Toasts.hpp:21:
- void sendChannelNotification(const QString &channelName,
+ static void sendChannelNotification(const QString &channelName,
@@ -66,11 +66,12 @@ | |||
return Toasts::findStringFromReaction(static_cast<ToastReaction>(i)); | |||
} | |||
|
|||
void Toasts::sendChannelNotification(const QString &channelName, Platform p) | |||
void Toasts::sendChannelNotification(const QString &channelName, | |||
const QString &channelTitle, Platform p) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
warning: parameter 'channelTitle' is unused [misc-unused-parameters]
const QString &channelTitle, Platform p) | |
const QString & /*channelTitle*/, Platform p) |
Pull request checklist:
CHANGELOG.md
was updated, if applicableDescription
I noticed that toasts are due to be refactored to be cross platform, so this change may or may not be value add at the moment. It's pretty basic, just adding the stream title to the notification and could also be used for the cross platform approach.
Partially closes #1297