Skip to content

For the borderless window to work on X11, flags must be reset first#137

Open
Hellzed wants to merge 3 commits into
ngld:masterfrom
Hellzed:fix-x11-borderless-window
Open

For the borderless window to work on X11, flags must be reset first#137
Hellzed wants to merge 3 commits into
ngld:masterfrom
Hellzed:fix-x11-borderless-window

Conversation

@Hellzed

@Hellzed Hellzed commented Aug 22, 2018

Copy link
Copy Markdown

Before this, main window title bar still appears on some window managers (eg. GNOME). From what I gather, we need to reset the full set of flags to something customizable with self.setWindowFlags(QtCore.Qt.CustomizeWindowHint) first before setting an individual flag with self.setWindowFlag().
At the moment, I don't have a PC or Mac to check this doesn't cause a regression elsewhere.

@ngld

ngld commented Aug 22, 2018

Copy link
Copy Markdown
Owner

Hm... does this work with self.setWindowFlag(QtCore.Qt.CustomizeWindowHint, enabled)?
I think the way you implemented it breaks if a user deactivates the custom titlebar.
Also, please disable your automated error reports if you change anything code related. Receiving those errors only makes sifting through the error reports harder.

@Hellzed

Hellzed commented Aug 22, 2018

Copy link
Copy Markdown
Author

Sorry for the spam, I nuked my .knossos and forgot it was the default crash report upload.
Actually deactivating the custom title bar was broken before... Qt window flags are weird, but manually resetting the QtCore.Qt.Window works for me when disabling the custom title bar.

@ngld

ngld commented Aug 31, 2018

Copy link
Copy Markdown
Owner

Sorry for the later reply. Unfortunately, I can't accept the PyQt5 update since it breaks stuff on Windows for several reasons: a) It has weird bugs in QtWebEngine (selects open in the wrong place) and b) The 32bit version of it no longer ships with 32bit builds.

Can you tell me more about the crash?

@Hellzed

Hellzed commented Sep 1, 2018

Copy link
Copy Markdown
Author

Here's the bug: https://bugreports.qt.io/browse/QTBUG-66346
It affects series 5.9.4 and 5.10.1... It was fixed in 5.9.5 and 5.11, but not in 5.10 series.
With PyQt, instead of broken rendering, it just crashes.
Workarounds mentioned in the bug comments do help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants