Commit dfce5003 authored by Krešimir Čohar's avatar Krešimir Čohar Committed by Nate Graham

[Kickoff] Make Kickoff separators leaner (less opaque) and narrower (the...

[Kickoff] Make Kickoff separators leaner (less opaque) and narrower (the length of a highlighted menu item)

Summary: Leaner separators in Kickoff (less opaque/more see-through, narrower/shorter)

Test Plan:
This patch would spruce up the much-needed separators in Kickoff that were discussed in D15011 and ultimately added in D15206. It would reduce the opacity of the separators by almost half and make the top separator (between the search field and the rest of the Kickoff menu) as long/wide as a highlighted menu item. This variant was actually put forward in D15011.

>>! In D15011#314514, @ngraham wrote:
> Perhaps the white background and blinking cursor appear too attention-getting in comparison to the rest of Kickoff, which tries very hard to be flat and gray and avoid separators (e.g. between the different sections and the tab bar at the bottom or the header on top). I wonder if we wouldn't feel like the search field was so jarring if there were better differentiation between other elements. For example:
>
> {F6212312}
>

See the difference in the screenshots below:

{F6427577}
{F6427578}

Reviewers: ngraham, #vdg, romangg

Reviewed By: ngraham, #vdg, romangg

Subscribers: romangg, filipf, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D16937
parent 7caafc8f
......@@ -121,8 +121,8 @@ Item {
height: Math.floor(units.devicePixelRatio)
color: Qt.tint(PlasmaCore.ColorScope.textColor, Qt.rgba(PlasmaCore.ColorScope.backgroundColor.r, PlasmaCore.ColorScope.backgroundColor.g, PlasmaCore.ColorScope.backgroundColor.b, 0.7))
width: root.width
opacity: 0.6
width: root.width - 2 * units.gridUnit
anchors {
top: header.top
......@@ -480,6 +480,7 @@ Item {
width: root.width
height: Math.floor(units.devicePixelRatio)
color: Qt.tint(PlasmaCore.ColorScope.textColor, Qt.rgba(PlasmaCore.ColorScope.backgroundColor.r, PlasmaCore.ColorScope.backgroundColor.g, PlasmaCore.ColorScope.backgroundColor.b, 0.7))
opacity: 0.6
anchors {
top: header.top
......
......@@ -54,19 +54,20 @@ FocusScope {
listView.decrementCurrentIndex();
}
PlasmaExtras.ScrollArea {
id: scrollArea
frameVisible: false
anchors.fill: parent
ListView {
id: listView
focus: true
keyNavigationWraps: true
boundsBehavior: Flickable.StopAtBounds
highlight: KickoffHighlight {}
highlightMoveDuration : 0
highlightResizeDuration: 0
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment