on xenial help sign in
merge lp:~aacid/unity8/create_less_components
commitcf2fee684f4f26f2f1c4723e5cf1c77559a8de78 authorLukáš Tinkl <lukas.tinkl@canonical.com> dateFri 3 Mar 2017, 00:56 (10 years ago) parent00b0fd9244905e
4 files changed+96−93
qml/ApplicationMenus/MenuBar.qml+13−12 qml/ApplicationMenus/MenuItem.qml+6−6 qml/ApplicationMenus/MenuPopup.qml+75−73 tests/qmltests/ApplicationMenus/tst_MenuBar.qml+2−2
qml/ApplicationMenus/MenuBar.qml+13−12
@@ -104,6 +104,11 @@ Item {
104104 onModelReset: d.firstInvisibleIndex = undefined
105105 }
106106
107+ Component {
108+ id: menuComponent
109+ MenuPopup { }
110+ }
111+
107112 Repeater {
108113 id: rowRepeater
109114
@@ -127,7 +132,14 @@ Item {
127132
128133 function show() {
129134 if (!__popup) {
130− __popup = menuComponent.createObject(root, { objectName: visualItem.objectName + "-menu" });
135+ __popup = menuComponent.createObject(root,
136+ {
137+ objectName: visualItem.objectName + "-menu",
138+ desiredX: Qt.binding(function() { return visualItem.x - units.gu(1); }),
139+ desiredY: Qt.binding(function() { return root.height; }),
140+ unityMenuModel: Qt.binding(function() { return root.unityMenuModel.submenu(visualItem.__ownIndex); })
141+ });
142+ __popup.reset();
131143 __popup.childActivated.connect(dismiss);
132144 // force the current item to be the newly popped up menu
133145 } else {
@@ -170,17 +182,6 @@ Item {
170182 onDismissAll: visualItem.dismiss()
171183 }
172184
173− Component {
174− id: menuComponent
175− MenuPopup {
176− desiredX: visualItem.x - units.gu(1)
177− desiredY: parent.height
178− unityMenuModel: root.unityMenuModel.submenu(visualItem.__ownIndex)
179−
180− Component.onCompleted: reset();
181− }
182− }
183−
184185 RowLayout {
185186 id: column
186187 spacing: units.gu(1)
qml/ApplicationMenus/MenuItem.qml+6−6
@@ -34,7 +34,7 @@ ActionItem {
3434 val += units.gu(1) + title.contentWidth;
3535 if (hasSubmenu) {
3636 val += units.gu(1) + chevronIcon.width;
37− } else if (shortcut != undefined) {
37+ } else if (menuData && menuData.shortcut != undefined) {
3838 val += units.gu(3) + shortcutLabel.contentWidth;
3939 }
4040 return val + units.gu(1);
@@ -49,9 +49,9 @@ ActionItem {
4949 enabled: root.enabled
5050
5151 // FIXME - SDK Action:text modifies menu text with html underline for mnemonic
52− text: menuData.label.replace("_", "&").replace("<u>", "&").replace("</u>", "")
53− checkable: menuData.isCheck || menuData.isRadio
54− checked: menuData.isToggled
52+ text: menuData ? menuData.label.replace("_", "&").replace("<u>", "&").replace("</u>", "") : ""
53+ checkable: menuData && (menuData.isCheck || menuData.isRadio)
54+ checked: menuData && menuData.isToggled
5555 }
5656
5757 width: {
@@ -127,8 +127,8 @@ ActionItem {
127127 color: enabled ? theme.palette.normal.overlaySecondaryText :
128128 theme.palette.disabled.overlaySecondaryText
129129
130− visible: menuData.shortcut != undefined && !root.hasSubmenu && QuickUtils.keyboardAttached
131− text: menuData.shortcut ? menuData.shortcut : ""
130+ visible: menuData && menuData.shortcut != undefined && !root.hasSubmenu && QuickUtils.keyboardAttached
131+ text: menuData && menuData.shortcut ? menuData.shortcut : ""
132132 }
133133 }
134134
qml/ApplicationMenus/MenuPopup.qml+75−73
@@ -276,6 +276,79 @@ UbuntuShape {
276276 }
277277 }
278278
279+ Component {
280+ id: separatorComponent
281+ ListItems.ThinDivider {
282+ // Parent will be loader
283+ objectName: parent.objectName + "-separator"
284+ implicitHeight: units.dp(2)
285+ }
286+ }
287+
288+ Component {
289+ id: menuItemComponent
290+ MenuItem {
291+ // Parent will be loader
292+ id: menuItem
293+ menuData: parent.__menuData
294+ objectName: parent.objectName + "-actionItem"
295+
296+ width: MathUtils.clamp(implicitWidth, d.__minimumWidth, d.__maximumWidth)
297+
298+ action.onTriggered: {
299+ submenuHoverTimer.stop();
300+
301+ d.currentItem = parent;
302+
303+ if (hasSubmenu) {
304+ if (!popup) {
305+ var model = root.unityMenuModel.submenu(__ownIndex);
306+ popup = submenuComponent.createObject(focusScope, {
307+ objectName: parent.objectName + "-",
308+ unityMenuModel: model,
309+ substractWidth: true,
310+ desiredX: Qt.binding(function() { return root.width }),
311+ desiredY: Qt.binding(function() {
312+ var dummy = listView.contentY; // force a recalc on contentY change.
313+ return mapToItem(container, 0, y).y;
314+ })
315+ });
316+ popup.retreat.connect(function() {
317+ popup.destroy();
318+ popup = null;
319+ menuItem.forceActiveFocus();
320+ });
321+ popup.childActivated.connect(function() {
322+ popup.destroy();
323+ popup = null;
324+ root.childActivated();
325+ });
326+ } else if (popup) {
327+ popup.visible = true;
328+ }
329+ } else {
330+ root.unityMenuModel.activate(__ownIndex);
331+ root.childActivated();
332+ }
333+ }
334+
335+ Connections {
336+ target: d
337+ onCurrentIndexChanged: {
338+ if (popup && d.currentIndex != __ownIndex) {
339+ popup.visible = false;
340+ }
341+ }
342+ onDismissAll: {
343+ if (popup) {
344+ popup.destroy();
345+ popup = null;
346+ }
347+ }
348+ }
349+ }
350+ }
351+
279352 ColumnLayout {
280353 id: menuColumn
281354 spacing: 0
@@ -289,6 +362,8 @@ UbuntuShape {
289362 id: loader
290363 objectName: root.objectName + "-item" + __ownIndex
291364
365+ property Item popup: null
366+ property var __menuData: model
292367 property int __ownIndex: index
293368 property bool __isSeparator: model.isSeparator
294369
@@ -301,80 +376,7 @@ UbuntuShape {
301376 return menuItemComponent;
302377 }
303378
304− property Item popup: null
305−
306379 Layout.fillWidth: true
307−
308− Component {
309− id: menuItemComponent
310− MenuItem {
311− id: menuItem
312− menuData: model
313− objectName: loader.objectName + "-actionItem"
314−
315− width: MathUtils.clamp(implicitWidth, d.__minimumWidth, d.__maximumWidth)
316−
317− action.onTriggered: {
318− submenuHoverTimer.stop();
319−
320− d.currentItem = loader;
321−
322− if (hasSubmenu) {
323− if (!popup) {
324− var model = root.unityMenuModel.submenu(__ownIndex);
325− popup = submenuComponent.createObject(focusScope, {
326− objectName: loader.objectName + "-",
327− unityMenuModel: model,
328− substractWidth: true,
329− desiredX: Qt.binding(function() { return root.width }),
330− desiredY: Qt.binding(function() {
331− var dummy = listView.contentY; // force a recalc on contentY change.
332− return mapToItem(container, 0, y).y;
333− })
334− });
335− popup.retreat.connect(function() {
336− popup.destroy();
337− popup = null;
338− menuItem.forceActiveFocus();
339− });
340− popup.childActivated.connect(function() {
341− popup.destroy();
342− popup = null;
343− root.childActivated();
344− });
345− } else if (popup) {
346− popup.visible = true;
347− }
348− } else {
349− root.unityMenuModel.activate(__ownIndex);
350− root.childActivated();
351− }
352− }
353−
354− Connections {
355− target: d
356− onCurrentIndexChanged: {
357− if (popup && d.currentIndex != __ownIndex) {
358− popup.visible = false;
359− }
360− }
361− onDismissAll: {
362− if (popup) {
363− popup.destroy();
364− popup = null;
365− }
366− }
367− }
368− }
369− }
370−
371− Component {
372− id: separatorComponent
373− ListItems.ThinDivider {
374− objectName: loader.objectName + "-separator"
375− implicitHeight: units.dp(2)
376− }
377− }
378380 }
379381
380382 }
tests/qmltests/ApplicationMenus/tst_MenuBar.qml+2−2
@@ -81,10 +81,10 @@ Item {
8181 name: "MenuBar"
8282 when: windowShown
8383
84− function cleanup() {
84+ function init() {
8585 menuBar.dismiss();
8686 activatedSpy.clear();
87− menuBackend.modelData = null;
87+ waitForRendering(menuBar);
8888 }
8989
9090 function test_mouseNavigation() {