fb2f4b7 · unity8 · git.iko.soy

git.iko.soy/unity8

on xenial help sign in

Fix Z-ordering of prompt surfaces

fb2f4b7Daniel d'Andrada10 years ago

qml/Stages/ApplicationWindow.qml+25−9

compare with xenial

unifiedsplit

@@ -121,15 +121,12 @@ FocusScope {

121

122 property bool surfaceOldEnoughToBeResized: false

123

124− property var focusedSurface: {

125− if (promptSurfacesRepeater.count == 0) {

124+ property Item focusedSurface: promptSurfacesRepeater.count === 0 ? surfaceContainer

126− return surfaceContainer;

127− } else {

128− return promptSurfacesRepeater.itemAt(promptSurfacesRepeater.count - 1);

125+ : promptSurfacesRepeater.first

129− }

130− }

131 onFocusedSurfaceChanged: {

126 onFocusedSurfaceChanged: {

127+ if (focusedSurface) {

132− focusedSurface.focus = true;

128+ focusedSurface.focus = true;

129+ }

133 }

130 }

134 }

131 }

135

132

@@ -158,6 +155,7 @@ FocusScope {

158 objectName: "screenshotImage"

155 objectName: "screenshotImage"

159 anchors.fill: parent

156 anchors.fill: parent

160 antialiasing: !root.interactive

157 antialiasing: !root.interactive

158+ z: 1

161

159

162 function take() {

160 function take() {

163 // Save memory by using a half-resolution (thus quarter size) screenshot.

161 // Save memory by using a half-resolution (thus quarter size) screenshot.

@@ -175,6 +173,7 @@ FocusScope {

175 visible: active

173 visible: active

176 active: false

174 active: false

177 anchors.fill: parent

175 anchors.fill: parent

176+ z: screenshotImage.z + 1

178 sourceComponent: Component {

177 sourceComponent: Component {

179 Splash {

178 Splash {

180 id: splash

179 id: splash

@@ -191,6 +190,7 @@ FocusScope {

191

190

192 SurfaceContainer {

191 SurfaceContainer {

193 id: surfaceContainer

192 id: surfaceContainer

193+ z: splashLoader.z + 1

194 requestedWidth: root.requestedWidth

195 requestedHeight: root.requestedHeight

196 surfaceOrientationAngle: application && application.rotatesWindowContents ? root.surfaceOrientationAngle : 0

@@ -208,12 +208,28 @@ FocusScope {

208 }

209 }

210 delegate: SurfaceContainer {

211+ id: promptSurfaceContainer

211− interactive: index == (promptSurfacesRepeater.count - 1) && root.interactive

212+ interactive: index === 0 && root.interactive

212 surface: model.surface

213 surface: model.surface

213 width: root.width

214 width: root.width

214 height: root.height

215 height: root.height

215 isPromptSurface: true

216 isPromptSurface: true

217+ z: surfaceContainer.z + (promptSurfacesRepeater.count - index)

218+ property int index: model.index

219+ onIndexChanged: updateFirst()

220+ Component.onCompleted: updateFirst()

221+ function updateFirst() {

222+ if (index === 0) {

223+ promptSurfacesRepeater.first = promptSurfaceContainer;

224+ }

225+ }

226+ }

227+ onCountChanged: {

228+ if (count === 0) {

229+ first = null;

230+ }

216 }

231 }

232+ property Item first: null

217 }

233 }

218

234

219 // SurfaceContainer size drives ApplicationWindow size

235 // SurfaceContainer size drives ApplicationWindow size

tests/mocks/Unity/Application/MirSurfaceListModel.cpp+13−1

@@ -72,6 +72,18 @@ void MirSurfaceListModel::appendSurface(MirSurface *surface)

72 }

73}

75+void MirSurfaceListModel::prependSurface(MirSurface *surface)

76+{

77+ beginInsertRows(QModelIndex(), 0, 0);

78+ m_surfaceList.prepend(surface);

79+ connectSurface(surface);

80+ endInsertRows();

81+ Q_EMIT countChanged();

82+ if (m_surfaceList.count() == 1) {

83+ Q_EMIT firstChanged();

84+ }

85+}

86+

75void MirSurfaceListModel::connectSurface(MirSurface *surface)

87void MirSurfaceListModel::connectSurface(MirSurface *surface)

76{

88{

77 connect(surface, &MirSurface::raiseRequested, this, [this, surface](){ this->raise(surface); });

89 connect(surface, &MirSurface::raiseRequested, this, [this, surface](){ this->raise(surface); });

@@ -143,7 +155,7 @@ MirSurfaceInterface *MirSurfaceListModel::createSurface()

143 Mir::RestoredState,

155 Mir::RestoredState,

144 screenshotUrl);

156 screenshotUrl);

145

157

146− appendSurface(surface);

158+ prependSurface(surface);

147

159

148 return surface;

160 return surface;

149}

161}

tests/mocks/Unity/Application/MirSurfaceListModel.h+1

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

49 Q_INVOKABLE unity::shell::application::MirSurfaceInterface *createSurface();

51private:

52+ void prependSurface(MirSurface *surface);

52 void raise(MirSurface *surface);

53 void raise(MirSurface *surface);

53 void moveSurface(int from, int to);

54 void moveSurface(int from, int to);

54 void connectSurface(MirSurface *surface);

55 void connectSurface(MirSurface *surface);

tests/qmltests/Stages/ApplicationCheckBox.qml+1−1

@@ -156,7 +156,7 @@ Column {

156 width: height

157 height: promptsLabel.height * 0.7

158 anchors.verticalCenter: parent.verticalCenter

159− onClicked: d.application.promptSurfaceList.get(d.application.promptSurfaceList.count - 1).close()

159+ onClicked: d.application.promptSurfaceList.get(0).close()

160 enabled: d.application && d.application.promptSurfaceList.count > 0

161 Label { text: "➖"; anchors.centerIn: parent; enabled: parent.enabled }

162 }

tests/qmltests/Stages/tst_ApplicationWindow.qml+1−1

@@ -106,7 +106,7 @@ Rectangle {

106 enabled: parent.promptSurfaceList !== null && parent.promptSurfaceList.count > 0

107 activeFocusOnPress: false

108 text: "Remove"

109− onClicked: { parent.promptSurfaceList.get(parent.promptSurfaceList.count - 1).close(); }

109+ onClicked: { parent.promptSurfaceList.get(0).close(); }

110 }

111

112 Button {