Coordinated migration to UITK 1.3

[Michał Sawicz]

The migration to 1.3 needs to happen in a coordinated way, so that shell and all apps we care of migrate within the same OTA window to reduce visual inconsistencies to a minimum.

This bug is a means to collect all the projects affected and keep tabs on what's happening.

NOTES:
- The toolkit has the following modules:
   Ubuntu.Components 1.3
   Ubuntu.Components.ListItems 1.3
   Ubuntu.Components.Pickers 1.3
   Ubuntu.Components.Popups 1.3
   Ubuntu.Components.Styles 1.3
   Ubuntu.Components.Themes 1.3
   Ubuntu.Components.Themes.Ambiance 1.3
   Ubuntu.Components.Themes.SuruDark 1.3

- The following modules were not altered recently:
   Ubuntu.Layouts 1.0
   Ubuntu.PerformanceMetrics 1.0

- Theme is now theme (lowercase)
- PageHeadStyle now has a "config" property

In manifest.json file use:
   "framework": "ubuntu-sdk-15.04.1-qml"

[Olivier Tilloy (osomon)]

The migration was done for webbrowser-app a while back, it was released with version 0.23+15.10.20150810-0ubuntu1.

[Launchpad Janitor (janitor)]

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

[Joey Chan (qqworini)]

How about "Ubuntu.Content 1.1" ? also change to 1.3 ?

[Zsombor Egri (zsombi)]

@Joey Ubuntu.Content is not part of the Ubuntu UI Toolkit project the version of that is not driven by the toolkit versioning.

The toolkit has the following modules:
Ubuntu.Components 1.3
Ubuntu.Components.ListItems 1.3
Ubuntu.Components.Pickers 1.3
Ubuntu.Components.Popups 1.3
Ubuntu.Components.Styles 1.3
Ubuntu.Components.Themes 1.3
Ubuntu.Components.Themes.Ambiance 1.3
Ubuntu.Components.Themes.SuruDark 1.3

The following modules were not altered recently:
Ubuntu.Layouts 1.0
Ubuntu.PerformanceMetrics 1.0

[Stefano Verzegnassi (verzegnassi-stefano)]

Can we then target all the changes included in UITK 1.3.1676+15.04.20151008-0ubuntu1, landed in OTA-7 (including new Label property, ListItemLayout, etc.)?

[Andrea Bernabei (faenil)]

Is the plan just to update the imports to 1.3? Or should we adapt the apps to use the new features coming from 1.3?

So far all MRs implemented the former, but I haven't seen any using the new 1.3 specific components, hence the question :)

[Launchpad Janitor (janitor)]

This bug was fixed in the package camera-app - 3.0.0+16.04.20151026-0ubuntu1

---------------
camera-app (3.0.0+16.04.20151026-0ubuntu1) xenial; urgency=medium

  [ CI Train Bot ]
  * New rebuild forced.
  * Resync trunk.

  [ Florian Boucault ]
  * Migrated to using Ubuntu UI Toolkit version 1.3 only. Migrated to
    using QtQuick 2.4 and QtQuick.Window 2.2 only. (LP: #1508363)

  [ James Henstridge ]
  * After shooting a video, request a thumbnail so that it will be
    available in the cache when needed (e.g. by the Gallery or Dash).

 -- Florian Boucault <email address hidden> Mon, 26 Oct 2015 13:19:17 +0000

[Michał Sawicz (saviq)]

W dniu 24.10.2015 o 11:59, Andrea Bernabei pisze:
> Is the plan just to update the imports to 1.3? Or should we adapt the
> apps to use the new features coming from 1.3?
>
> So far all MRs implemented the former, but I haven't seen any using the
> new 1.3 specific components, hence the question :)

Remember *click* apps can only rely on a UITK from a few weeks ago to be
available, as that's when the framework was defined.

As for debs, do what you need, but care to depend on the correct version
of UITK, or higher.

[Victor Thompson (vthompson)]

The Music and Weather app teams have been trying to determine how we can read the MainView or Page's Header height with the new API. With the updated imports to UITK 1.3 we are unable to do things like the following:

edgeLoader.item.flickable.contentY = -page.header.height

A large number of apps (clock, weather, filemanager, and probably others) use a PageWithBottomEdge component that make this or similar assignments. [1][2][3][4]

How can we access the height of the header in a reasonable fashion that also includes any Header Sections that may also be defined?

1- http://bazaar.launchpad.net/~ubuntu-weather-dev/ubuntu-weather-app/reboot/view/head:/app/components/PageWithBottomEdge.qml#L111
2 - http://bazaar.launchpad.net/~ubuntu-clock-dev/ubuntu-clock-app/trunk/view/head:/app/upstreamcomponents/PageWithBottomEdge.qml#L107
3 - http://bazaar.launchpad.net/~ubuntu-filemanager-dev/ubuntu-filemanager-app/trunk/view/head:/src/app/qml/upstream/PageWithBottomEdge.qml#L108
4 - http://bazaar.launchpad.net/~music-app-dev/music-app/trunk/view/head:/app/ui/NowPlaying.qml#L143

[Andrea Bernabei (faenil)]

@Saviq: my question still stands :)

[Launchpad Janitor (janitor)]

This bug was fixed in the package ubuntu-system-settings - 0.3+16.04.20151023.1-0ubuntu1

---------------
ubuntu-system-settings (0.3+16.04.20151023.1-0ubuntu1) xenial; urgency=medium

  [ CI Train Bot ]
  * New rebuild forced.

  [ Ken VanDine ]
  * Bump Ubuntu.Components imports to 1.3 and fixed UbuntuShape
    deprecations (LP: #1508363)

  [ jonas-drange ]
  * re-add silentMode property to repair the silent mode switch

 -- <email address hidden> (Jonas G. Drange) Fri, 23 Oct 2015 14:57:29 +0000

[Zsombor Egri (zsombi)]

Victor and others, in order to get the header height you need to set the Page.header first. Page.header is null by default, so no wonder you cannot get the height. You can do something like

Page {
    header: PageHeader {
    }

}

Then you will have the header set and you can use that to get the height. The component is present in UITK trunk, however I do not see its documentation under d.u.c...

[Victor Thompson (vthompson)]

Zsombor,

Functionally, there already is a header, so why should a developer need to define one explicitly? Won't explicitly defining one override the already visible header?

What I'm asking for is simply a means of ascertaining the current header's height. I believe the inability to do so, currently, is a regression.

[Zsombor Egri (zsombi)]

Victor,

You are right, there is one header per application. Which means that all pages in an application use that header. However that would be broken when you'd use the AdaptivePageLayout, which has as many headers as columns. Therefore we are preparing to move the header in the page, so each page will get a header of its own. Yet we have both solutions in place, thats' why you still see a header even if you don't set one on the Page.

So, in the future of 1.3, you will have to set the header of the page yourself. Which means there will be as many headers as many pages require one. The one from MainView will disappear once we finish all the work we have for the header and its subcomponents.

When you say "With the updated imports to UITK 1.3 we are unable to do things like the following: edgeLoader.item.flickable.contentY = -page.header.height" - I am sorry to say that the Page.header property was never documented, and used for private purposes. We do not promise any compatibility for private or undocumented properties, especially when the property comes from a super type, which is also internal. Whilst it is a regression, switching to 1.3 would anyways require you to go with the way I described before - specifying the header for your page within the page.

[Jenkins Bot (ubuntu-core-apps-jenkins-bot)]

Fix committed into lp:ubuntu-rssreader-app at revision None, scheduled for release in ubuntu-rssreader-app, milestone Unknown

[Launchpad Janitor (janitor)]

This bug was fixed in the package ubuntu-system-settings-online-accounts - 0.7+16.04.20151030.1-0ubuntu1

---------------
ubuntu-system-settings-online-accounts (0.7+16.04.20151030.1-0ubuntu1) xenial; urgency=medium

  [ Alberto Mardegan ]
  * Use version 1.3 of Ubuntu.Components (LP: #1508363, #1511055)

  [ CI Train Bot ]
  * Resync trunk.

 -- David Barth <email address hidden> Fri, 30 Oct 2015 15:41:24 +0000

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

Fix committed into lp:music-app at revision 932, scheduled for release in music-app, milestone 3.0

[Launchpad Janitor (janitor)]

This bug was fixed in the package unity8 - 8.11+16.04.20151104-0ubuntu1

---------------
unity8 (8.11+16.04.20151104-0ubuntu1) xenial; urgency=medium

  [ Albert Astals Cid ]
  * Add an Item that proxies for old and new audio roles (LP: #1493851)
  * CroppedImageMinimumSourceSize: Fix 'Binding loop detected for
    property "imageAspectRatio"'
  * Make the scope settings list scroll to text visible area (LP:
    #1499084)
  * Move to importing QtQuick 2.4 & friends
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)

  [ Andrea Cimitan ]
  * Add sharing widget to zoomable image and video playback
  * Make zoomable and video playback widgets edge to edge
  * Updated all 1.3 UbuntuShape to use new APIs where possible
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)

  [ CI Train Bot ]
  * Resync trunk.

  [ Christopher Lee ]
  * Quick spelling fix in process control output.

  [ Daniel d'Andrada ]
  * Enable support for overriding application orientations based on
    device type (LP: #1478637)
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)

  [ Josh Arenson ]
  * Enable support for overriding application orientations based on
    device type (LP: #1478637)

  [ Michael Terry ]
  * Avoid showing the shutdown dialog when turning on the screen if your
    device is under heavy load. (LP: #1508563)
  * Handle lifecycle policy exceptions ourselves, instead of letting
    qtmir do it for us and allow non-Touch apps to opt-out of the Touch
    lifecycle.

  [ Michael Zanetti ]
  * Fixes for the panel buttons (LP: #1510360, #1504269, #1431566,
    #1443319)
  * Use SDK 1.3 across all files (LP: #1503498, #1508363, #1449628)

  [ Michał Sawicz ]
  * Add missing copyright to Cursor.qml
  * Enable support for overriding application orientations based on
    device type (LP: #1478637)
  * Support server->client visibility change to stop rendering
    (lp:#1475678) (LP: #1475678)

  [ Nick Dedekind ]
  * Support server->client visibility change to stop rendering
    (lp:#1475678) (LP: #1475678)

 -- Michał Sawicz <email address hidden> Wed, 04 Nov 2015 14:58:41 +0000

[Launchpad Janitor (janitor)]

This bug was fixed in the package gallery-app - 0.0.67+16.04.20151106-0ubuntu1

---------------
gallery-app (0.0.67+16.04.20151106-0ubuntu1) xenial; urgency=medium

  [ Arthur Mello ]
  * Update QML code to work with Ubuntu.Components 1.3 (LP: #1508363)
  * Use a Loader for OrganicMediaLst delegates to avoid crashes when
    loading multiple files (LP: #1475210, #1470109)

  [ CI Train Bot ]
  * New rebuild forced.

 -- Bill Filler <email address hidden> Fri, 06 Nov 2015 14:05:26 +0000

[Pat McGowan (pat-mcgowan)]

Remaining apps will get out over the next couple weeks

[Jenkins Bot (ubuntu-core-apps-jenkins-bot)]

Fix committed into lp:ubuntu-calculator-app at revision 241, scheduled for release in ubuntu-calculator-app, milestone Unknown

[Jenkins Bot (ubuntu-core-apps-jenkins-bot)]

Fix committed into lp:ubuntu-weather-app at revision 180, scheduled for release in ubuntu-weather-app, milestone 2.0

[Jenkins Bot (ubuntu-core-apps-jenkins-bot)]

Fix committed into lp:ubuntu-docviewer-app at revision 210, scheduled for release in ubuntu-docviewer-app, milestone Unknown

[Jenkins Bot (ubuntu-core-apps-jenkins-bot)]

Fix committed into lp:ubuntu-clock-app at revision 421, scheduled for release in ubuntu-clock-app, milestone 3.7

[Pat McGowan (pat-mcgowan)]

Closing even with tasks yet to land, should update soon