Bug 412020

Summary:	Certain style/themes cause Konsole split to create new window
Product:	[Plasma] Oxygen	Reporter:	Kurt Hindenburg <khindenburg>
Component:	general	Assignee:	Unassigned bugs <unassigned-bugs-null>
Status:	REPORTED ---
Severity:	normal	CC:	justin.zobel, maxmustermann1884, nate, nik
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Kurt Hindenburg 2019-09-18 13:22:33 UTC

STEPS TO REPRODUCE
1. Change to Oxygen Application Style
2. Open konsole and create new splits - the exact method I haven't figured out yet but it is easy to do
3. Eventually a new split will open a new window with an unusable terminal


Noticed while testing/fixing  https://bugs.kde.org/show_bug.cgi?id=411387 - there are some hardcoded issues that need adjusted for themes/styles.

Comment 1 Kurt Hindenburg 2019-09-18 13:25:02 UTC

A ton of these also show up after the new window appears

qt.qpa.xcb: QXcbConnection: XCB error: 8 (BadMatch), sequence: 9742, resource id: 96469018, major code: 130 (Unknown), minor code: 3

Comment 2 Kurt Hindenburg 2019-10-02 15:53:48 UTC

Git commit 63ef8872bfc485248a8de6c7b45909afc68bfcc9 by Kurt Hindenburg.
Committed on 02/10/2019 at 15:52.
Pushed by hindenburg into branch 'master'.

Fix split view crashes when using keyboard navigation with some themes

The current code uses hard-coded values for some theme items.  This
fix uses the scrollbar width.  There are other theme issues with split
views.
Test using Oxygen application style (+ increase scrollbar width)

Most patch by Nikolay Zlatev <nik@astrapaging.com>
Related: bug 411387

M  +11   -8    src/ViewSplitter.cpp

https://invent.kde.org/kde/konsole/commit/63ef8872bfc485248a8de6c7b45909afc68bfcc9

Comment 3 Kurt Hindenburg 2019-10-02 15:53:51 UTC

Git commit 63ef8872bfc485248a8de6c7b45909afc68bfcc9 by Kurt Hindenburg.
Committed on 02/10/2019 at 15:52.
Pushed by scmsync into branch 'master'.

Fix split view crashes when using keyboard navigation with some themes

The current code uses hard-coded values for some theme items.  This
fix uses the scrollbar width.  There are other theme issues with split
views.
Test using Oxygen application style (+ increase scrollbar width)

Most patch by Nikolay Zlatev <nik@astrapaging.com>
Related: bug 411387

M  +11   -8    src/ViewSplitter.cpp

https://commits.kde.org/konsole/63ef8872bfc485248a8de6c7b45909afc68bfcc9

Comment 4 Nikolay Zlatev 2019-10-03 14:06:40 UTC

I have tested with Breeze, kvantum, Oxygen, Windows, Fusion

This issue appear only on Oxygen, and only with compositor enabled.

If compositor is disabled everything works fine.
Maybe this is Oxygen/Kwin problem.

Comment 5 Justin Zobel 2020-11-03 06:48:33 UTC

(In reply to Nikolay Zlatev from comment #4)
> I have tested with Breeze, kvantum, Oxygen, Windows, Fusion
> 
> This issue appear only on Oxygen, and only with compositor enabled.
> 
> If compositor is disabled everything works fine.
> Maybe this is Oxygen/Kwin problem.

It certainly appears so, have reassigned to Oxygen.

Comment 6 maxmustermann1884 2021-02-21 12:16:50 UTC

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