git.iko.soy/unity8
on xenial help sign in
fix issues from review
fb4af0bMichael Zanetti
qml/Launcher/LauncherPanel.qml+6−1
compare with xenial
unifiedsplit
@@ -483,7 +483,7 @@ Rectangle {
483 id: quickListShape
483 id: quickListShape
484 objectName: "quickListShape"
484 objectName: "quickListShape"
485 anchors.fill: quickList
485 anchors.fill: quickList
486− opacity: quickList.state === "open" ? 0.95 : 0
486+ opacity: quickList.state === "open" ? 0.96 : 0
487 visible: opacity > 0
487 visible: opacity > 0
488 rotation: root.rotation
488 rotation: root.rotation
489
489
@@ -555,6 +555,11 @@ Rectangle {
555 text: (model.clickable ? "" : "<b>") + model.label + (model.clickable ? "" : "</b>")
555 text: (model.clickable ? "" : "<b>") + model.label + (model.clickable ? "" : "</b>")
556 highlightWhenPressed: model.clickable
556 highlightWhenPressed: model.clickable
557
557
558+ // FIXME: This is a workaround for the theme not being context sensitive. I.e. the
559+ // ListItems don't know that they are sitting in a themed Popover where the color
560+ // needs to be inverted.
561+ __foregroundColor: Theme.palette.normal.highlightText
562+
558 onClicked: {
563 onClicked: {
559 if (!model.clickable) {
564 if (!model.clickable) {
560 return;
565 return;
tests/qmltests/Launcher/tst_Launcher.qml+9−9
@@ -402,9 +402,9 @@ Item {
402
402
403 function test_quicklist_positioning(data) {
403 function test_quicklist_positioning(data) {
404 revealer.dragLauncherIntoView();
404 revealer.dragLauncherIntoView();
405− var quickList = findChild(launcher, "quickList")
405+ var quickList = findChild(launcher, "quickList");
406− var draggedItem = findChild(launcher, "launcherDelegate" + data.itemIndex)
406+ var draggedItem = findChild(launcher, "launcherDelegate" + data.itemIndex);
407− var quickListShape = findChild(launcher, "quickListShape")
407+ var quickListShape = findChild(launcher, "quickListShape");
408
408
409 // Position launcher to where we need it
409 // Position launcher to where we need it
410 var listView = findChild(launcher, "launcherListView");
410 var listView = findChild(launcher, "launcherListView");
@@ -415,20 +415,20 @@ Item {
415 }
415 }
416
416
417 // Doing longpress
417 // Doing longpress
418− mousePress(draggedItem, draggedItem.width / 2, draggedItem.height / 2)
418+ mousePress(draggedItem, draggedItem.width / 2, draggedItem.height / 2);
419− tryCompare(quickListShape, "opacity", 0.95)
419+ tryCompare(quickListShape, "opacity", 0.96);
420 mouseRelease(draggedItem);
420 mouseRelease(draggedItem);
421
421
422− verify(quickList.y >= units.gu(1))
422+ verify(quickList.y >= units.gu(1));
423− verify(quickList.y + quickList.height + units.gu(1) <= launcher.height)
423+ verify(quickList.y + quickList.height + units.gu(1) <= launcher.height);
424
424
425 // Click somewhere in the empty space to dismiss the quicklist
425 // Click somewhere in the empty space to dismiss the quicklist
426 mouseClick(launcher, launcher.width - units.gu(1), units.gu(1));
426 mouseClick(launcher, launcher.width - units.gu(1), units.gu(1));
427− tryCompare(quickListShape, "visible", false)
427+ tryCompare(quickListShape, "visible", false);
428
428
429 // Click somewhere in the empty space to dismiss the launcher
429 // Click somewhere in the empty space to dismiss the launcher
430 mouseClick(launcher, launcher.width - units.gu(1), units.gu(1));
430 mouseClick(launcher, launcher.width - units.gu(1), units.gu(1));
431− revealer.waitUntilLauncherDisappears()
431+ revealer.waitUntilLauncherDisappears();
432 }
432 }
433
433
434 function test_quicklist_click_data() {
434 function test_quicklist_click_data() {