Bug 474166

Summary:	KRunner opens on wrong screen
Product:	[Plasma] krunner	Reporter:	Natalie Clarius <natalie_clarius>
Component:	general	Assignee:	Plasma Bugs List <plasma-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	alexander.lohnau, natalie_clarius, nate, qydwhotmail
Priority:	NOR	Keywords:	qt6
Version First Reported In:	master
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/plasma-workspace/-/commit/80c0903a03f06259f9a70729808fd3e1cb45e0b5	Version Fixed/Implemented In:
Sentry Crash Report:

Description Natalie Clarius 2023-09-05 00:54:45 UTC

SUMMARY
***
In the Plasma dev session, KRunner opens on the wrong screen; seemingly consistently the same screen within one session, but not consistently the primary, leftmost or some specific screen across sessions.
***

SOFTWARE/OS VERSIONS
git master everything

ADDITIONAL INFORMATION
I added some debug statements and it seems that https://invent.kde.org/plasma/plasma-workspace/-/blob/master/krunner/view.cpp#L152 correctly determines the correct screen, but is only executed the first time KRunner is opened in a session, so subsequent launches would just place it on the screen that happened to be active the first time it was opened.

Comment 1 Bug Janitor Service 2023-09-05 01:15:06 UTC

A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/3255

Comment 2 Nate Graham 2023-09-05 01:57:58 UTC

*** Bug 473528 has been marked as a duplicate of this bug. ***

Comment 3 David Redondo 2023-09-06 08:28:51 UTC

Git commit 80c0903a03f06259f9a70729808fd3e1cb45e0b5 by David Redondo, on behalf of Natalie Clarius.
Committed on 06/09/2023 at 09:53.
Pushed by nclarius into branch 'master'.

krunner: fix window showing on wrong screen

Call positionOnScreen every time setVisible is set and not just during initialization of View, so that the correct screen can be determined whenever the window is shown, rather than always using the screen that happened to be active the first time KRunner was launched.

M  +10   -5    krunner/view.cpp

https://invent.kde.org/plasma/plasma-workspace/-/commit/80c0903a03f06259f9a70729808fd3e1cb45e0b5