Bug 420035

Summary:	drkonqi UI issues in Wayland session
Product:	[Applications] drkonqi	Reporter:	Andreas Sturmlechner <asturm>
Component:	general	Assignee:	Plasma Bugs List <plasma-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	bugseforuns, nate
Priority:	NOR
Version:	5.18.4
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:	https://commits.kde.org/drkonqi/262389b1e49ba12f6d8df8db71e350fb62698c3e	Version Fixed In:
Sentry Crash Report:

Description Andreas Sturmlechner 2020-04-13 14:15:27 UTC

Having just gone through a bug report using drkonqi inside a Plasma Wayland session, it was not a very pleasant experience.

1. Bug list to browse for similar bugs does not scroll properly, and only seems to refresh on mouse hover - but even then looks very glitchy.

2. The text area do describe the bug did not display anything. So I had to enter my text blindly. :) Also, on the second try parts of the previous step (similar bugs list) were leaking over the text area.

Operating System: Gentoo Linux 
KDE Plasma Version: 5.18.4
KDE Frameworks Version: 5.69.0
Qt Version: 5.14.2
Kernel Version: 5.6.3-gentoo
OS Type: 64-bit
Processors: 8 × Intel® Core™ i7-4790K CPU @ 4.00GHz
Memory: 15.4 GiB of RAM

Comment 1 Patrick Silva 2020-04-13 17:34:40 UTC

Second issue was already reported, see bug 418755.

Comment 2 David Edmundson 2020-04-14 16:48:39 UTC

Git commit 262389b1e49ba12f6d8df8db71e350fb62698c3e by David Edmundson.
Committed on 14/04/2020 at 16:48.
Pushed by davidedmundson into branch 'Plasma/5.18'.

Avoid creating subwindow on internal page widget

Summary:
KWallet::openWallet takes a windowId as a parameter.

QWidget->winId() is a dangerous call, it will create an underlying
platform window when called. This happens even if we're not the toplevel
widget. On X11 this passed winId must have always been broken, but not
in a way that caused a problem.

This leaves QtWayland in a very confused state with the side effect that
all scrolling widgets become broken.
Related: bug 418755

Test Plan:
Bisected drkonqi till we found the cause
Verified fix by running crashtest (with some mods so I could report a bug)
I can't confirm the winId is correct on X11, but the code looks safe

Reviewers: #plasma, sitter

Reviewed By: sitter

Subscribers: sitter, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D28832

M  +10   -2    src/bugzillaintegration/reportassistantpages_bugzilla.cpp

https://commits.kde.org/drkonqi/262389b1e49ba12f6d8df8db71e350fb62698c3e