[unity8] It's impossible to close some dialogs (missing 'X' window button)

Bug #1668053 reported by dinamic
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
High
Michał Sawicz
Mir
Invalid
Undecided
Unassigned
Ubuntu UX
New
Undecided
Unassigned
qtmir (Ubuntu)
Invalid
High
Unassigned
qtubuntu (Ubuntu)
Invalid
High
Unassigned
unity8 (Ubuntu)
Fix Released
High
Lukáš Tinkl

Bug Description

ubuntu 17.04 Unity 8
[unity8] kate, impossible to close "Print Preview" window

see attached screenshot
Print Preview window doesn't have windows controls or a close button

Launch Kate, from the menu File > Print Preview
try to close the window, you can't because doh

Related branches

Revision history for this message
dinamic (dinamic6661) wrote :
dinamic (dinamic6661)
description: updated
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

I'm not sure but AFAIK Mir does not yet have the sufficient client API for child windows to specify whether they support close/minimize/maximize (which is independent information from the window type). It's possible we're trying to guess the right feature set right now.

tags: added: clientapi
Changed in ubuntu-ux:
status: New → Invalid
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in qtmir (Ubuntu):
status: New → Confirmed
Changed in qtubuntu (Ubuntu):
status: New → Confirmed
Changed in unity8 (Ubuntu):
status: New → Confirmed
kevin gunn (kgunn72)
Changed in canonical-devices-system-image:
importance: Undecided → High
Changed in qtmir (Ubuntu):
importance: Undecided → High
Changed in qtubuntu (Ubuntu):
importance: Undecided → High
Changed in unity8 (Ubuntu):
importance: Undecided → High
no longer affects: mir
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Incomplete for Mir. I'm not sure but we may still be lacking sufficient client API in Mir to specify what buttons to show (and to grey out)

Changed in mir:
status: New → Incomplete
Changed in unity8 (Ubuntu):
assignee: nobody → Lukáš Tinkl (lukas-kde)
Changed in unity8 (Ubuntu):
status: Confirmed → In Progress
tags: added: unity8-desktop
Changed in ubuntu-ux:
status: Invalid → New
kevin gunn (kgunn72)
Changed in canonical-devices-system-image:
milestone: none → u8c-1
assignee: nobody → Michał Sawicz (saviq)
Revision history for this message
Lukáš Tinkl (lukas-kde) wrote :

Qt apps are not required to have any Close button as part of the dialog UI, even the standard builtin ones like Print Preview (http://doc.qt.io/qt-5/qprintpreviewdialog.html).

We need a way to close such a window (dialog), without resorting to fixing hundreds of apps themselves.

Revision history for this message
Michał Sawicz (saviq) wrote :

We might need to change what we map a surface like that to. Maybe in Mir case it needs to be a normal window, not a dialog?

Michał Sawicz (saviq)
Changed in canonical-devices-system-image:
milestone: u8c-1 → u8c-2
Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

From my perspective it is incorrect to say 'impossible to close "Print Preview" window' I can run kate in miral-shell, open the print-preview and close it (Alt-F4) fine.

Now I do agree it seems daft that there's neither a "Close" button (or equivalent) on the window, nor an "X" in the titlebar, but isn't that down to a design issue?

Changed in canonical-devices-system-image:
status: New → In Progress
Revision history for this message
Lukáš Tinkl (lukas-kde) wrote :

Just realized we have the same issue with the maximize button. If the window/dialog is resizeable (and it is in this specific case), we should display that in the decoration too.

Revision history for this message
Michał Sawicz (saviq) wrote :

Sure, when this window stops being a dialog, it will get it.

Revision history for this message
Lukáš Tinkl (lukas-kde) wrote :

No, I really mean every dialog (e.g. Open File dialog), those that are resizable should get a maximize button imho, just like under u7

Revision history for this message
Michał Sawicz (saviq) wrote :

Assuming that maximizing it will not exceed max size, yes.

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Window types generally have nothing to do with which window buttons should be visible. Window type dictates if there is a titlebar, and if so then...

Close button: Must always be visible because apps exist which have no other way of closing their windows.

Maximize button: Should only be visible for a window whose resize constraints allow it to be maximized.

Minimize button: Almost always visible, unless the window is modal to its parent.

Revision history for this message
Michał Sawicz (saviq) wrote :

That's not what the Mir Surfaces document dictates.

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

I know, but we've been confused by thinking that's a Mir/Unity8 design document. The parts about good dialog design really describe how to build a good toolkit (qtubuntu), and how to build a good shell assuming that's the only toolkit you need to support.

Legacy apps exist that require the shell must provide a close button on the dialog. There's no getting around it. Users are just going to keep reporting this bug until we fix it.

summary: - [unity8] kate, impossible to close "Print Preview" window
+ [unity8] It's impossible to close some dialogs (missing 'X' window
+ button)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 8.15+17.04.20170404.7-0ubuntu2

---------------
unity8 (8.15+17.04.20170404.7-0ubuntu2) zesty; urgency=medium

  [ Michael Zanetti ]
  * Make the default store uri point to gnome software center

 -- Timo Jyrinki <email address hidden> Thu, 06 Apr 2017 10:49:29 +0300

Changed in unity8 (Ubuntu):
status: In Progress → Fix Released
Michał Sawicz (saviq)
Changed in canonical-devices-system-image:
status: In Progress → Fix Released
Changed in qtubuntu (Ubuntu):
status: Confirmed → Invalid
Changed in qtmir (Ubuntu):
status: Confirmed → Invalid
Changed in mir:
status: Incomplete → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.