9b88f0b · unity8 · git.iko.soy

CMakeLists.txt+1−1

compare with xenial

unifiedsplit

@@ -57,7 +57,7 @@ find_package(Qt5DBus 5.4 REQUIRED)

57find_package(Qt5Concurrent 5.4 REQUIRED)

58find_package(Qt5Sql 5.4 REQUIRED)

59

60−pkg_check_modules(APPLICATION_API REQUIRED unity-shell-application=18)

60+pkg_check_modules(APPLICATION_API REQUIRED unity-shell-application=19)

61pkg_check_modules(GEONAMES REQUIRED geonames>=0.2)

62pkg_check_modules(GIO REQUIRED gio-2.0>=2.32)

63pkg_check_modules(GLIB REQUIRED glib-2.0>=2.32)

debian/changelog+6

@@ -1,3 +1,9 @@

1+unity8 (8.13) UNRELEASED; urgency=medium

2+

3+ * Make use of MirSurface::inputBounds and drop UbuntuKeyboardInfo hack

4+

5+ -- Daniel d'Andrada <daniel.dandrada@canonical.com> Tue, 12 Jul 2016 08:49:26 -0300

6+

1unity8 (8.12+16.10.20160711-0ubuntu1) yakkety; urgency=medium

7unity8 (8.12+16.10.20160711-0ubuntu1) yakkety; urgency=medium

2

8

3 [ Josh Arenson ]

9 [ Josh Arenson ]

debian/control+4−3

@@ -34,7 +34,7 @@ Build-Depends: cmake,

34 libubuntugestures5-dev,

35 libubuntugestures5-private-dev,

36 libudev-dev,

37− libunity-api-dev (>= 7.115),

37+ libunity-api-dev (>= 7.116),

38 libusermetricsoutput1-dev,

39# Need those X11 libs touch emulation from mouse events in manual QML tests on a X11 desktop

40 libx11-dev[!armhf],

@@ -136,6 +136,7 @@ Recommends: indicator-session,

136Breaks: indicator-network (<< 0.5.1+14.10.20141014),

137 ubuntu-touch-session (<< 0.107),

138 unity8-greeter (<< 7.89),

139+ ubuntu-keyboard (<< 0.100),

139Replaces: ubuntu-touch-session (<< 0.82~),

140Replaces: ubuntu-touch-session (<< 0.82~),

140 unity8-greeter (<< 7.89),

141 unity8-greeter (<< 7.89),

141Conflicts: ubuntu-system-settings-wizard,

142Conflicts: ubuntu-system-settings-wizard,

@@ -150,7 +151,7 @@ Depends: qml-module-qtquick-layouts,

150 qtdeclarative5-ubuntu-settings-components (>= 0.7),

151 qtdeclarative5-ubuntu-settings-components (>= 0.7),

151 qtdeclarative5-unity-notifications-plugin (>= 0.1.2) | unity-notifications-impl,

152 qtdeclarative5-unity-notifications-plugin (>= 0.1.2) | unity-notifications-impl,

152 ubuntu-thumbnailer-impl-0,

153 ubuntu-thumbnailer-impl-0,

153− unity-application-impl-17,

154+ unity-application-impl-19,

154 unity-notifications-impl-3,

155 unity-notifications-impl-3,

155 unity-plugin-scopes | unity-scopes-impl,

156 unity-plugin-scopes | unity-scopes-impl,

156 unity-scopes-impl-12,

157 unity-scopes-impl-12,

@@ -196,7 +197,7 @@ Pre-Depends: ${misc:Pre-Depends},

196Depends: ${misc:Depends},

197Depends: ${misc:Depends},

197 ${shlibs:Depends},

198 ${shlibs:Depends},

198Provides: unity-application-impl,

199Provides: unity-application-impl,

199− unity-application-impl-16,

200+ unity-application-impl-19,

200Replaces: unity8-autopilot (<< 8.02+15.04.20150422-0ubuntu1)

201Replaces: unity8-autopilot (<< 8.02+15.04.20150422-0ubuntu1)

201Description: Fake environment for running Unity 8 shell

202Description: Fake environment for running Unity 8 shell

202 Provides fake implementations of some QML modules used by Unity 8 shell

203 Provides fake implementations of some QML modules used by Unity 8 shell

qml/Components/InputMethod.qml+8−44

@@ -1,5 +1,5 @@

1/*

3 *

4 * This program is free software; you can redistribute it and/or modify

5 * it under the terms of the GNU General Public License as published by

@@ -16,13 +16,12 @@

16

17import QtQuick 2.4

18import Unity.Application 0.1

19−import Ubuntu.Components 1.3

20import Ubuntu.Gestures 0.1

19import Ubuntu.Gestures 0.1

21

20

22Item {

21Item {

23 id: root

22 id: root

24

23

25− property int transitionDuration: UbuntuAnimation.FastDuration

24+ readonly property rect visibleRect: surfaceItem.surface ? surfaceItem.surface.inputBounds : Qt.rect(0, 0, 0, 0)

26

25

27 MirSurfaceItem {

26 MirSurfaceItem {

28 id: surfaceItem

27 id: surfaceItem

@@ -42,51 +41,16 @@ Item {

42 }

41 }

43

42

44 TouchGate {

43 TouchGate {

45− x: UbuntuKeyboardInfo.x

44+ x: root.visibleRect.x

46− y: UbuntuKeyboardInfo.y

45+ y: root.visibleRect.y

47− width: UbuntuKeyboardInfo.width

46+ width: root.visibleRect.width

48− height: UbuntuKeyboardInfo.height

47+ height: root.visibleRect.height

49

48

50 targetItem: surfaceItem

49 targetItem: surfaceItem

51 }

50 }

52

51

53− state: {

54− if (surfaceItem.surface &&

52+ visible: surfaceItem.surface &&

55 surfaceItem.surfaceState != Mir.HiddenState &&

53 surfaceItem.surfaceState != Mir.HiddenState &&

56 surfaceItem.surfaceState != Mir.MinimizedState &&

54 surfaceItem.surfaceState != Mir.MinimizedState &&

57− root.enabled) {

55+ root.enabled

58− return "shown";

59− } else {

60− return "hidden";

61− }

62− }

63−

64− states: [

65− State {

66− name: "shown"

67− PropertyChanges {

68− target: root

69− visible: true

70− y: 0

71− }

72− },

73− State {

74− name: "hidden"

75− PropertyChanges {

76− target: root

77− visible: false

78− // half-way down because the vkb occupies only the lower half of the surface

79− // TODO: consider keyboard rotation

80− y: root.parent.height / 2.0

81− }

82− }

83− ]

84−

85− transitions: [

86− Transition {

87− from: "*"; to: "*"

88− PropertyAction { property: "visible"; value: true }

89− UbuntuNumberAnimation { property: "y"; duration: transitionDuration }

90− }

91− ]

92}

56}

qml/Shell.qml+1−1

@@ -663,7 +663,7 @@ StyledItem {

663 paused: callManager.hasCalls || !greeter || greeter.shown ||

664 wizard.active

665 delayed: dialogs.hasActiveDialog || notifications.hasNotification ||

666− inputMethod.state === "shown" ||

666+ inputMethod.visible ||

667 (launcher.shown && !launcher.lockedVisible) ||

668 panel.indicators.shown || stage.dragProgress > 0

669 usageScenario: shell.usageScenario

qml/Stages/DesktopStage.qml+5−1

@@ -156,6 +156,10 @@ AbstractStage {

156 }

157 }

158 }

159+

160+ readonly property real virtualKeyboardHeight: SurfaceManager.inputMethodSurface

161+ ? SurfaceManager.inputMethodSurface.inputBounds.height

162+ : 0

159 }

163 }

160

164

161 Connections {

165 Connections {

@@ -273,7 +277,7 @@ AbstractStage {

273 property: "y"

277 property: "y"

274 value: appDelegate.requestedY -

278 value: appDelegate.requestedY -

275 Math.min(appDelegate.requestedY - PanelState.panelHeight,

279 Math.min(appDelegate.requestedY - PanelState.panelHeight,

276− Math.max(0, UbuntuKeyboardInfo.height - (appContainer.height - (appDelegate.requestedY + appDelegate.height))))

280+ Math.max(0, priv.virtualKeyboardHeight - (appContainer.height - (appDelegate.requestedY + appDelegate.height))))

277 when: root.oskEnabled && appDelegate.focus && appDelegate.state == "normal"

281 when: root.oskEnabled && appDelegate.focus && appDelegate.state == "normal"

278 && SurfaceManager.inputMethodSurface

282 && SurfaceManager.inputMethodSurface

279 && SurfaceManager.inputMethodSurface.state != Mir.HiddenState

283 && SurfaceManager.inputMethodSurface.state != Mir.HiddenState

tests/mocks/Unity/Application/CMakeLists.txt−1

@@ -7,7 +7,6 @@ set(FakeUnityApplicationQml_SOURCES

7 MirSurfaceListModel.cpp

8 ObjectListModel.h

9 SurfaceManager.cpp

10− UbuntuKeyboardInfo.cpp

11 VirtualKeyboard.cpp

10 VirtualKeyboard.cpp

12 ${APPLICATION_API_INCLUDEDIR}/unity/shell/application/ApplicationInfoInterface.h

11 ${APPLICATION_API_INCLUDEDIR}/unity/shell/application/ApplicationInfoInterface.h

13 ${APPLICATION_API_INCLUDEDIR}/unity/shell/application/ApplicationManagerInterface.h

12 ${APPLICATION_API_INCLUDEDIR}/unity/shell/application/ApplicationManagerInterface.h

tests/mocks/Unity/Application/MirSurface.cpp+21

@@ -87,6 +87,8 @@ MirSurface::MirSurface(const QString& name,

87 m_zombieTimer.setInterval(100);

88 m_zombieTimer.setSingleShot(true);

89 connect(&m_zombieTimer, &QTimer::timeout, this, [this](){ this->setLive(false); });

90+

91+ updateInputBoundsAfterResize();

90}

92}

91

93

92MirSurface::~MirSurface()

94MirSurface::~MirSurface()

@@ -326,6 +328,13 @@ void MirSurface::doResize(int width, int height)

326 if (changed) {

328 if (changed) {

327 Q_EMIT sizeChanged(QSize(width, height));

329 Q_EMIT sizeChanged(QSize(width, height));

328 }

330 }

331+

332+ updateInputBoundsAfterResize();

333+}

334+

335+void MirSurface::updateInputBoundsAfterResize()

336+{

337+ setInputBounds(QRect(0, 0, m_width, m_height));

329}

338}

330

339

331bool MirSurface::isSlowToResize() const

340bool MirSurface::isSlowToResize() const

@@ -439,6 +448,18 @@ bool MirSurface::focused() const

439 return controller ? controller->focusedSurface() == this : false;

448 return controller ? controller->focusedSurface() == this : false;

440}

449}

441

450

451+QRect MirSurface::inputBounds() const

452+{

453+ return m_inputBounds;

454+}

455+

456+void MirSurface::setInputBounds(const QRect &boundsRect)

457+{

458+ if (boundsRect != m_inputBounds) {

459+ m_inputBounds = boundsRect;

460+ Q_EMIT inputBoundsChanged(m_inputBounds);

461+ }

462+}

442#if MIRSURFACE_DEBUG

463#if MIRSURFACE_DEBUG

443#undef DEBUG_MSG

464#undef DEBUG_MSG

444#define DEBUG_MSG(params) qDebug().nospace() << "MirFocusController::" << __func__ << " " << params

465#define DEBUG_MSG(params) qDebug().nospace() << "MirFocusController::" << __func__ << " " << params

tests/mocks/Unity/Application/MirSurface.h+8

@@ -102,6 +102,7 @@ public:

102 Mir::ShellChrome shellChrome() const override;

103

104 bool focused() const override;

105+ QRect inputBounds() const override;

105

106

106 Q_INVOKABLE void requestFocus() override;

107 Q_INVOKABLE void requestFocus() override;

107

108

@@ -128,6 +129,8 @@ public:

128 Q_INVOKABLE void setWidthIncrement(int);

129 Q_INVOKABLE void setWidthIncrement(int);

129 Q_INVOKABLE void setHeightIncrement(int);

130 Q_INVOKABLE void setHeightIncrement(int);

130

131

132+ Q_INVOKABLE virtual void setInputBounds(const QRect &boundsRect);

133+

131 /////

134 /////

132 // internal mock stuff

135 // internal mock stuff

133

136

@@ -160,6 +163,9 @@ Q_SIGNALS:

160 void raiseRequested();

163 void raiseRequested();

161 void closeRequested();

164 void closeRequested();

162

165

166+protected:

167+ virtual void updateInputBoundsAfterResize();

168+

163private Q_SLOTS:

169private Q_SLOTS:

164 void applyDelayedResize();

170 void applyDelayedResize();

165

171

@@ -202,6 +208,8 @@ private:

202 QHash<qintptr, View> m_views;

208 QHash<qintptr, View> m_views;

203

209

204 QTimer m_zombieTimer;

210 QTimer m_zombieTimer;

211+

212+ QRect m_inputBounds;

205};

213};

206

214

207#endif // MOCK_MIR_SURFACE_H

215#endif // MOCK_MIR_SURFACE_H

tests/mocks/Unity/Application/UbuntuKeyboardInfo.cppdeleted−58

@@ -1,58 +0,0 @@

1−/*

3− *

4− * This program is free software: you can redistribute it and/or modify it under

5− * the terms of the GNU Lesser General Public License version 3, as published by

6− * the Free Software Foundation.

7− *

8− * This program is distributed in the hope that it will be useful, but WITHOUT

9− * ANY WARRANTY; without even the implied warranties of MERCHANTABILITY,

10− * SATISFACTORY QUALITY, or FITNESS FOR A PARTICULAR PURPOSE. See the GNU

11− * Lesser General Public License for more details.

12− *

13− * You should have received a copy of the GNU Lesser General Public License

14− * along with this program. If not, see <http://www.gnu.org/licenses/>.

15− */

16−

17−#include "UbuntuKeyboardInfo.h"

18−

19−UbuntuKeyboardInfo::UbuntuKeyboardInfo(QObject *parent)

20− : QObject(parent),

21− m_x(0),

22− m_y(0),

23− m_width(200),

24− m_height(0)

25−{

26−}

27−

28−void UbuntuKeyboardInfo::setX(qreal value)

29−{

30− if (value != m_x) {

31− m_x = value;

32− Q_EMIT xChanged(m_x);

33− }

34−}

35−

36−void UbuntuKeyboardInfo::setY(qreal value)

37−{

38− if (value != m_y) {

39− m_y = value;

40− Q_EMIT yChanged(m_y);

41− }

42−}

43−

44−void UbuntuKeyboardInfo::setWidth(qreal value)

45−{

46− if (value != m_width) {

47− m_width = value;

48− Q_EMIT widthChanged(m_width);

49− }

50−}

51−

52−void UbuntuKeyboardInfo::setHeight(qreal value)

53−{

54− if (value != m_height) {

55− m_height = value;

56− Q_EMIT heightChanged(m_height);

57− }

58−}

tests/mocks/Unity/Application/UbuntuKeyboardInfo.hdeleted−57

@@ -1,57 +0,0 @@

1−/*

3− *

4− * This program is free software: you can redistribute it and/or modify it under

5− * the terms of the GNU Lesser General Public License version 3, as published by

6− * the Free Software Foundation.

7− *

8− * This program is distributed in the hope that it will be useful, but WITHOUT

9− * ANY WARRANTY; without even the implied warranties of MERCHANTABILITY,

10− * SATISFACTORY QUALITY, or FITNESS FOR A PARTICULAR PURPOSE. See the GNU

11− * Lesser General Public License for more details.

12− *

13− * You should have received a copy of the GNU Lesser General Public License

14− * along with this program. If not, see <http://www.gnu.org/licenses/>.

15− */

16−

17−#ifndef UBUNTU_KEYBOARD_INFO_H

18−#define UBUNTU_KEYBOARD_INFO_H

19−

20−#include <QLocalSocket>

21−#include <QTimer>

22−

23−class UbuntuKeyboardInfo : public QObject {

24− Q_OBJECT

25− Q_PROPERTY(qreal x READ x WRITE setX NOTIFY xChanged)

26− Q_PROPERTY(qreal y READ y WRITE setY NOTIFY yChanged)

27− Q_PROPERTY(qreal width READ width WRITE setWidth NOTIFY widthChanged)

28− Q_PROPERTY(qreal height READ height WRITE setHeight NOTIFY heightChanged)

29−public:

30− UbuntuKeyboardInfo(QObject *parent = 0);

31− virtual ~UbuntuKeyboardInfo() {}

32− qreal x() const { return m_x; }

33− qreal y() const { return m_y; }

34− qreal width() const { return m_width; }

35− qreal height() const { return m_height; }

36−

37− // Just in the fake implementation

38− void setX(qreal value);

39− void setY(qreal value);

40− void setWidth(qreal value);

41− void setHeight(qreal value);

42−

43−Q_SIGNALS:

44− void xChanged(qreal x);

45− void yChanged(qreal y);

46− void widthChanged(qreal width);

47− void heightChanged(qreal height);

48−

49−private:

50− QLocalSocket m_socket;

51− qint32 m_x;

52− qint32 m_y;

53− qint32 m_width;

54− qint32 m_height;

55−};

56−

57−#endif // UBUNTU_KEYBOARD_INFO_H

tests/mocks/Unity/Application/VirtualKeyboard.cpp+20−1

@@ -1,5 +1,5 @@

1/*

3 *

4 * This program is free software; you can redistribute it and/or modify

5 * it under the terms of the GNU General Public License as published by

@@ -34,3 +34,22 @@ VirtualKeyboard::VirtualKeyboard()

34VirtualKeyboard::~VirtualKeyboard()

35{

36}

37+

38+void VirtualKeyboard::updateInputBoundsAfterResize()

39+{

40+ int width = this->width();

41+

42+ int height;

43+ if (this->width() > this->height()) {

44+ // landscape

45+ height = this->height() * 0.4;

46+ } else {

47+ // portrait

48+ height = this->width() * 0.6;

49+ }

50+

51+ int x = 0;

52+ int y = this->height() - height;

53+

54+ setInputBounds(QRect(x, y, width, height));

55+}

tests/mocks/Unity/Application/VirtualKeyboard.h+4−1

@@ -1,5 +1,5 @@

1/*

3 *

4 * This program is free software; you can redistribute it and/or modify

5 * it under the terms of the GNU General Public License as published by

@@ -27,6 +27,9 @@ class VirtualKeyboard : public MirSurface

27public:

28 explicit VirtualKeyboard();

29 ~VirtualKeyboard();

30+

31+protected:

32+ void updateInputBoundsAfterResize() override;

30};

33};

31

34

32Q_DECLARE_METATYPE(VirtualKeyboard*)

35Q_DECLARE_METATYPE(VirtualKeyboard*)

tests/mocks/Unity/Application/plugin.cpp−8

@@ -19,7 +19,6 @@

19#include "ApplicationManager.h"

20#include "MirSurfaceItem.h"

21#include "SurfaceManager.h"

22−#include "UbuntuKeyboardInfo.h"

23

22

24// unity-api

23// unity-api

25#include <unity/shell/application/Mir.h>

24#include <unity/shell/application/Mir.h>

@@ -53,12 +52,6 @@ QObject* surfaceManagerSingleton(QQmlEngine*, QJSEngine*)

53 return SurfaceManager::instance();

52 return SurfaceManager::instance();

54}

53}

55

54

56−QObject* ubuntuKeyboardInfoSingleton(QQmlEngine*, QJSEngine*)

57−{

58− createUnityApplicationSharedSingletons();

59− return new UbuntuKeyboardInfo;

60−}

61−

62QObject* mirFocusControllerSingleton(QQmlEngine*, QJSEngine*)

55QObject* mirFocusControllerSingleton(QQmlEngine*, QJSEngine*)

63{

56{

64 createUnityApplicationSharedSingletons();

57 createUnityApplicationSharedSingletons();

@@ -86,7 +79,6 @@ void FakeUnityApplicationQmlPlugin::registerTypes(const char *uri)

86

79

87 qmlRegisterSingletonType<ApplicationManager>(uri, 0, 1, "ApplicationManager", applicationManagerSingleton);

80 qmlRegisterSingletonType<ApplicationManager>(uri, 0, 1, "ApplicationManager", applicationManagerSingleton);

88 qmlRegisterSingletonType<SurfaceManager>(uri, 0, 1, "SurfaceManager", surfaceManagerSingleton);

81 qmlRegisterSingletonType<SurfaceManager>(uri, 0, 1, "SurfaceManager", surfaceManagerSingleton);

89− qmlRegisterSingletonType<UbuntuKeyboardInfo>(uri, 0, 1, "UbuntuKeyboardInfo", ubuntuKeyboardInfoSingleton);

90

82

91 qmlRegisterUncreatableType<Mir>(uri, 0, 1, "Mir", "Mir provides enum values, it can't be instantiated");

83 qmlRegisterUncreatableType<Mir>(uri, 0, 1, "Mir", "Mir provides enum values, it can't be instantiated");

92}

84}

tests/mocks/Unity/Application/resources/VirtualKeyboard.qml+1−6

@@ -1,5 +1,5 @@

1/*

3 *

4 * This program is free software; you can redistribute it and/or modify

5 * it under the terms of the GNU Lesser General Public License as published by

@@ -38,9 +38,4 @@ Item {

38 anchors.fill: parent

39 }

40 }

41−

42− Binding { target: UbuntuKeyboardInfo; property: "x"; value: screenshotImage.x}

43− Binding { target: UbuntuKeyboardInfo; property: "y"; value: screenshotImage.y}

44− Binding { target: UbuntuKeyboardInfo; property: "width"; value: screenshotImage.width}

45− Binding { target: UbuntuKeyboardInfo; property: "height"; value: screenshotImage.height}

46}

41}

tests/qmltests/Stages/tst_DesktopStage.qml+3−3

@@ -534,14 +534,14 @@ Item {

534 verify(dashAppDelegate);

535 dashAppDelegate.requestedHeight = data.windowHeight;

536 dashAppDelegate.requestedY = data.windowY;

537− UbuntuKeyboardInfo.height = 0;

537+ SurfaceManager.inputMethodSurface.setInputBounds(Qt.rect(0, 0, 0, 0));

538 var initialY = dashAppDelegate.y;

539 verify(initialY > PanelState.panelHeight);

540

541− UbuntuKeyboardInfo.height = root.height / 2;

541+ SurfaceManager.inputMethodSurface.setInputBounds(Qt.rect(0, root.height / 2, root.width, root.height / 2));

542 tryCompare(dashAppDelegate, "y", data.targetDisplacement);

543

544− UbuntuKeyboardInfo.height = 0;

544+ SurfaceManager.inputMethodSurface.setInputBounds(Qt.rect(0, 0, 0, 0));

545 tryCompare(dashAppDelegate, "y", initialY);

546 SurfaceManager.inputMethodSurface.state = oldOSKState;

547 }

tests/qmltests/Tutorial/tst_Tutorial.qml+1−1

@@ -715,7 +715,7 @@ Rectangle {

715 surface.setState(Mir.RestoredState);

716

717 var inputMethod = findInvisibleChild(shell, "inputMethod");

718− tryCompare(inputMethod, "state", "shown");

718+ tryCompare(inputMethod, "visible", true);

719

720 verify(tutorialLeftLoader.shown);

721 }