Merge lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/realLayoutDirectionChange into lp:ubuntu-ui-toolkit/staging

[Cris Dywan]

Change layoutDirection from gallery (via UCApplication)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

FAILED: Continuous integration, rev:2180
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/1471/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/7787/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/1471/rebuild

review: Needs Fixing (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

FAILED: Continuous integration, rev:2180
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/1686/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/7788/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/1686/rebuild

review: Needs Fixing (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

FAILED: Continuous integration, rev:2180
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci/912/
Executed test runs:
    FAILURE: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/1686/console
    FAILURE: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/1471/console
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/7789/console
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/7788
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/7787

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci/912/rebuild

review: Needs Fixing (continuous-integration)

[lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/realLayoutDirectionChange]

2181. By Cris Dywan on 2017-02-27

Update components.api

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

PASSED: Continuous integration, rev:2181
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/1472/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/7790/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/1472/rebuild

review: Approve (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

PASSED: Continuous integration, rev:2181
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/1687/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/7791/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/1687/rebuild

review: Approve (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

PASSED: Continuous integration, rev:2181
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci/913/
Executed test runs:
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/1687
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/1472
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/7792/console
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/7791
    SUCCESS: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/7790

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci/913/rebuild

review: Approve (continuous-integration)

[Adnane Belmadiaf (daker)]

LGTM, i still see some issues in RTL mode with APL but they are not related to this MR.

review: Approve

[Zsombor Egri (zsombi)]

All fine, thanks for having it!

review: Approve

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

PASSED: Continuous integration, rev:2181
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/1485/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/7823/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/1485/rebuild

review: Approve (continuous-integration)

[ubuntu-sdk-build-bot (ubuntu-sdk-build-bot)]

PASSED: Continuous integration, rev:2181
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/1700/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-update-mp/7824/console

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-stable/1700/rebuild

review: Approve (continuous-integration)

=== modified file 'components.api'
--- components.api	2017-02-21 14:57:05 +0000
+++ components.api	2017-02-27 15:48:33 +0000
@@ -1635,6 +1635,7 @@
 Ubuntu.Components.UCApplication 1.0 0.1: QtObject
     property string applicationName
     property QtObject inputMethod
+    property Qt.LayoutDirection layoutDirection
 Ubuntu.Components.UCFontUtils 1.0 0.1: QtObject
     function double sizeToPixels(string size)
     function double modularScale(string size)

=== modified file 'examples/ubuntu-ui-toolkit-gallery/MainPage.qml'
--- examples/ubuntu-ui-toolkit-gallery/MainPage.qml	2016-06-17 13:42:23 +0000
+++ examples/ubuntu-ui-toolkit-gallery/MainPage.qml	2017-02-27 15:48:33 +0000
@@ -30,14 +30,14 @@
             Action {
                 text: i18n.tr('Right to Left')
                 iconName: 'flash-on'
-                visible: !gallery.rtl
-                onTriggered: gallery.rtl = !gallery.rtl
+                visible: UbuntuApplication.layoutDirection == Qt.LeftToRight
+                onTriggered: UbuntuApplication.layoutDirection = Qt.RightToLeft
             },
             Action {
                 text: i18n.tr('Left to Right')
                 iconName: 'flash-off'
-                visible: gallery.rtl
-                onTriggered: gallery.rtl = !gallery.rtl
+                visible: UbuntuApplication.layoutDirection == Qt.RightToLeft
+                onTriggered: UbuntuApplication.layoutDirection = Qt.LeftToRight
             },
             Action {
                 text: i18n.tr('Use dark theme')

=== modified file 'examples/ubuntu-ui-toolkit-gallery/ubuntu-ui-toolkit-gallery.qml'
--- examples/ubuntu-ui-toolkit-gallery/ubuntu-ui-toolkit-gallery.qml	2016-01-18 12:50:44 +0000
+++ examples/ubuntu-ui-toolkit-gallery/ubuntu-ui-toolkit-gallery.qml	2017-02-27 15:48:33 +0000
@@ -29,9 +29,8 @@
     width: units.gu(120)
     height: units.gu(75)
 
-    LayoutMirroring.enabled: rtl
+    LayoutMirroring.enabled: Qt.application.layoutDirection == Qt.RightToLeft
     LayoutMirroring.childrenInherit: true
-    property bool rtl: Qt.application.layoutDirection == Qt.RightToLeft
 
     AdaptivePageLayout {
         id: layout

=== modified file 'src/UbuntuToolkit/ucapplication.cpp'
--- src/UbuntuToolkit/ucapplication.cpp	2017-02-09 11:00:07 +0000
+++ src/UbuntuToolkit/ucapplication.cpp	2017-02-27 15:48:33 +0000
@@ -42,6 +42,10 @@
     // Make sure we receive application name changes from C++ modules
     connect(QCoreApplication::instance(), &QCoreApplication::applicationNameChanged,
             this, &UCApplication::applicationNameChanged);
+    // Changes to the default layout direction (RTL, LTR)
+    QGuiApplication* application = qobject_cast<QGuiApplication*>(QCoreApplication::instance());
+    connect(application, &QGuiApplication::layoutDirectionChanged,
+            this, &UCApplication::layoutDirectionChanged);
 }
 
 UCApplication::~UCApplication()
@@ -91,4 +95,18 @@
     m_inputMethod = inputMethod;
 }
 
+/*!
+ * \internal
+ * The (default) layout direction. Can be overridden for testing,
+ * unlike Qt.application.layoutDirection.
+ */
+Qt::LayoutDirection UCApplication::layoutDirection() {
+    return QGuiApplication::layoutDirection();
+}
+
+void UCApplication::setLayoutDirection(Qt::LayoutDirection layoutDirection) {
+    return QGuiApplication::setLayoutDirection(layoutDirection);
+}
+
+
 UT_NAMESPACE_END

=== modified file 'src/UbuntuToolkit/ucapplication_p.h'
--- src/UbuntuToolkit/ucapplication_p.h	2016-09-09 17:49:07 +0000
+++ src/UbuntuToolkit/ucapplication_p.h	2017-02-27 15:48:33 +0000
@@ -33,6 +33,7 @@
     Q_OBJECT
     Q_PROPERTY(QString applicationName READ applicationName WRITE setApplicationName NOTIFY applicationNameChanged)
     Q_PROPERTY(QObject* inputMethod READ inputMethod WRITE setInputMethod NOTIFY inputMethodChanged)
+    Q_PROPERTY(Qt::LayoutDirection layoutDirection READ layoutDirection WRITE setLayoutDirection NOTIFY layoutDirectionChanged)
 
 private:
     Q_DISABLE_COPY(UCApplication)
@@ -53,11 +54,13 @@
     // getter
     QString applicationName();
     QObject* inputMethod();
+    Qt::LayoutDirection layoutDirection();
 
     // setter
     void setContext(QQmlContext* context);
     void setApplicationName(const QString& applicationName);
     void setInputMethod(QObject* inputMethod);
+    void setLayoutDirection(Qt::LayoutDirection layoutDirection);
 
 private:
     QQmlContext* m_context;
@@ -67,6 +70,7 @@
 Q_SIGNALS:
     void applicationNameChanged();
     void inputMethodChanged();
+    void layoutDirectionChanged();
 };
 
 UT_NAMESPACE_END