Bug 420035 - drkonqi UI issues in Wayland session
Summary: drkonqi UI issues in Wayland session
Status: RESOLVED FIXED
Alias: None
Product: drkonqi
Classification: Applications
Component: general (show other bugs)
Version: 5.18.4
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-04-13 14:15 UTC by Andreas Sturmlechner
Modified: 2020-04-14 16:48 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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