Differential D12102 Diff 33150 kcms/translations/package/contents/ui/main.qml

Changeset View

Standalone View

View Options

kcms/translations/package/contents/ui/main.qml

  • This file was added.
1​/*
2​ * Copyright (C) 2015 Marco Martin <mart@kde.org>
3​ * Copyright (C) 2018 Eike Hein <hein@kde.org>
4​ *
5​ * This library is free software; you can redistribute it and/or
6​ * modify it under the terms of the GNU Library General Public
7​ * License as published by the Free Software Foundation; either
8​ * version 2 of the License, or (at your option) any later version.
9​ *
10​ * This library is distributed in the hope that it will be useful,
11​ * but WITHOUT ANY WARRANTY; without even the implied warranty of
12​ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
13​ * Library General Public License for more details.
14​ *
15​ * You should have received a copy of the GNU Library General Public License
16​ * along with this library; see the file COPYING.LIB. If not, write to
17​ * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
18​ * Boston, MA 02110-1301, USA.
19​*/
20​
21​import QtQuick 2.1
22​import QtQuick.Layouts 1.1
23​import QtQuick.Controls 2.3 as QtControls
24​import org.kde.kirigami 2.4 as Kirigami
25​import org.kde.plasma.core 2.1 as PlasmaCore
26​import org.kde.kcm 1.2
27​
28​ScrollViewKCM {
29​ id: root
30​
31​ ConfigModule.quickHelp: i18n("Language")
32​
33​ PlasmaCore.SortFilterModel {
34​ id: availableLanguagesModel
35​
36​ sourceModel: kcm.translationsModel
37​
38​ filterRole: "IsSelected"
39​ filterCallback: function(source_row, value) { return !value; }
40​
41​ sortRole: "display"
42​ }
43​
44​ Kirigami.OverlaySheet {
45​ id: addLanguagesSheet
46​
47​ parent: root.parent
48​
49​ topPadding: 0
50​ leftPadding: 0
51​ rightPadding: 0
52​ bottomPadding: 0
53​
54​ header: Kirigami.Heading { text: i18nc("@title:window", "Add Languages") }
55​
56​ property var selectedLanguages: []
57​
58​ onSheetOpenChanged: selectedLanguages = []
59​
60​ ListView {
aspotashevUnsubmitted
Not Done

Please add context:

i18nc("@title:window", "Add Languages")
aspotashev: Please add context: ``` i18nc("@title:window", "Add Languages") ```
61​ implicitWidth: 18 * Kirigami.Units.gridUnit
62​
63​ model: availableLanguagesModel
64​
65​ delegate: Kirigami.BasicListItem {
66​ Layout.fillWidth: true
67​
martUnsubmitted
Not Done

same consideration for Layout.* in delegates

mart: same consideration for Layout.* in delegates
martUnsubmitted
Not Done

sorry, counter what i told you before, setting an explicit width: in lists in overlaysheets actually breaks them, https://imgur.com/a/CPMhtYX
this line should be removed

mart: sorry, counter what i told you before, setting an explicit width: in lists in overlaysheets…
68​ property string languageCode: model.LanguageCode
69​
70​ reserveSpaceForIcon: false
71​
72​ label: model.display
73​
74​ checkable: true
75​ onCheckedChanged: {
76​ if (checked) {
77​ addLanguagesSheet.selectedLanguages.push(index);
78​
79​ // There's no property change notification for pushing to an array
80​ // in a var prop, so we can't bind selectedLanguages.length to
81​ // addLanguagesButton.enabled.
82​ addLanguagesButton.enabled = true;
83​ } else {
84​ addLanguagesSheet.selectedLanguages = addLanguagesSheet.selectedLanguages.filter(function(item) { return item !== index });
85​
86​ // There's no property change notification for pushing to an array
87​ // in a var prop, so we can't bind selectedLanguages.length to
88​ // addLanguagesButton.enabled.
89​ if (!addLanguagesSheet.selectedLanguages.length) {
90​ addLanguagesButton.enabled = false;
91​ }
92​ }
93​ }
94​ }
95​ }
96​
97​ footer: RowLayout {
98​ QtControls.Button {
99​ id: addLanguagesButton
100​
101​ Layout.alignment: Qt.AlignHCenter
102​
103​ text: i18nc("@action:button", "Add")
104​
105​ enabled: false
106​
107​ onClicked: {
108​ var langs = kcm.translationsModel.selectedLanguages;
109​ addLanguagesSheet.selectedLanguages.sort().forEach(function(index) {
aspotashevUnsubmitted
Not Done

Please add context for translators:

i18nc("@action:button", "Add")
aspotashev: Please add context for translators: ``` i18nc("@action:button", "Add") ```
110​ langs.push(availableLanguagesModel.get(index).LanguageCode);
111​ });
112​
113​ kcm.translationsModel.selectedLanguages = langs;
114​
115​ addLanguagesSheet.sheetOpen = false;
116​ }
117​ }
118​ }
119​ }
120​
121​ header: ColumnLayout {
122​ id: messagesLayout
123​
124​ spacing: Kirigami.Units.largeSpacing
125​
126​ Kirigami.InlineMessage {
127​ Layout.fillWidth: true
128​
129​ type: Kirigami.MessageType.Information
130​
131​ text: i18nc("@info", "There are no languages available on this system.")
132​
133​ visible: !availableLanguagesModel.count
134​ }
135​
136​ Kirigami.InlineMessage {
137​ Layout.fillWidth: true
138​
139​ type: kcm.everSaved ? Kirigami.MessageType.Positive : Kirigami.MessageType.Information
140​
141​ text: (kcm.everSaved ? i18nc("@info", "Your changes will take effect the next time you log in.")
142​ : i18nc("@info", "There are currently no preferred languages configured."))
143​
144​ visible: !languagesList.count || kcm.everSaved
145​ }
146​
aspotashevUnsubmitted
Not Done

Please add context:

i18nc("@info", "There are no languages available on this system.")
aspotashev: Please add context: ``` i18nc("@info", "There are no languages available on this system.") ```
147​ Kirigami.InlineMessage {
148​ Layout.fillWidth: true
149​
150​ type: Kirigami.MessageType.Error
151​
152​ text: i18ncp("@info %2 is the language code",
153​ "The translation files for the language with the code '%2' could not be found. The language will be removed from your configuration. If you want to add it back, please install the localization files for it and add the language again.",
154​ "The translation files for the languages with the codes '%2' could not be found. These languages will be removed from your configuration. If you want to add them back, please install the localization files for it and the languages again.",
155​ kcm.translationsModel.missingLanguages.length,
156​ kcm.translationsModel.missingLanguages.join("', '"))
157​
158​ visible: kcm.translationsModel.missingLanguages.length
aspotashevUnsubmitted
Not Done

Please add context "@info"

aspotashev: Please add context "@info"
159​ }
160​
161​ QtControls.Label {
162​ Layout.fillWidth: true
163​
164​ visible: languagesList.count
165​
166​ text: i18n("The language at the top of this list is is the one you want to see and use most often.")
167​ }
168​ }
169​
170​ view: ListView {
171​ id: languagesList
aspotashevUnsubmitted
Not Done

Please add context "@info":

i18ncp("@info %2 is the language code",
aspotashev: Please add context "@info": ``` i18ncp("@info %2 is the language code", ```
172​
173​ model: PlasmaCore.SortFilterModel {
174​ sourceModel: kcm.translationsModel
175​
176​ filterRole: "IsSelected"
177​ filterCallback: function(source_row, value) { return value; }
178​
179​ sortRole: "SelectedPriority"
180​ }
181​
182​ delegate: Kirigami.SwipeListItem {
183​ id: listItem
184​
185​ Layout.fillWidth: true
186​ Layout.margins: 0
martUnsubmitted
Not Done

Layout.* attached properties are useless there, as items in a listview are not in a layout.
the correct thing to do here (as documented by qt) is even is not pretty:

width: languagesList.width

mart: Layout.* attached properties are useless there, as items in a listview are not in a layout. the…
187​
188​ contentItem: RowLayout {
189​ width: implicitWidth
190​ height: Math.max(implicitHeight, Kirigami.Units.iconSizes.smallMedium)
191​
192​ anchors.verticalCenter: parent.verticalCenter
193​
194​ Kirigami.Icon {
195​ visible: model.IsMissing
196​
197​ Layout.alignment: Qt.AlignVCenter
198​
199​ width: Kirigami.Units.iconSizes.smallMedium
200​ height: width
201​
202​ source: "error"
203​ color: Kirigami.Theme.negativeTextColor
204​ }
205​
206​ QtControls.Label {
207​ Layout.fillWidth: true
208​
209​ Layout.alignment: Qt.AlignVCenter
210​
211​ text: (index == 0) ? i18nc("@item:inlistbox 1 = Language name", "%1 (Default)", model.display) : model.display
212​
213​ color: (model.IsMissing ? Kirigami.Theme.negativeTextColor
214​ : (listItem.checked || (listItem.pressed && !listItem.checked && !listItem.sectionDelegate)
215​ ? listItem.activeTextColor : listItem.textColor))
216​ }
217​ }
218​
219​ actions: [
220​ Kirigami.Action {
221​ enabled: !model.IsMissing && index > 0
222​ iconName: "go-top"
223​ tooltip: i18nc("@info:tooltip", "Make default")
224​ onTriggered: kcm.translationsModel.selectedLanguages.splice(0, 0, kcm.translationsModel.selectedLanguages.splice(index, 1)[0])
225​ },
226​ Kirigami.Action {
227​ enabled: !model.IsMissing
228​ iconName: "list-remove"
229​ tooltip: i18nc("@info:tooltip", "Remove")
230​ onTriggered: kcm.translationsModel.selectedLanguages = kcm.translationsModel.selectedLanguages.filter(function(item) { return item !== model.LanguageCode })
231​ }]
232​ }
233​ }
234​
235​ footer: RowLayout {
236​ id: footerLayout
237​
238​ QtControls.Button {
239​ Layout.alignment: Qt.AlignHCenter
240​
241​ enabled: availableLanguagesModel.count
242​
243​ text: i18nc("@action:button", "Add languages...")
244​
245​ onClicked: addLanguagesSheet.sheetOpen = !addLanguagesSheet.sheetOpen
246​
247​ checkable: true
248​ checked: addLanguagesSheet.sheetOpen
249​ }
250​ }
251​}
252​
aspotashevUnsubmitted
Not Done

Please add context "@action:button"

aspotashev: Please add context "@action:button"