Having discussed this a few times I've reached the following conclusion:

"automatic" is not a MirSurfaceState - if it were a MirSurfaceState,
then it would be correct to return it from get_state() and for it to be
implemented as "looking the same as fullscreen or restored".

By comparison, "restored" can look like "maximized" on the phone or as
windows on desktop and is what get_state() returns.

Instead it is additional metadata about the surface requesting a
specific treatment of the MirSurfaceState.

So, I think the right approach for Mir is to have an additional surface
attribute:

enum MirSurfaceStatePolicy
{
    mir_surface_state_policy_default,
    mir_surface_state_policy_qt_automatic,
    ...
};

With corresponding setter and getter functions on the MirSurfaceSpec*

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to camera-app in Ubuntu.
https://bugs.launchpad.net/bugs/1535397

Title:
  [enhancement] Implement support for QWindow::visibility set to
  Automatic

Status in The Avila project:
  New
Status in Canonical System Image:
  Triaged
Status in canonical-pocket-desktop:
  New
Status in Mir:
  New
Status in QtMir:
  New
Status in camera-app package in Ubuntu:
  New
Status in gallery-app package in Ubuntu:
  New
Status in mediaplayer-app package in Ubuntu:
  New
Status in mir package in Ubuntu:
  New
Status in qtubuntu package in Ubuntu:
  Triaged
Status in unity8 package in Ubuntu:
  Triaged

Bug description:
  We need to support windows setting their visibility to "Automatic",
  when in Staged mode the window should be fullscreen, whereas in
  windowed, it should be a normal window.

  Qt API: http://doc.qt.io/qt-5/qwindow.html#Visibility-enum

To manage notifications about this bug go to:
https://bugs.launchpad.net/avila/+bug/1535397/+subscriptions

-- 
Mailing list: https://launchpad.net/~touch-packages
Post to     : touch-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~touch-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to