Merge lp:~nik90/unav/unav-convergence-part-2 into lp:unav

Proposed by costales
Status: Merged
Merged at revision: 69
Proposed branch: lp:~nik90/unav/unav-convergence-part-2
Merge into: lp:unav
Diff against target: 95 lines (+24/-2)
4 files modified
qml/Location.qml (+1/-1)
qml/Main.qml (+1/-0)
qml/components/CloseHeaderAction.qml (+1/-0)
qml/tuto/components/Walkthrough.qml (+21/-1)
To merge this branch: bzr merge lp:~nik90/unav/unav-convergence-part-2
Reviewer Review Type Date Requested Status
costales Approve
Review via email: mp+296137@code.launchpad.net

Commit message

- Adds keyboard shortcuts to the welcome wizard
- The sidebar can be closed by pressing "Escape"
- Search page can be opened by pressing "Ctrl+F"

Description of the change

As Nekhelesh said it is done and it's blocking another branches, I would like to propose it.

To post a comment you must log in.
Revision history for this message
costales (costales) wrote :

As it's a big change, I would prefer to approve and refining in the end version.
What do you think? :)

review: Approve
Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

> As it's a big change, I would prefer to approve and refining in the end
> version.
> What do you think? :)

This is a small change. This MP adds very minor features and code which won't affect phone users and tablet users in any way unless they attach a keyboard. So current functionality won't be compromised or affected in any way.

I am not sure what you mean by end version. My plan was to slowly add keyboard shortcuts to improve convergence.

Revision history for this message
costales (costales) wrote :

Great :)

By end version, all together and ready for big QA from us :)

a hug!

--
Sent using Dekko from my Ubuntu device

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'qml/Location.qml'
2--- qml/Location.qml 2016-05-05 19:21:33 +0000
3+++ qml/Location.qml 2016-05-31 20:05:36 +0000
4@@ -298,7 +298,7 @@
5 height: units.gu(2.5)
6 width: height
7 visible: model.icon !== ""
8- source: model.icon
9+ source: model.icon ? model.icon : ""
10 SlotsLayout.position: SlotsLayout.Last
11 }
12 }
13
14=== modified file 'qml/Main.qml'
15--- qml/Main.qml 2016-05-20 17:26:15 +0000
16+++ qml/Main.qml 2016-05-31 20:05:36 +0000
17@@ -210,6 +210,7 @@
18 Action {
19 id: searchAction
20 iconName: "find"
21+ shortcut: "Ctrl+F"
22 text: i18n.tr("Search")
23 enabled: navigationPage.buttonsEnabled
24 onTriggered: {
25
26=== modified file 'qml/components/CloseHeaderAction.qml'
27--- qml/components/CloseHeaderAction.qml 2016-04-28 16:07:19 +0000
28+++ qml/components/CloseHeaderAction.qml 2016-05-31 20:05:36 +0000
29@@ -20,6 +20,7 @@
30 id: closeButton
31 iconName: "close"
32 visible: mainPageStack.columns !== 1
33+ shortcut: "Escape"
34 onTriggered: {
35 mainPageStack.hideSideBar()
36 mainPageStack.removePages(mainPageStack.primaryPage)
37
38=== modified file 'qml/tuto/components/Walkthrough.qml'
39--- qml/tuto/components/Walkthrough.qml 2016-04-14 15:32:41 +0000
40+++ qml/tuto/components/Walkthrough.qml 2016-05-31 20:05:36 +0000
41@@ -31,7 +31,7 @@
42 property list<Component> model
43
44 // Property to set the color of bottom cirle to indicate the user's progress
45- property color completeColor: "lightblue"
46+ property color completeColor: "#398DFF"
47
48 // Property to set the color of the bottom circle to indicate the slide still left to cover
49 property color inCompleteColor: "lightgrey"
50@@ -98,8 +98,11 @@
51 }
52
53 MouseArea {
54+ hoverEnabled: true
55 anchors.fill: parent
56 onClicked: walkthrough.finished()
57+ onEntered: skipLabel.color = completeColor
58+ onExited: skipLabel.color = skipTextColor
59 }
60 }
61
62@@ -131,10 +134,21 @@
63 duration: UbuntuAnimation.FastDuration
64 }
65 }
66+
67+ MouseArea {
68+ anchors.fill: parent
69+ onClicked: listView.currentIndex = index
70+ }
71 }
72 }
73 }
74
75+ Keys.onRightPressed: {
76+ if (listView.currentIndex !== listView.count-1) {
77+ listView.currentIndex++
78+ }
79+ }
80+
81 ActionButton {
82 id: rightchevron
83
84@@ -152,6 +166,12 @@
85 onClicked: listView.currentIndex++
86 }
87
88+ Keys.onLeftPressed: {
89+ if (listView.currentIndex !== 0) {
90+ listView.currentIndex--
91+ }
92+ }
93+
94 ActionButton {
95 id: leftchevron
96

Subscribers

People subscribed via source and target branches