Bug 424795 - Unfocused windows don't get brought forward when activated externally
Summary: Unfocused windows don't get brought forward when activated externally
Status: RESOLVED UPSTREAM
Alias: None
Product: kwin
Classification: Plasma
Component: wayland-generic (show other bugs)
Version: git master
Platform: Other Linux
: VHI normal
Target Milestone: ---
Assignee: KWin default assignee
URL: https://gitlab.freedesktop.org/waylan...
Keywords: usability, wayland
: 427445 432913 439567 440032 441866 444357 445128 448671 450926 451377 451504 (view as bug list)
Depends on:
Blocks: 429770
  Show dependency treegraph
 
Reported: 2020-07-29 12:38 UTC by Andrey
Modified: 2023-01-21 13:34 UTC (History)
25 users (show)

See Also:
Latest Commit:
Version Fixed In: Qt 6.x


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey 2020-07-29 12:38:30 UTC
SUMMARY
Browser does load the link, it's just it's window is not activated

STEPS TO REPRODUCE
1. Open some link in konsole via it's "Open Link" context menu action. 

OBSERVED RESULT
It seems like nothing happens. Actually, the link is opened in browser on background plane.

EXPECTED RESULT
Browser window with loaded link should bring to first plane.

Operating System: Ubuntu 20.04
KDE Plasma Version: 5.19.80
KDE Frameworks Version: 5.73.0
Qt Version: 5.14.2
Comment 1 Nate Graham 2020-07-29 20:05:54 UTC
Confirmed, I believe this is a known issue but I don't know which bug is tracking it.
Comment 2 David Edmundson 2020-08-05 12:46:09 UTC
It is a known issue, there is no specification to do this.

GTK use a custom one which doesn't help anyone.
Comment 3 Nate Graham 2020-08-12 22:00:13 UTC
So someone has to propose one and then all the compositors have to implement it?
Comment 4 David Edmundson 2020-08-12 22:28:09 UTC
and all toolkits
Comment 5 Nate Graham 2020-08-12 22:33:10 UTC
Is the custom GNOME implementation upstreamable, or would we/someone need to start from scratch?
Comment 6 Nate Graham 2020-10-11 03:18:11 UTC
*** Bug 427445 has been marked as a duplicate of this bug. ***
Comment 7 Seqularise 2020-10-26 13:21:24 UTC
Can anyone please inform us about this bug? is there any work behind this? In fact, I personally would be fine if some hacks belonging to gnome, kde or wlroots (doesnt matter) were used to solve this problem.
Comment 8 Nate Graham 2020-10-26 13:23:12 UTC
Yes, a cross-desktop protocol is being worked on upstream: https://gitlab.freedesktop.org/wayland/wayland-protocols/-/merge_requests/50
Comment 9 Nate Graham 2021-02-16 20:09:42 UTC
*** Bug 432913 has been marked as a duplicate of this bug. ***
Comment 10 Nate Graham 2021-05-05 16:14:24 UTC
Since the Wayland protocol has been merged, this isn't blocked by anything anymore and we can implement support on our side.
Comment 11 David Edmundson 2021-05-21 15:43:43 UTC
I don't know why this was reopened. We know we don't support this yet.
Comment 12 Nate Graham 2021-05-21 15:53:42 UTC
Because with the activation protocol merged upstream, we now can, and having a bug report open to track that work is reasonable.
Comment 13 David Edmundson 2021-07-06 17:16:05 UTC
*** Bug 439567 has been marked as a duplicate of this bug. ***
Comment 14 Patrick Silva 2021-07-20 19:18:39 UTC
*** Bug 440032 has been marked as a duplicate of this bug. ***
Comment 15 Nate Graham 2021-09-02 17:48:19 UTC
*** Bug 441866 has been marked as a duplicate of this bug. ***
Comment 16 David Edmundson 2021-09-05 23:14:14 UTC
This has changed a lot in 5.23.

It won't fix everything at once, but leaving this open won't help with that.
Comment 17 Nate Graham 2021-09-07 13:36:52 UTC
I haven't noticed any changes in git master here. What specific changes need to be implemented to make it work in apps? Where should bug reports be filed?
Comment 18 Nate Graham 2021-09-10 16:05:05 UTC
OK, I have begun to file bug reports on individual use case which are still broken (for me, all of them):

Bug 442264
Bug 442265
Bug 442266
Bug 442267
Bug 442268
Bug 442269

Should I continue? Is this useful?
Comment 19 David Edmundson 2021-09-10 16:10:27 UTC
>Is this useful?

Not really.

It's only useful if the receiving software is known to have activation support. Otherwise they need to be adjusted before we can start investigating any potential issues on our side.
Comment 20 Nate Graham 2021-09-10 16:22:30 UTC
So literally every app/toolkit needs to be changed to opt into the activation protocol? I fear that this is going to feel broken forever... :(
Comment 21 Patrick Silva 2021-09-18 00:53:14 UTC
This bug is still reproducible on Plasma 5.23 beta even with KDE software.

1. open a text file with Kate via Dolphin
2. minimize Kate
3. open another text file with Kate via Dolphin

Result: Kate is still minimized
Comment 22 Nate Graham 2021-09-18 00:56:13 UTC
KDE apps won't be fixed until activation support ia added into Qt. It's still in progress; see https://codereview.qt-project.org/c/qt/qtwayland/+/321246
Comment 23 Patrick Silva 2021-10-25 11:25:04 UTC
*** Bug 444357 has been marked as a duplicate of this bug. ***
Comment 24 Patrick Silva 2021-11-07 19:51:10 UTC
*** Bug 445128 has been marked as a duplicate of this bug. ***
Comment 25 Patrick Silva 2022-01-17 21:14:08 UTC
*** Bug 448671 has been marked as a duplicate of this bug. ***
Comment 26 postix 2022-01-17 21:40:38 UTC
(In reply to Nate Graham from comment #22)
> KDE apps won't be fixed until activation support ia added into Qt. It's
> still in progress; see
> https://codereview.qt-project.org/c/qt/qtwayland/+/321246

It's merged, but I can still reproduce 5.24.80 on KDE Neon Unstable see #448671.
Comment 27 Nate Graham 2022-01-17 21:42:18 UTC
That's only in Qt6. Perhaps we should look into backporting it in our patch collection. It's a pretty huge change though, and stretches the definition of a bugfix. If fixing a bug required adding a major new feature, is it really a bugfix?
Comment 28 Patrick Silva 2022-01-18 11:51:04 UTC
Strawberry music player is built against Qt 6 on Arch Linux and this bug is reproducible with it.
Comment 29 Andrius Štikonas 2022-01-18 11:52:49 UTC
(In reply to Patrick Silva from comment #28)
> Strawberry music player is built against Qt 6 on Arch Linux and this bug is
> reproducible with it.

Is it using Qt6 with the patch (which I guess is Qt6 master)?
Comment 30 Patrick Silva 2022-01-18 12:06:08 UTC
I do not know. Which Qt version will include this fix/feature?
Comment 31 Andrius Štikonas 2022-01-18 12:08:06 UTC
(In reply to Patrick Silva from comment #30)
> I do not know. Which Qt version will include this fix/feature?

Probably 6.3.0
Comment 32 Nate Graham 2022-01-18 15:25:41 UTC
Both the sending and receiving apps need to implement activation support, or be using a toolkit that does it automatically.

It's not enough for one of the apps to be using a compatible version of Qt6; the other app that sent the activation request have to be using it, too. So currently this will only reliably work between Qt6 apps, or GTK4 apps, or between GTK4 and Qt6 apps.

It sucks, I know. I'm somewhat baffled baffled by the fact that this wasn't, like, built into the core Wayland protocol. But it is what it is, and at least thanks To Aleix, we have it now.
Comment 33 Patrick Silva 2022-01-18 15:39:01 UTC
Thanks for clarifying Nate. Probably we will have many duplicates of this bug in the next years.
Comment 34 Nate Graham 2022-01-18 15:39:39 UTC
Indeed. :(
Comment 35 postix 2022-01-18 16:38:40 UTC
(In reply to Nate Graham from comment #32)
> It sucks, I know. I'm somewhat baffled baffled by the fact that this wasn't,
> like, built into the core Wayland protocol.

Just to understand the situation: Has this been built into the (core) Wayland protocol by now or if not, wouldn't it make sense to put it in there now, as of "better later than never"?
Comment 36 Andrius Štikonas 2022-01-18 16:44:56 UTC
(In reply to postix from comment #35)
> Just to understand the situation: Has this been built into the (core)
> Wayland protocol by now or if not, wouldn't it make sense to put it in there
> now, as of "better later than never"?

So Wayland 2.0?
Comment 37 Nate Graham 2022-01-18 16:49:10 UTC
The activation protocol was done as an extension to the core protocol. I don't know if the core protocol itself is still open for changes. Someone who understands Wayland better than me would have to answer that question.
Comment 38 Patrick Silva 2022-02-28 23:44:31 UTC
*** Bug 450926 has been marked as a duplicate of this bug. ***
Comment 39 Patrick Silva 2022-03-12 17:02:22 UTC
*** Bug 451377 has been marked as a duplicate of this bug. ***
Comment 40 Patrick Silva 2022-03-14 19:30:10 UTC
*** Bug 451504 has been marked as a duplicate of this bug. ***