Differential D12041 Diff 31652 desktoppackage/contents/configuration/ConfigurationContainmentAppearance.qml

Changeset View

Standalone View

View Options

desktoppackage/contents/configuration/ConfigurationContainmentAppearance.qml

Context not available.
21import QtQuick.Controls 1.0 as QtControls 21​import QtQuick.Controls 1.0 as QtControls
22import QtQuick.Layouts 1.1 22​import QtQuick.Layouts 1.1
23 23​
24​import org.kde.kconfig 1.0 // for KAuthorized
25​import org.kde.plasma.wallpapers.image 2.0 as Wallpaper
26​
24ColumnLayout { 27​ColumnLayout {
25 id: root 28​ id: root
26 29​
Context not available.
65 68​
66 Row { 69​ Row {
67 spacing: units.largeSpacing / 2 70​ spacing: units.largeSpacing / 2
68 anchors.right: wallpaperRow.right 71​ // anchors.right: wallpaperRow.right
ngrahamUnsubmitted
Not Done

Don't leave commented-out code; remove it

ngraham: Don't leave commented-out code; remove it
69 visible: pluginComboBox.count > 1 72​ visible: pluginComboBox.count > 1
70 Item { 73​ // Item {
71 width: units.largeSpacing 74​ // width: units.largeSpacing
72 height: parent.height 75​ // height: parent.height
73 } 76​ // }
ngrahamUnsubmitted
Not Done

Don't leave commented-out code; remove it

ngraham: Don't leave commented-out code; remove it
74 QtControls.Label { 77​ QtControls.Label {
78​ width: formAlignment - units.largeSpacing
75 anchors.verticalCenter: pluginComboBox.verticalCenter 79​ anchors.verticalCenter: pluginComboBox.verticalCenter
76 text: i18nd("plasma_shell_org.kde.plasma.desktop", "Layout:") 80​ text: i18nd("plasma_shell_org.kde.plasma.desktop", "Layout:")
81​ horizontalAlignment: Text.AlignRight
77 } 82​ }
78 QtControls.ComboBox { 83​ QtControls.ComboBox {
79 id: pluginComboBox 84​ id: pluginComboBox
Context not available.
149 root.configurationChanged() 154​ root.configurationChanged()
150 } 155​ }
151 } 156​ }
157​ QtControls.Button {
158​ iconName: "get-hot-new-stuff"
159​ text: i18nd("plasma_shell_org.kde.plasma.desktop", "Get Wallpaper Plugins...")
ngrahamUnsubmitted
Not Done

It might be a bit wordy, but to be consistent with the form used everywhere else, this should say, "Get new Wallpaper Plugins..."

ngraham: It might be a bit wordy, but to be consistent with the form used everywhere else, this should…
ZrenAuthorUnsubmitted
Not Done

"Get New Wallpaper" is capitalized. Are we using lowercase so the "Wallpaper Plugins" stands out more?

lowercase new vs uppercase New

Zren: "Get New Wallpaper" is capitalized. Are we using lowercase so the "Wallpaper Plugins" stands…
ngrahamUnsubmitted
Not Done

Sorry for the confusion, I was more commenting on the fact that the word "New" would be included. Following the pattern is a good idea, though I would favor lowercasing the word "new" everywhere (would be in another patch though).

ngraham: Sorry for the confusion, I was more commenting on the fact that the word "New" would be…
160​ visible: KAuthorized.authorize("ghns")
161​ onClicked: imageWallpaper.getNewWallpaperPlugin(this)
162​
163​ Wallpaper.Image {
164​ id: imageWallpaper
165​ }
166​ }
152 } 167​ }
153 168​
154 Item { 169​ Item {
Context not available.