data/linux.x86.linphone
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: show notification only on primary screen

Browse source 
Prevents multiple notification windows from appearing (one per screen)
when receiving an incoming call. Now only shows on the primary display.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Eduard Wisch 2026-02-06 18:08:28 +01:00
parent d00cff194f
commit ccd5a806e2
1 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
Linphone/core/notifier/Notifier.cpp
View file

@ -135,8 +135,8 @@ bool Notifier::createNotification(Notifier::NotificationType type, QVariantMap d
mMutex->unlock();
return false;
}
QList<QScreen *> allScreens = QGuiApplication::screens();
if (allScreens.size() > 0) { // Ensure to have a screen to avoid errors
QScreen *primaryScreen = QGuiApplication::primaryScreen();
if (primaryScreen) { // Ensure to have a screen to avoid errors
QQuickItem *previousWrapper = nullptr;
bool showAsTool = false;
#ifdef Q_OS_MACOS
@ -149,12 +149,12 @@ bool Notifier::createNotification(Notifier::NotificationType type, QVariantMap d
}
}
#endif
for (int i = 0; i < allScreens.size(); ++i) {
// Only show notification on primary screen (not on all screens)
{
++mInstancesNumber;
// Use QQuickView to create a visual root object that is
// independant from current application Window
QScreen *screen = allScreens[i];
QScreen *screen = primaryScreen;
auto engine = App::getInstance()->mEngine;
const QUrl url(QString(NotificationsPath) + Notifier::Notifications[type].filename);
QObject::connect(