Commit 1658a360 authored by Nate Graham's avatar Nate Graham 🔩

Use view background color for sidebar to match other sidebars

Summary:
Generally, left sidebars in Kirigami-style apps (e.g. System Settings) use the view
background color. Let's do the same in Discover when it's displaying the Desktop view.

Need to bump the Kirigami version to have access to `Theme.colorSet`

Test Plan:
Sidebar is now a pleasing off-white, same as System Settings
{F6824729}

Reviewers: #vdg, #discover_software_store, apol

Reviewed By: #discover_software_store, apol

Subscribers: mart, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D21229
parent 0b4837cb
......@@ -23,7 +23,7 @@ import QtQuick.Layouts 1.1
import QtQuick.Controls 2.1
import org.kde.discover 2.0
import org.kde.discover.app 1.0
import org.kde.kirigami 2.0 as Kirigami
import org.kde.kirigami 2.8 as Kirigami
import "navigation.js" as Navigation
Kirigami.GlobalDrawer {
......@@ -38,6 +38,9 @@ Kirigami.GlobalDrawer {
topPadding: drawer.wideScreen ? -toploader.height - leftPadding : 0
bottomPadding: 0
// In desktop view, it's a sidebar, and sidebars get the view BG color
Kirigami.Theme.colorSet: modal ? Kirigami.Theme.Window : Kirigami.Theme.View
resetMenuOnTriggered: false
onBannerClicked: {
......
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