Mir

Qt apps not fullscreening under miral-kiosk

Bug #1717910 reported by Gerry Boland on 2017-09-18
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mir
Fix Released
Critical
Alan Griffiths

Bug Description

Running:
./bin/miral-kiosk --x11-displays=1200x800

and for example:
QT_QPA_PLATFORM=ubuntumirclient /usr/lib/x86_64-linux-gnu/qt5/examples/widgets/mainwindows/mainwindow/mainwindow

results in a not-fullscreen window in the Miral shell.

Digging I see that the kiosk window manager isn't implementing place_new_window. Adding this simple patch: https://pastebin.ubuntu.com/25564661/ does fix the issue.

I will admit Qtubuntu probably has an issue too, as the kiosk WM does send resize events in advise_new_window, but Qt seems to be ignoring them

Related branches

Alan Griffiths (alan-griffiths) wrote :

The patch assumes a single output. The correct logic is in KioskWindowManagerPolicy::advise_new_window() - which is the wrong place but worked with gtk-mir.

We don't really care about gtk-mir, so moving that code to KioskWindowManagerPolicy::place_new_window() is probably the best, as it likely works with Qt and is the correct place for it.

Changed in mir:
status: New → Triaged
importance: Undecided → Critical
milestone: none → 1.0.0
Alan Griffiths (alan-griffiths) wrote :

But the actual problem is that we don't handle Qt creating "hidden" windows and then "restoring" them correctly.

Changed in mir:
assignee: nobody → Alan Griffiths (alan-griffiths)
status: Triaged → In Progress
Changed in mir:
status: In Progress → Fix Committed
Changed in mir:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers