efdd41f · unity8 · git.iko.soy

git.iko.soy/unity8

on xenial help sign in

set lp:~lukas-kde/unity8/fix-window-title-vs-menu as prereq

efdd41fMichael Zanetti10 years ago

qml/Panel/Panel.qml+8−8

compare with xenial

unifiedsplit

@@ -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

@@ -85,15 +85,15 @@ Item {

85 property bool showWindowDecorationControls: (revealControls && PanelState.decorationsVisible) ||

86 PanelState.decorationsAlwaysVisible

88− property bool showPointerMenu: revealControls &&

88+ property bool showPointerMenu: revealControls && enablePointerMenu &&

89 (PanelState.decorationsVisible || mode == "staged")

91− property bool enablePointerMenu: revealControls &&

92− applicationMenus.available &&

91+ property bool enablePointerMenu: applicationMenus.available &&

93 applicationMenus.model

92 applicationMenus.model

95 property bool showTouchMenu: !greeterShown &&

94 property bool showTouchMenu: !greeterShown &&

96− !showPointerMenu

95+ !showPointerMenu &&

96+ !showWindowDecorationControls

98 property bool enableTouchMenus: showTouchMenu &&

99 applicationMenus.available &&

@@ -211,7 +211,7 @@ Item {

211 opacity: d.showPointerMenu ? 1 : 0

212 visible: opacity != 0

213 Behavior on opacity { UbuntuNumberAnimation { duration: UbuntuAnimation.SnapDuration } }

214− active: __applicationMenus.model && !callHint.visible

214+ active: d.showPointerMenu && !callHint.visible

215

216 width: parent.width - windowControlButtons.width - units.gu(2) - __indicators.barWidth

217

@@ -220,7 +220,7 @@ Item {

220 sourceComponent: MenuBar {

221 id: bar

222 objectName: "menuBar"

223− anchors.left: parent.left

223+ anchors.left: parent ? parent.left : undefined

224 anchors.margins: units.gu(1)

225 height: menuBarLoader.height

226 enableKeyFilter: valid && PanelState.decorationsVisible

@@ -339,7 +339,7 @@ Item {

339 maximumLineCount: 1

340 fontSize: "medium"

341 font.weight: Font.Medium

342− color: Theme.palette.selected.backgroundText

342+ color: theme.palette.selected.backgroundText

343 opacity: __applicationMenus.visible && !__applicationMenus.expanded ? 1 : 0

344 visible: opacity != 0

345 Behavior on opacity { NumberAnimation { duration: UbuntuAnimation.SnapDuration } }

qml/Stage/DecoratedWindow.qml+1

@@ -38,6 +38,7 @@ FocusScope {

38 property alias maximizeButtonShown: decoration.maximizeButtonShown

39 property alias interactive: applicationWindow.interactive

40 readonly property alias orientationChangesEnabled: applicationWindow.orientationChangesEnabled

41+ property alias windowControlButtonsVisible: decoration.windowControlButtonsVisible

42 // Changing this will actually add/remove a decoration, meaning, requestedHeight will take the decoration into account.

43 // Changing this will actually add/remove a decoration, meaning, requestedHeight will take the decoration into account.

43 property bool hasDecoration: true

44 property bool hasDecoration: true

qml/Stage/Stage.qml+3−2

@@ -391,7 +391,7 @@ FocusScope {

391 Binding {

392 target: PanelState

393 property: "decorationsVisible"

394− value: priv.focusedAppDelegate !== null && priv.focusedAppDelegate.maximized // FIXME for Locally integrated menus

394+ value: mode == "windowed" && priv.focusedAppDelegate && priv.focusedAppDelegate.maximized && !root.spreadShown

395 }

396

397 Binding {

@@ -1393,6 +1393,7 @@ FocusScope {

1393 requestedHeight: appContainer.height;

1394 }

1395 PropertyChanges { target: touchControls; enabled: true }

1396+ PropertyChanges { target: decoratedWindow; windowControlButtonsVisible: false }

1396 },

1397 },

1397 State {

1398 State {

1398 name: "fullscreen"; when: appDelegate.fullscreen && !appDelegate.minimized

1399 name: "fullscreen"; when: appDelegate.fullscreen && !appDelegate.minimized

@@ -1415,7 +1416,7 @@ FocusScope {

1415 }

1416 }

1416 PropertyChanges { target: touchControls; enabled: true }

1417 PropertyChanges { target: touchControls; enabled: true }

1417 PropertyChanges { target: resizeArea; enabled: true }

1418 PropertyChanges { target: resizeArea; enabled: true }

1418− PropertyChanges { target: decoratedWindow; shadowOpacity: .3}

1419+ PropertyChanges { target: decoratedWindow; shadowOpacity: .3; windowControlButtonsVisible: true}

1419 },

1420 },

1420 State {

1421 State {

1421 name: "restored";

1422 name: "restored";

qml/Stage/WindowDecoration.qml+1

@@ -34,6 +34,7 @@ MouseArea {

34 property var menu: undefined

35 property bool enableMenus: true

36 property bool windowMoving: false

37+ property alias windowControlButtonsVisible: buttons.visible

38 readonly property real buttonsWidth: buttons.width + row.spacing

39 readonly property real buttonsWidth: buttons.width + row.spacing

tests/qmltests/Panel/tst_Panel.qml+3−25

@@ -779,40 +779,18 @@ PanelTest {

779

780 var appTitle = findChild(panel, "panelTitle"); verify(appTitle);

781 var appMenuRow = findChild(panel.applicationMenus, "panelRow"); verify(appMenuRow);

782− var appMenuBar = findChild(panel, "menuBar"); verify(appMenuBar);

782+ var menuBarLoader = findChild(panel, "menuBarLoader"); verify(menuBarLoader);

783

784 tryCompare(appTitle, "visible", true, undefined, "App title should be visible");

785− tryCompare(appMenuBar, "visible", false, undefined, "App menu bar should not be visible");

785+ tryCompare(menuBarLoader, "visible", false, undefined, "App menu bar should not be visible");

786

787 mouseMove(panel, panel.width/2, panel.panelHeight);

788

789+ var appMenuBar = findChild(panel, "menuBar"); verify(appMenuBar);

789 tryCompare(appTitle, "visible", false, undefined, "App title should not be visible on mouse hover");

790 tryCompare(appTitle, "visible", false, undefined, "App title should not be visible on mouse hover");

790 tryCompare(appMenuBar, "visible", true, undefined, "App menu bar should be visible on mouse hover");

791 tryCompare(appMenuBar, "visible", true, undefined, "App menu bar should be visible on mouse hover");

791 }

792 }

792

793

793− function test_windowedApplicationMenuShowOnMouseHoverWhenDecorationsShown() {

794− PanelState.title = "Fake Title";

795− panel.mode = "windowed";

796− mouseEmulation.checked = false;

797−

798− var appTitle = findChild(panel, "panelTitle"); verify(appTitle);

799− var appMenuRow = findChild(panel.applicationMenus, "panelRow"); verify(appMenuRow);

800− var appMenuBar = findChild(panel, "menuBar"); verify(appMenuBar);

801−

802− tryCompare(appTitle, "visible", true, undefined, "App title should be visible");

803− tryCompare(appMenuBar, "visible", false, undefined, "App menu bar should not be visible");

804−

805− mouseMove(panel, panel.width/2, panel.panelHeight);

806−

807− tryCompare(appTitle, "visible", true, undefined, "App title should still be visible on mouse hover when panel decorations are not visible");

808− tryCompare(appMenuBar, "visible", false, undefined, "App menu bar should be visible on mouse hover when panel decorations are not visible");

809−

810− PanelState.decorationsVisible = true;

811−

812− tryCompare(appTitle, "visible", false, undefined, "App title should still be visible on mouse hover when panel decorations are visible");

813− tryCompare(appMenuBar, "visible", true, undefined, "App menu bar should be visible on mouse hover when panel decorations not visible");

814− }

815−

816 function test_keyboardNavigation_data() {

794 function test_keyboardNavigation_data() {

817 return [

795 return [

818 {tag: "tab to start", doTab: false},

796 {tag: "tab to start", doTab: false},

tests/qmltests/Stage/tst_DesktopStage.qml+1−5

@@ -870,11 +870,7 @@ Item {

870 maximizeAppDelegate(appDelegate);

871

872 // Close the window and restart the application

873− var closeButton = findChild(appDelegate, "closeWindowButton");

873+ appDelegate.close();

874− appDelegate = null;

875− verify(closeButton);

876− mouseClick(closeButton);

877− closeButton = null;

878 tryCompare(topSurfaceList, "count", originalWindowCount);

874 tryCompare(topSurfaceList, "count", originalWindowCount);

879 wait(100); // plus some spare room

875 wait(100); // plus some spare room

880 appDelegate = startApplication("dialer-app");

876 appDelegate = startApplication("dialer-app");