bb15401 · unity8 · git.iko.soy

git.iko.soy/unity8

first support for quicklists

commitbb154016aab8a960eb25bc3d21386b71415ad8ff authorMichael Zanetti <michael.zanetti@canonical.com> dateMon 19 Aug 2013, 17:10 (13 years ago) parentb741133

13 files changed+160−7

compare with xenial

unifiedsplit

Launcher/Launcher.qml+2−2 Launcher/LauncherPanel.qml+55−1 plugins/Unity/Launcher/common/quicklistentry.cpp+5 plugins/Unity/Launcher/common/quicklistentry.h+2 plugins/Unity/Launcher/launcheritem.cpp+3 plugins/Unity/Launcher/launchermodel.cpp+3 plugins/Unity/Launcher/quicklistmodel.cpp+2 tests/mocks/Unity/Launcher/MockLauncherItem.cpp−1 tests/mocks/Unity/Launcher/MockLauncherModel.cpp+1−3 tests/mocks/Unity/Launcher/MockLauncherModel.h+3 tests/mocks/Unity/Launcher/MockQuickListModel.cpp+2 tests/mocks/Unity/Launcher/plugin.cpp+3 tests/qmltests/Launcher/tst_Launcher.qml+79

Launcher/Launcher.qml+2−2

@@ -73,7 +73,7 @@ Item {

7373 id: dismissTimer

7474 interval: 5000

7575 onTriggered: {

76− if (!panel.moving) {

76+ if (!panel.preventHiding) {

7777 root.state = ""

7878 } else {

7979 dismissTimer.restart()

@@ -162,7 +162,7 @@ Item {

162162 root.dashItemSelected(index)

163163 }

164164

165− onMovingChanged: {

165+ onPreventHidingChanged: {

166166 if (dismissTimer.running) {

167167 dismissTimer.restart();

168168 }

Launcher/LauncherPanel.qml+55−1

@@ -19,6 +19,7 @@ import Ubuntu.Components 0.1

1919import Ubuntu.Components.ListItems 0.1 as ListItems

2020import Unity 0.1

2121import Unity.Launcher 0.1

22+import Ubuntu.Components.Popups 0.1

2223import "../Components/ListItems"

2324

2425Item {

@@ -29,7 +30,8 @@ Item {

2930 property var model

3031 property bool inverted: true

3132 property bool dragging: false

32− property bool moving: launcherListView.moving || launcherListView.flicking || dndArea.draggedIndex >= 0

33+ property bool moving: launcherListView.moving || launcherListView.flicking

34+ property bool preventHiding: moving || dndArea.draggedIndex >= 0 || dndArea.quickListPopover !== null || dndArea.pressed

3335 property int highlightIndex: -1

3436

3537 signal applicationSelected(string desktopFile)

@@ -219,6 +221,7 @@ Item {

219221

220222 MouseArea {

221223 id: dndArea

224+ objectName: "dndArea"

222225 anchors {

223226 fill: parent

224227 topMargin: launcherListView.topMargin

@@ -234,6 +237,7 @@ Item {

234237 property bool postDragging: false

235238 property int startX

236239 property int startY

240+ property var quickListPopover: null

237241

238242 onPressed: {

239243 selectedItem = launcherListView.itemAt(mouseX, mouseY + launcherListView.realContentY)

@@ -295,6 +299,13 @@ Item {

295299 onPressAndHold: {

296300 draggedIndex = Math.floor((mouseY + launcherListView.realContentY) / launcherListView.realItemHeight);

297301

302+ // Opening QuickList

303+ var quickListModel = launcherListView.model.get(draggedIndex).quickList

304+ var quickListAppId = launcherListView.model.get(draggedIndex).appId

305+

306+ quickListPopover = PopupUtils.open(popoverComponent, selectedItem,

307+ {model: quickListModel, appId: quickListAppId})

308+

298309 launcherListView.interactive = false

299310

300311 var yOffset = draggedIndex > 0 ? (mouseY + launcherListView.realContentY) % (draggedIndex * launcherListView.realItemHeight) : mouseY + launcherListView.realContentY

@@ -314,6 +325,7 @@ Item {

314325 var distance = Math.max(Math.abs(mouseX - startX), Math.abs(mouseY - startY))

315326 if (!preDragging && distance > units.gu(1.5)) {

316327 preDragging = true;

328+ PopupUtils.close(quickListPopover)

317329 }

318330 if (distance > launcherListView.itemHeight) {

319331 selectedItem.dragging = true

@@ -419,4 +431,46 @@ Item {

419431 }

420432 }

421433 }

434+

435+ Component {

436+ id: popoverComponent

437+

438+ Popover {

439+ id: popover

440+ property var model

441+ property string appId

442+ contentWidth: quickListColumn.width

443+

444+ // FIXME: There's a bug in the Popover positioning that it covers the item in case it is rotated.

445+ // https://bugs.launchpad.net/ubuntu-ui-toolkit/+bug/1204470

446+ // For now, let's move the Popover around with callerMargin.

447+ // Remove popupMargin once the bug is fixed.

448+ property int popupMargin: root.inverted ? launcherListView.itemHeight : 0;

449+

450+ callerMargin: units.gu(1) + popupMargin

451+

452+ Column {

453+ id: quickListColumn

454+ width: units.gu(30)

455+ height: childrenRect.height

456+

457+ Repeater {

458+ model: popover.model

459+

460+ ListItems.Standard {

461+ objectName: "quickListEntry" + index

462+ text: (model.clickable ? "" : "<b>") + model.label + (model.clickable ? "" : "</b>")

463+ highlightWhenPressed: model.clickable

464+ onClicked: {

465+ if (!model.clickable) {

466+ return;

467+ }

468+ LauncherModel.quickListActionInvoked(appId, index);

469+ PopupUtils.close(popover);

470+ }

471+ }

472+ }

473+ }

474+ }

475+ }

422476}

plugins/Unity/Launcher/common/quicklistentry.cpp+5

@@ -52,3 +52,8 @@ void QuickListEntry::setIcon(const QString &icon)

5252{

5353 m_icon = icon;

5454}

55+

56+bool QuickListEntry::hasAction() const

57+{

58+ return !m_actionId.isEmpty();

59+}

plugins/Unity/Launcher/common/quicklistentry.h+2

@@ -35,6 +35,8 @@ public:

3535 QString icon() const;

3636 void setIcon(const QString &icon);

3737

38+ bool hasAction() const;

39+

3840private:

3941 QString m_actionId;

4042 QString m_text;

plugins/Unity/Launcher/launcheritem.cpp+3

@@ -33,6 +33,9 @@ LauncherItem::LauncherItem(const QString &appId, const QString &desktopFile, con

3333 m_count(0),

3434 m_quickList(new QuickListModel(this))

3535{

36+ QuickListEntry nameAction;

37+ nameAction.setText(m_name);

38+ m_quickList->appendAction(nameAction);

3639 QuickListEntry pinningAction;

3740 pinningAction.setActionId("pin_item");

3841 pinningAction.setText("Pin to Launcher");

plugins/Unity/Launcher/launchermodel.cpp+3

@@ -163,6 +163,9 @@ void LauncherModel::quickListActionInvoked(const QString &appId, int actionIndex

163163 QString actionId = model->get(actionIndex).actionId();

164164

165165 // Check if this is one of the launcher actions we handle ourselves

166+ if (actionId == "no_action") {

167+ return;

168+ }

166169 if (actionId == "pin_item") {

167170 if (item->pinned()) {

168171 requestRemove(appId);

plugins/Unity/Launcher/quicklistmodel.cpp+2

@@ -66,6 +66,8 @@ QVariant QuickListModel::data(const QModelIndex &index, int role) const

6666 return m_list.at(index.row()).text();

6767 case RoleIcon:

6868 return m_list.at(index.row()).icon();

69+ case RoleClickable:

70+ return m_list.at(index.row()).hasAction();

6971 }

7072 return QVariant();

7173}

tests/mocks/Unity/Launcher/MockLauncherItem.cpp−1

@@ -35,7 +35,6 @@ MockLauncherItem::MockLauncherItem(const QString &appId, const QString& desktopF

3535 m_count(8),

3636 m_quickList(new MockQuickListModel(this))

3737{

38−

3938}

4039

4140QString MockLauncherItem::appId() const

tests/mocks/Unity/Launcher/MockLauncherModel.cpp+1−3

@@ -135,9 +135,7 @@ void MockLauncherModel::requestRemove(const QString &appId)

135135

136136void MockLauncherModel::quickListActionInvoked(const QString &appId, int actionIndex)

137137{

138− Q_UNUSED(appId)

139− Q_UNUSED(actionIndex)

140− // Nothing to mock yet...

138+ Q_EMIT quickListTriggered(appId, actionIndex);

141139}

142140

143141int MockLauncherModel::findApp(const QString &appId)

tests/mocks/Unity/Launcher/MockLauncherModel.h+3

@@ -44,6 +44,9 @@ public:

4444 Q_INVOKABLE void requestRemove(const QString &appId);

4545 Q_INVOKABLE void quickListActionInvoked(const QString &appId, int actionIndex);

4646

47+Q_SIGNALS:

48+ void quickListTriggered(const QString &appId, int index);

49+

4750private:

4851 int findApp(const QString &appId);

4952

tests/mocks/Unity/Launcher/MockQuickListModel.cpp+2

@@ -35,6 +35,8 @@ QVariant MockQuickListModel::data(const QModelIndex &index, int role) const

3535 return QLatin1String("test menu entry ") + QString::number(index.row());

3636 case RoleIcon:

3737 return QLatin1String("copy.png");

38+ case RoleClickable:

39+ return index.row() == 1 ? false : true;

3840 }

3941 return QVariant();

4042}

tests/mocks/Unity/Launcher/plugin.cpp+3

@@ -20,6 +20,7 @@

2020#include "plugin.h"

2121#include "MockLauncherModel.h"

2222#include "MockLauncherItem.h"

23+#include "MockQuickListModel.h"

2324

2425#include <unity/shell/launcher/LauncherModelInterface.h>

2526#include <unity/shell/launcher/LauncherItemInterface.h>

@@ -39,7 +40,9 @@ void TestLauncherPlugin::registerTypes(const char* uri)

3940 // @uri Unity.Launcher

4041 qmlRegisterUncreatableType<LauncherModelInterface>(uri, 0, 1, "LauncherModelInterface", "Abstract Interface. Cannot be instantiated.");

4142 qmlRegisterUncreatableType<LauncherItemInterface>(uri, 0, 1, "LauncherItemInterface", "Abstract Interface. Cannot be instantiated.");

43+ qmlRegisterUncreatableType<QuickListModelInterface>(uri, 0, 1, "QuickListModelInterface", "Abstract Interface. Cannot be instantiated.");

4244

4345 qmlRegisterSingletonType<MockLauncherModel>(uri, 0, 1, "LauncherModel", modelProvider);

4446 qmlRegisterUncreatableType<MockLauncherItem>(uri, 0, 1, "LauncherItem", "Can't create LauncherItems in QML. Get them from the LauncherModel");

47+ qmlRegisterUncreatableType<MockQuickListModel>(uri, 0, 1, "QuickListModel", "Can't create QuickLists in QML. Get them from the LauncherItems");

4548}

tests/qmltests/Launcher/tst_Launcher.qml+79

@@ -49,6 +49,11 @@ Item {

4949 property int maxPanelX: 0

5050 }

5151

52+ SignalSpy {

53+ id: signalSpy

54+ target: LauncherModel

55+ }

56+

5257 Connections {

5358 target: testCase.findChild(launcher, "launcherPanel")

5459

@@ -306,5 +311,79 @@ Item {

306311 tryCompare(draggedItem, "itemOpacity", 1)

307312 tryCompare(fakeDragItem, "visible", false)

308313 }

314+

315+ function test_quicklist_dismiss() {

316+ revealer.dragLauncherIntoView();

317+ var draggedItem = findChild(launcher, "launcherDelegate5")

318+ var item0 = findChild(launcher, "launcherDelegate0")

319+ var fakeDragItem = findChild(launcher, "fakeDragItem")

320+ var dndArea = findChild(launcher, "dndArea")

321+

322+ // Initial state

323+ compare(dndArea.quickListPopover, null)

324+

325+ // Doing longpress

326+ mousePress(draggedItem, draggedItem.width / 2, draggedItem.height / 2)

327+ tryCompare(fakeDragItem, "visible", true) // Wait longpress happening

328+ verify(dndArea.quickListPopover != null)

329+

330+ // Dragging a bit (> 1.5 gu)

331+ mouseMove(draggedItem, -units.gu(2), draggedItem.height / 2)

332+

333+ // QuickList needs to be closed when a drag operation starts

334+ tryCompare(dndArea, "quickListPopover", null)

335+

336+ mouseRelease(draggedItem);

337+ }

338+

339+ function test_quicklist_click_data() {

340+ return [

341+ {tag: "non-clickable", index: 1, clickable: false },

342+ {tag: "clickable", index: 2, clickable: true },

343+ ];

344+ }

345+

346+ function test_quicklist_click(data) {

347+ revealer.dragLauncherIntoView();

348+ var clickedItem = findChild(launcher, "launcherDelegate5")

349+ var dndArea = findChild(launcher, "dndArea")

350+

351+ // Initial state

352+ compare(dndArea.quickListPopover, null)

353+

354+ // Doing longpress

355+ mousePress(clickedItem, clickedItem.width / 2, clickedItem.height / 2)

356+ tryCompare(clickedItem, "itemOpacity", 0) // Wait for longpress to happen

357+ verify(dndArea.quickListPopover != null)

358+

359+ mouseRelease(clickedItem);

360+

361+ var quickListEntry = findChild(dndArea.quickListPopover, "quickListEntry" + data.index)

362+

363+ signalSpy.clear();

364+ signalSpy.signalName = "quickListTriggered"

365+

366+ mouseClick(quickListEntry, quickListEntry.width / 2, quickListEntry.height / 2)

367+

368+ if (data.clickable) {

369+ // QuickList needs to be closed when some clickable item is clicked

370+ tryCompare(dndArea, "quickListPopover", null)

371+

372+ compare(signalSpy.count, 1, "Quicklist signal wasn't triggered")

373+ compare(signalSpy.signalArguments[0][0], LauncherModel.get(5).appId)

374+ compare(signalSpy.signalArguments[0][1], 2)

375+

376+ } else {

377+

378+ // QuickList must not be closed when a non-clickable item is clicked

379+ verify(dndArea.quickListPopover != null)

380+

381+ compare(signalSpy.count, 0, "Quicklist signal must NOT be triggered when clicking a non-clickable item")

382+

383+ // Click somewhere in the empty space to dismiss the quicklist

384+ mouseClick(launcher, launcher.width - units.gu(1), units.gu(1));

385+ tryCompare(dndArea, "quickListPopover", null)

386+ }

387+ }

309388 }

310389}