Differential D28152 Diff 78973 kcmkwin/kwinrules/rulesmodel.cpp

Changeset View

Standalone View

View Options

kcmkwin/kwinrules/rulesmodel.cpp

  • This file was added.
1​/*
2​ * Copyright (c) 2004 Lubos Lunak <l.lunak@kde.org>
3​ * Copyright (c) 2020 Ismael Asensio <isma.af@gmail.com>
4​ *
5​ * This program is free software; you can redistribute it and/or
6​ * modify it under the terms of the GNU General Public License as
7​ * published by the Free Software Foundation; either version 2 of
8​ * the License or (at your option) version 3 or any later version
9​ * accepted by the membership of KDE e.V. (or its successor approved
10​ * by the membership of KDE e.V.), which shall act as a proxy
11​ * defined in Section 14 of version 3 of the license.
12​ *
13​ * This program is distributed in the hope that it will be useful,
14​ * but WITHOUT ANY WARRANTY; without even the implied warranty of
15​ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
16​ * GNU General Public License for more details.
17​ *
18​ * You should have received a copy of the GNU General Public License
19​ * along with this program. If not, see <http://www.gnu.org/licenses/>.
20​ */
21​
22​#include "rulesmodel.h"
23​#include <rules.h>
24​
25​#include <QFileInfo>
26​#include <QIcon>
27​#include <QQmlEngine>
28​#include <QTemporaryFile>
29​#include <QtDBus>
30​
31​#include <KColorSchemeManager>
32​#include <KConfig>
33​#include <KLocalizedString>
34​#include <KWindowSystem>
35​
36​
37​namespace KWin
38​{
39​
40​RulesModel::RulesModel(QObject *parent)
41​ : QAbstractListModel(parent)
42​{
43​ qmlRegisterUncreatableType<RuleItem>("org.kde.kcms.kwinrules", 1, 0, "RuleItem",
44​ QStringLiteral("Do not create objects of type RuleItem"));
broulikUnsubmitted
Not Done

I think this should be registered in the KCM constructor, not inside of another type

broulik: I think this should be registered in the KCM constructor, not inside of another type
iasensioAuthorUnsubmitted
Not Done

It's related to the 2nd entry point (kwin_rules_dialog). Since it does not include the kcm, but just the model, this is the most outer object common to both paths.

iasensio: It's related to the 2nd entry point (`kwin_rules_dialog`). Since it does not include the kcm…
45​ qmlRegisterUncreatableType<RulesModel>("org.kde.kcms.kwinrules", 1, 0, "RulesModel",
46​ QStringLiteral("Do not create objects of type RulesModel"));
47​
48​ populateRuleList();
49​}
50​
51​RulesModel::~RulesModel()
52​{
53​}
54​
55​QHash< int, QByteArray > RulesModel::roleNames() const
56​{
57​ return {
58​ {KeyRole, QByteArrayLiteral("key")},
broulikUnsubmitted
Done

No need to delete those

broulik: No need to delete those
59​ {NameRole, QByteArrayLiteral("name")},
60​ {IconRole, QByteArrayLiteral("icon")},
61​ {IconNameRole, QByteArrayLiteral("iconName")},
62​ {SectionRole, QByteArrayLiteral("section")},
63​ {DescriptionRole, QByteArrayLiteral("description")},
64​ {EnabledRole, QByteArrayLiteral("enabled")},
65​ {SelectableRole, QByteArrayLiteral("selectable")},
66​ {ValueRole, QByteArrayLiteral("value")},
67​ {TypeRole, QByteArrayLiteral("type")},
68​ {PolicyRole, QByteArrayLiteral("policy")},
69​ {PolicyModelRole, QByteArrayLiteral("policyModel")},
70​ {OptionsModelRole, QByteArrayLiteral("options")},
71​ };
72​}
73​
74​int RulesModel::rowCount(const QModelIndex &parent) const
75​{
76​ if (parent.isValid()) {
77​ return 0;
78​ }
79​ return m_ruleList.size();
80​}
81​
82​QVariant RulesModel::data(const QModelIndex &index, int role) const
83​{
84​ if (!checkIndex(index, CheckIndexOption::IndexIsValid | CheckIndexOption::ParentIsInvalid)) {
85​ return QVariant();
86​ }
87​
88​ const RuleItem *rule = m_ruleList.at(index.row());
89​
90​ switch (role) {
91​ case KeyRole:
92​ return rule->key();
93​ case NameRole:
94​ return rule->name();
95​ case IconRole:
96​ return rule->icon();
97​ case IconNameRole:
98​ return rule->iconName();
99​ case DescriptionRole:
100​ return rule->description();
101​ case SectionRole:
102​ return rule->section();
103​ case EnabledRole:
104​ return rule->isEnabled();
105​ case SelectableRole:
106​ return !rule->hasFlag(RuleItem::AlwaysEnabled);
107​ case ValueRole:
108​ return rule->value();
109​ case TypeRole:
110​ return rule->type();
111​ case PolicyRole:
112​ return rule->policy();
113​ case PolicyModelRole:
114​ return rule->policyModel();
115​ case OptionsModelRole:
116​ return rule->options();
117​ }
118​ return QVariant();
119​}
120​
121​bool RulesModel::setData(const QModelIndex &index, const QVariant &value, int role)
122​{
123​ if (!checkIndex(index, CheckIndexOption::IndexIsValid | CheckIndexOption::ParentIsInvalid)) {
124​ return false;
125​ }
126​
127​ RuleItem *rule = m_ruleList.at(index.row());
128​
129​ switch (role) {
130​ case EnabledRole:
131​ if (value.toBool() == rule->isEnabled()) {
132​ return true;
133​ }
134​ rule->setEnabled(value.toBool());
135​ break;
136​ case ValueRole:
137​ if (value == rule->value()) {
138​ return true;
139​ }
140​ rule->setValue(value);
141​ break;
142​ case PolicyRole:
143​ if (value.toInt() == rule->policy()) {
144​ return true;
145​ }
146​ rule->setPolicy(value.toInt());
147​ break;
148​ default:
149​ return false;
150​ }
broulikUnsubmitted
Done

Check if you changed the value before emitting this for optimization

broulik: Check if you changed the value before emitting this for optimization
151​
152​ emit dataChanged(index, index, QVector<int>{role});
153​
154​ if (rule->hasFlag(RuleItem::AffectsDescription)) {
155​ emit descriptionChanged();
156​ }
157​ if (rule->hasFlag(RuleItem::AffectsWarning)) {
158​ emit warningMessageChanged();
159​ }
160​
161​ return true;
162​}
163​
164​RuleItem *RulesModel::addRule(RuleItem *rule)
165​{
166​ m_ruleList << rule;
167​ m_rules.insert(rule->key(), rule);
168​
169​ return rule;
170​}
171​
172​bool RulesModel::hasRule(const QString& key) const
173​{
174​ return m_rules.contains(key);
175​}
176​
177​
178​RuleItem *RulesModel::ruleItem(const QString& key) const
broulikUnsubmitted
Done

Avoid operator[] which may detach and insert new items into the list unexpectedly. (It can't here because this method is const but generally we try to avoid it)

broulik: Avoid `operator[]` which may detach and insert new items into the list unexpectedly. (It can't…
179​{
180​ return m_rules.value(key);
181​}
182​
183​QString RulesModel::description() const
184​{
185​ const QString desc = m_rules["description"]->value().toString();
186​ if (!desc.isEmpty()) {
187​ return desc;
188​ }
189​ return defaultDescription();
190​}
191​
192​void RulesModel::setDescription(const QString &description)
193​{
194​ setData(index(0, 0), description, RulesModel::ValueRole);
195​}
196​
197​QString RulesModel::defaultDescription() const
198​{
199​ const QString wmclass = m_rules["wmclass"]->value().toString();
200​ const QString title = m_rules["title"]->isEnabled() ? m_rules["title"]->value().toString() : QString();
201​
202​ if (!title.isEmpty()) {
203​ return i18n("Window settings for %1", title);
204​ }
205​ if (!wmclass.isEmpty()) {
206​ return i18n("Settings for %1", wmclass);
207​ }
208​
209​ return i18n("New window settings");
210​}
211​
212​QString RulesModel::warningMessage() const
213​{
214​ if (wmclassWarning()) {
215​ return i18n("You have specified the window class as unimportant.\n"
216​ "This means the settings will possibly apply to windows from all applications."
ngrahamUnsubmitted
Done

Maybe "Exact Match" should be the default settting so this message isn't always shown for every new rule you add

ngraham: Maybe "Exact Match" should be the default settting so this message isn't always shown for every…
iasensioAuthorUnsubmitted
Not Done

Agree. I have set it in a simple way for new rules added.

I would like though, to set this directly on the config schema defaults, but for that to work I'll need to modify some bits of the KConfigXT helper classes, which I have planned for a later patch along some other changes to improve integration.

iasensio: Agree. I have set it in a simple way for new rules added. I would like though, to set this…
217​ " If you really want to create a generic setting, it is recommended"
218​ " you at least limit the window types to avoid special window types.");
219​ }
220​
221​ return QString();
222​}
223​
224​
225​bool RulesModel::wmclassWarning() const
226​{
227​ const bool no_wmclass = !m_rules["wmclass"]->isEnabled()
228​ || m_rules["wmclass"]->policy() == Rules::UnimportantMatch;
229​ const bool alltypes = !m_rules["types"]->isEnabled()
230​ || (m_rules["types"]->value() == 0)
231​ || (m_rules["types"]->value() == NET::AllTypesMask)
232​ || ((m_rules["types"]->value().toInt() | (1 << NET::Override)) == 0x3FF);
233​
234​ return (no_wmclass && alltypes);
235​}
236​
237​
238​void RulesModel::readFromSettings(RuleSettings *settings)
239​{
240​ beginResetModel();
241​
242​ for (RuleItem *rule : qAsConst(m_ruleList)) {
243​ const KConfigSkeletonItem *configItem = settings->findItem(rule->key());
244​ const KConfigSkeletonItem *configPolicyItem = settings->findItem(rule->policyKey());
245​
246​ if (!configItem
247​ || (configPolicyItem && configPolicyItem->property() == Rules::Unused)
248​ || (configItem->property().toString().isEmpty())) {
249​ rule->reset();
250​ continue;
251​ }
252​
253​ rule->setEnabled(true);
254​
255​ const QVariant value = configItem->property();
256​ rule->setValue(value);
257​
258​ if (rule->policyType() != RulePolicy::NoPolicy) {
259​ const int policy = configPolicyItem->property().toInt();
260​ rule->setPolicy(policy);
261​ }
262​ }
263​
264​ endResetModel();
265​
266​ emit descriptionChanged();
267​ emit warningMessageChanged();
268​}
269​
270​void RulesModel::writeToSettings(RuleSettings *settings) const
271​{
272​ const QString description = m_rules["description"]->value().toString();
273​ if (description.isEmpty()) {
274​ m_rules["description"]->setValue(defaultDescription());
275​ }
276​
277​ for (const RuleItem *rule : qAsConst(m_ruleList)) {
278​ KConfigSkeletonItem *configItem = settings->findItem(rule->key());
279​ KConfigSkeletonItem *configPolicyItem = settings->findItem(rule->policyKey());
280​
281​ Q_ASSERT (configItem);
282​
283​ if (rule->isEnabled()) {
284​ configItem->setProperty(rule->value());
285​ if (configPolicyItem) {
286​ configPolicyItem->setProperty(rule->policy());
287​ }
288​ } else {
289​ if (configPolicyItem) {
290​ configPolicyItem->setProperty(Rules::Unused);
291​ } else {
292​ // Rules without policy gets deactivated by an empty string
293​ configItem->setProperty(QString());
294​ }
295​ }
296​ }
297​}
298​
299​void RulesModel::importFromRules(Rules* rules)
300​{
301​ QTemporaryFile tempFile;
302​ if (!tempFile.open()) {
303​ return;
304​ }
305​ const auto cfg = KSharedConfig::openConfig(tempFile.fileName(), KConfig::SimpleConfig);
306​ RuleSettings *settings = new RuleSettings(cfg, QStringLiteral("tempSettings"));
307​
308​ settings->setDefaults();
309​ if (rules) {
310​ rules->write(settings);
311​ }
312​ readFromSettings(settings);
313​
314​ delete(settings);
315​}
316​
317​Rules *RulesModel::exportToRules() const
broulikUnsubmitted
Done

qDeleteAll(m_ruleList) before so you don't leak those rules?

broulik: `qDeleteAll(m_ruleList)` before so you don't leak those rules?
318​{
319​ QTemporaryFile tempFile;
320​ if (!tempFile.open()) {
321​ return nullptr;
322​ }
323​ const auto cfg = KSharedConfig::openConfig(tempFile.fileName(), KConfig::SimpleConfig);
324​
broulikUnsubmitted
Done

Just use the return value of addRule instead of looking it up in the dict again

broulik: Just use the return value of `addRule` instead of looking it up in the dict again
325​ RuleSettings *settings = new RuleSettings(cfg, QStringLiteral("tempSettings"));
326​
327​ writeToSettings(settings);
328​ Rules *rules = new Rules(settings);
329​
330​ delete(settings);
331​ return rules;
332​}
333​
334​
335​void RulesModel::populateRuleList()
336​{
337​ qDeleteAll(m_ruleList);
338​ m_ruleList.clear();
339​
340​ //Rule description
341​ auto description = addRule(new RuleItem(QLatin1String("description"),
342​ RulePolicy::NoPolicy, RuleItem::String,
343​ i18n("Description"), i18n("Window matching"),
344​ QIcon::fromTheme("entry-edit")));
345​ description->setFlag(RuleItem::AlwaysEnabled);
346​ description->setFlag(RuleItem::AffectsDescription);
347​
348​ // Window matching
349​ auto wmclass = addRule(new RuleItem(QLatin1String("wmclass"),
350​ RulePolicy::StringMatch, RuleItem::String,
351​ i18n("Window class (application)"), i18n("Window matching"),
352​ QIcon::fromTheme("window")));
353​ wmclass->setFlag(RuleItem::AlwaysEnabled);
354​ wmclass->setFlag(RuleItem::AffectsDescription);
355​ wmclass->setFlag(RuleItem::AffectsWarning);
356​
357​ auto wmclasscomplete = addRule(new RuleItem(QLatin1String("wmclasscomplete"),
358​ RulePolicy::NoPolicy, RuleItem::Boolean,
359​ i18n("Match whole window class"), i18n("Window matching"),
360​ QIcon::fromTheme("window")));
361​ wmclasscomplete->setFlag(RuleItem::AlwaysEnabled);
362​
363​ auto types = addRule(new RuleItem(QLatin1String("types"),
364​ RulePolicy::NoPolicy, RuleItem::FlagsOption,
365​ i18n("Window types"), i18n("Window matching"),
366​ QIcon::fromTheme("window-duplicate")));
367​ types->setOptionsData(windowTypesModelData());
368​ types->setFlag(RuleItem::AlwaysEnabled);
369​ types->setFlag(RuleItem::AffectsWarning);
370​
371​ addRule(new RuleItem(QLatin1String("windowrole"),
372​ RulePolicy::NoPolicy, RuleItem::String,
373​ i18n("Window role"), i18n("Window matching"),
374​ QIcon::fromTheme("dialog-object-properties")));
375​
376​ auto title = addRule(new RuleItem(QLatin1String("title"),
377​ RulePolicy::StringMatch, RuleItem::String,
378​ i18n("Window title"), i18n("Window matching"),
379​ QIcon::fromTheme("edit-comment")));
380​ title->setFlag(RuleItem::AffectsDescription);
381​
382​ addRule(new RuleItem(QLatin1String("clientmachine"),
383​ RulePolicy::StringMatch, RuleItem::String,
384​ i18n("Machine (hostname)"), i18n("Window matching"),
385​ QIcon::fromTheme("computer")));
386​
387​ // Size & Position
388​ addRule(new RuleItem(QLatin1String("position"),
389​ RulePolicy::SetRule, RuleItem::Coordinate,
390​ i18n("Position"), i18n("Size & Position"),
391​ QIcon::fromTheme("transform-move")));
392​
393​ addRule(new RuleItem(QLatin1String("size"),
394​ RulePolicy::SetRule, RuleItem::Coordinate,
395​ i18n("Size"), i18n("Size & Position"),
396​ QIcon::fromTheme("image-resize-symbolic")));
397​
398​ addRule(new RuleItem(QLatin1String("maximizehoriz"),
399​ RulePolicy::SetRule, RuleItem::Boolean,
400​ i18n("Maximized horizontally"), i18n("Size & Position"),
401​ QIcon::fromTheme("resizecol")));
402​
403​ addRule(new RuleItem(QLatin1String("maximizevert"),
404​ RulePolicy::SetRule, RuleItem::Boolean,
405​ i18n("Maximized vertically"), i18n("Size & Position"),
406​ QIcon::fromTheme("resizerow")));
407​
408​ auto desktop = addRule(new RuleItem(QLatin1String("desktop"),
409​ RulePolicy::SetRule, RuleItem::Option,
410​ i18n("Virtual Desktop"), i18n("Size & Position"),
411​ QIcon::fromTheme("virtual-desktops")));
412​ desktop->setOptionsData(virtualDesktopsModelData());
413​
414​#ifdef KWIN_BUILD_ACTIVITIES
415​ m_activities = new KActivities::Consumer(this);
416​
417​ auto activity = addRule(new RuleItem(QLatin1String("activity"),
418​ RulePolicy::SetRule, RuleItem::Option,
419​ i18n("Activity"), i18n("Size & Position"),
420​ QIcon::fromTheme("activities")));
421​ activity->setOptionsData(activitiesModelData());
422​
423​ // Activites consumer may update the available activities later
424​ connect(m_activities, &KActivities::Consumer::activitiesChanged,
425​ this, [this] { m_rules["activity"]->setOptionsData(activitiesModelData()); });
426​ connect(m_activities, &KActivities::Consumer::serviceStatusChanged,
427​ this, [this] { m_rules["activity"]->setOptionsData(activitiesModelData()); });
428​
429​#endif
430​
431​ addRule(new RuleItem(QLatin1String("screen"),
432​ RulePolicy::SetRule, RuleItem::Integer,
433​ i18n("Screen"), i18n("Size & Position"),
434​ QIcon::fromTheme("osd-shutd-screen")));
435​
436​ addRule(new RuleItem(QLatin1String("fullscreen"),
437​ RulePolicy::SetRule, RuleItem::Boolean,
438​ i18n("Fullscreen"), i18n("Size & Position"),
439​ QIcon::fromTheme("view-fullscreen")));
440​
441​ addRule(new RuleItem(QLatin1String("minimize"),
442​ RulePolicy::SetRule, RuleItem::Boolean,
443​ i18n("Minimized"), i18n("Size & Position"),
444​ QIcon::fromTheme("window-minimize")));
445​
446​ addRule(new RuleItem(QLatin1String("shade"),
447​ RulePolicy::SetRule, RuleItem::Boolean,
448​ i18n("Shaded"), i18n("Size & Position"),
449​ QIcon::fromTheme("window-shade")));
450​
451​ auto placement = addRule(new RuleItem(QLatin1String("placement"),
452​ RulePolicy::ForceRule, RuleItem::Option,
453​ i18n("Initial placement"), i18n("Size & Position"),
454​ QIcon::fromTheme("region")));
455​ placement->setOptionsData(placementModelData());
456​
457​ addRule(new RuleItem(QLatin1String("ignoregeometry"),
458​ RulePolicy::SetRule, RuleItem::Boolean,
459​ i18n("Ignore requested geometry"), i18n("Size & Position"),
460​ QIcon::fromTheme("view-time-schedule-baselined-remove"),
461​ i18n("Windows can ask to appear in a certain position.\n"
462​ "By default this overrides the placement strategy\n"
463​ "what might be nasty if the client abuses the feature\n"
464​ "to unconditionally popup in the middle of your screen.")));
465​
466​ addRule(new RuleItem(QLatin1String("minsize"),
467​ RulePolicy::ForceRule, RuleItem::Coordinate,
468​ i18n("Minimum Size"), i18n("Size & Position"),
469​ QIcon::fromTheme("image-resize-symbolic")));
470​
471​ addRule(new RuleItem(QLatin1String("maxsize"),
472​ RulePolicy::ForceRule, RuleItem::Coordinate,
473​ i18n("Maximum Size"), i18n("Size & Position"),
474​ QIcon::fromTheme("image-resize-symbolic")));
475​
476​ addRule(new RuleItem(QLatin1String("strictgeometry"),
477​ RulePolicy::ForceRule, RuleItem::Boolean,
478​ i18n("Obey geometry restrictions"), i18n("Size & Position"),
479​ QIcon::fromTheme("transform-crop-and-resize"),
480​ i18n("Eg. terminals or video players can ask to keep a certain aspect ratio\n"
481​ "or only grow by values larger than one\n"
482​ "(eg. by the dimensions of one character).\n"
483​ "This may be pointless and the restriction prevents arbitrary dimensions\n"
484​ "like your complete screen area.")));
485​
486​ // Arrangement & Access
487​ addRule(new RuleItem(QLatin1String("above"),
488​ RulePolicy::SetRule, RuleItem::Boolean,
489​ i18n("Keep above"), i18n("Arrangement & Access"),
490​ QIcon::fromTheme("window-keep-above")));
491​
492​ addRule(new RuleItem(QLatin1String("below"),
493​ RulePolicy::SetRule, RuleItem::Boolean,
494​ i18n("Keep below"), i18n("Arrangement & Access"),
495​ QIcon::fromTheme("window-keep-below")));
496​
497​ addRule(new RuleItem(QLatin1String("skiptaskbar"),
498​ RulePolicy::SetRule, RuleItem::Boolean,
499​ i18n("Skip taskbar"), i18n("Arrangement & Access"),
500​ QIcon::fromTheme("kt-show-statusbar"),
501​ i18n("Window shall (not) appear in the taskbar.")));
502​
503​ addRule(new RuleItem(QLatin1String("skippager"),
504​ RulePolicy::SetRule, RuleItem::Boolean,
505​ i18n("Skip pager"), i18n("Arrangement & Access"),
506​ QIcon::fromTheme("org.kde.plasma.pager"),
507​ i18n("Window shall (not) appear in the manager for virtual desktops")));
508​
509​ addRule(new RuleItem(QLatin1String("skipswitcher"),
510​ RulePolicy::SetRule, RuleItem::Boolean,
511​ i18n("Skip switcher"), i18n("Arrangement & Access"),
512​ QIcon::fromTheme("preferences-system-windows-effect-flipswitch"),
513​ i18n("Window shall (not) appear in the Alt+Tab list")));
514​
515​ addRule(new RuleItem(QLatin1String("shortcut"),
516​ RulePolicy::SetRule, RuleItem::Shortcut,
517​ i18n("Shortcut"), i18n("Arrangement & Access"),
518​ QIcon::fromTheme("configure-shortcuts")));
519​
520​ // Appearance & Fixes
521​ addRule(new RuleItem(QLatin1String("noborder"),
522​ RulePolicy::SetRule, RuleItem::Boolean,
523​ i18n("No titlebar and frame"), i18n("Appearance & Fixes"),
524​ QIcon::fromTheme("dialog-cancel")));
525​
526​ auto decocolor = addRule(new RuleItem(QLatin1String("decocolor"),
527​ RulePolicy::ForceRule, RuleItem::Option,
528​ i18n("Titlebar color scheme"), i18n("Appearance & Fixes"),
529​ QIcon::fromTheme("preferences-desktop-theme")));
530​ decocolor->setOptionsData(colorSchemesModelData());
531​
532​ addRule(new RuleItem(QLatin1String("opacityactive"),
533​ RulePolicy::ForceRule, RuleItem::Percentage,
534​ i18n("Active opacity"), i18n("Appearance & Fixes"),
535​ QIcon::fromTheme("edit-opacity")));
536​
537​ addRule(new RuleItem(QLatin1String("opacityinactive"),
538​ RulePolicy::ForceRule, RuleItem::Percentage,
539​ i18n("Inactive opacity"), i18n("Appearance & Fixes"),
540​ QIcon::fromTheme("edit-opacity")));
541​
542​ auto fsplevel = addRule(new RuleItem(QLatin1String("fsplevel"),
543​ RulePolicy::ForceRule, RuleItem::Option,
544​ i18n("Focus stealing prevention"), i18n("Appearance & Fixes"),
545​ QIcon::fromTheme("preferences-system-windows-effect-glide"),
546​ i18n("KWin tries to prevent windows from taking the focus\n"
547​ "(\"activate\") while you're working in another window,\n"
548​ "but this may sometimes fail or superact.\n"
549​ "\"None\" will unconditionally allow this window to get the focus while\n"
550​ "\"Extreme\" will completely prevent it from taking the focus.")));
551​ fsplevel->setOptionsData(focusModelData());
552​
553​ auto fpplevel = addRule(new RuleItem(QLatin1String("fpplevel"),
554​ RulePolicy::ForceRule, RuleItem::Option,
555​ i18n("Focus protection"), i18n("Appearance & Fixes"),
556​ QIcon::fromTheme("preferences-system-windows-effect-minimize"),
557​ i18n("This controls the focus protection of the currently active window.\n"
558​ "None will always give the focus away,\n"
559​ "Extreme will keep it.\n"
560​ "Otherwise it's interleaved with the stealing prevention\n"
561​ "assigned to the window that wants the focus.")));
562​ fpplevel->setOptionsData(focusModelData());
563​
564​ addRule(new RuleItem(QLatin1String("acceptfocus"),
565​ RulePolicy::ForceRule, RuleItem::Boolean,
566​ i18n("Accept focus"), i18n("Appearance & Fixes"),
567​ QIcon::fromTheme("preferences-desktop-cursors"),
568​ i18n("Windows may prevent to get the focus (activate) when being clicked.\n"
569​ "On the other hand you might wish to prevent a window\n"
570​ "from getting focused on a mouse click.")));
571​
572​ addRule(new RuleItem(QLatin1String("disableglobalshortcuts"),
573​ RulePolicy::ForceRule, RuleItem::Boolean,
574​ i18n("Ignore global shortcuts"), i18n("Appearance & Fixes"),
575​ QIcon::fromTheme("input-keyboard-virtual-off"),
576​ i18n("When used, a window will receive\n"
577​ "all keyboard inputs while it is active, including Alt+Tab etc.\n"
578​ "This is especially interesting for emulators or virtual machines.\n"
579​ "\n"
580​ "Be warned:\n"
581​ "you won't be able to Alt+Tab out of the window\n"
582​ "nor use any other global shortcut (such as Alt+F2 to show KRunner)\n"
583​ "while it's active!")));
584​
585​ addRule(new RuleItem(QLatin1String("closeable"),
586​ RulePolicy::ForceRule, RuleItem::Boolean,
587​ i18n("Closeable"), i18n("Appearance & Fixes"),
588​ QIcon::fromTheme("dialog-close")));
589​
590​ auto type = addRule(new RuleItem(QLatin1String("type"),
591​ RulePolicy::ForceRule, RuleItem::Option,
592​ i18n("Set window type"), i18n("Appearance & Fixes"),
593​ QIcon::fromTheme("window-duplicate")));
594​ type->setOptionsData(windowTypesModelData());
595​
596​ addRule(new RuleItem(QLatin1String("desktopfile"),
597​ RulePolicy::SetRule, RuleItem::String,
598​ i18n("Desktop file name"), i18n("Appearance & Fixes"),
599​ QIcon::fromTheme("application-x-desktop")));
600​
601​ addRule(new RuleItem(QLatin1String("blockcompositing"),
602​ RulePolicy::ForceRule, RuleItem::Boolean,
603​ i18n("Block compositing"), i18n("Appearance & Fixes"),
604​ QIcon::fromTheme("composite-track-on")));
605​}
606​
607​
608​const QHash<QString, QString> RulesModel::x11PropertyHash()
609​{
610​ static const auto propertyToRule = QHash<QString, QString> {
611​ /* The original detection dialog allows to choose depending on "Match complete window class":
612​ * if Match Complete == false: wmclass = "resourceClass"
613​ * if Match Complete == true: wmclass = "resourceName" + " " + "resourceClass"
614​ */
615​ { "resourceName", "wmclass" },
616​ { "caption", "title" },
617​ { "role", "windowrole" },
618​ { "clientMachine", "clientmachine" },
619​ { "x11DesktopNumber", "desktop" },
620​ { "maximizeHorizontal", "maximizehoriz" },
621​ { "maximizeVertical", "maximizevert" },
622​ { "minimized", "minimize" },
623​ { "shaded", "shade" },
624​ { "fullscreen", "fullscreen" },
625​ { "keepAbove", "above" },
626​ { "keepBelow", "below" },
627​ { "noBorder", "noborder" },
628​ { "skipTaskbar", "skiptaskbar" },
629​ { "skipPager", "skippager" },
630​ { "skipSwitcher", "skipswitcher" },
631​ { "type", "type" },
632​ { "desktopFile", "desktopfile" }
633​ };
634​ return propertyToRule;
635​};
636​
637​void RulesModel::prefillProperties(const QVariantMap &info)
638​{
639​ beginResetModel();
640​
641​ // Properties that cannot be directly applied via x11PropertyHash
642​ const QString position = QStringLiteral("%1,%2").arg(info.value("x").toInt())
643​ .arg(info.value("y").toInt());
644​ const QString size = QStringLiteral("%1,%2").arg(info.value("width").toInt())
645​ .arg(info.value("height").toInt());
646​
647​ if (!m_rules["position"]->isEnabled()) {
648​ m_rules["position"]->setValue(position);
649​ }
650​ if (!m_rules["size"]->isEnabled()) {
651​ m_rules["size"]->setValue(size);
652​ }
653​ if (!m_rules["minsize"]->isEnabled()) {
654​ m_rules["minsize"]->setValue(size);
655​ }
656​ if (!m_rules["maxsize"]->isEnabled()) {
657​ m_rules["maxsize"]->setValue(size);
658​ }
659​
660​ NET::WindowType window_type = static_cast<NET::WindowType>(info.value("type", 0).toInt());
661​ if (!m_rules["types"]->isEnabled() || m_rules["types"]->value() == 0) {
662​ if (window_type == NET::Unknown) {
663​ window_type = NET::Normal;
664​ }
665​ m_rules["types"]->setValue(1 << window_type);
666​ }
667​
668​ const auto ruleForProperty = x11PropertyHash();
669​ for (QString &property : info.keys()) {
670​ if (!ruleForProperty.contains(property)) {
671​ continue;
672​ }
673​ const QString ruleKey = ruleForProperty.value(property, QString());
674​ Q_ASSERT(hasRule(ruleKey));
675​
676​ // Only prefill disabled or empty properties
677​ if (!m_rules[ruleKey]->isEnabled() || m_rules[ruleKey]->value().toString().isEmpty()) {
678​ m_rules[ruleKey]->setValue(info.value(property));
679​ }
680​ }
681​
682​ endResetModel();
683​
684​ emit descriptionChanged();
685​ emit warningMessageChanged();
686​}
687​
688​
689​QList<OptionsModel::Data> RulesModel::windowTypesModelData() const
690​{
691​ static const auto modelData = QList<OptionsModel::Data> {
692​ //TODO: Find/create better icons
693​ { NET::Normal, i18n("Normal Window") , QIcon::fromTheme("window") },
694​ { NET::Dialog, i18n("Dialog Window") , QIcon::fromTheme("window-duplicate") },
695​ { NET::Utility, i18n("Utility Window") , QIcon::fromTheme("dialog-object-properties") },
696​ { NET::Dock, i18n("Dock (panel)") , QIcon::fromTheme("list-remove") },
697​ { NET::Toolbar, i18n("Toolbar") , QIcon::fromTheme("tools") },
698​ { NET::Menu, i18n("Torn-Off Menu") , QIcon::fromTheme("overflow-menu-left") },
699​ { NET::Splash, i18n("Splash Screen") , QIcon::fromTheme("embosstool") },
700​ { NET::Desktop, i18n("Desktop") , QIcon::fromTheme("desktop") },
701​ // { NET::Override, i18n("Unmanaged Window") }, deprecated
702​ { NET::TopMenu, i18n("Standalone Menubar"), QIcon::fromTheme("open-menu-symbolic") }
703​ };
704​ return modelData;
705​}
706​
davidedmundsonUnsubmitted
Not Done

This won't work on wayland, we will want to copy whatever we do in the virtual desktop KCM.

I think this is not regression, as the current code does the same.
At which point I'm happy to merge now and revist it later.

davidedmundson: This won't work on wayland, we will want to copy whatever we do in the virtual desktop KCM. I…
iasensioAuthorUnsubmitted
Not Done

I'll look into it. Thanks!

iasensio: I'll look into it. Thanks!
707​QList<OptionsModel::Data> RulesModel::virtualDesktopsModelData() const
708​{
709​ QList<OptionsModel::Data> modelData;
710​ for (int desktopId = 1; desktopId <= KWindowSystem::numberOfDesktops(); ++desktopId) {
711​ modelData << OptionsModel::Data{
712​ desktopId,
713​ QString::number(desktopId).rightJustified(2) + QStringLiteral(": ") + KWindowSystem::desktopName(desktopId),
714​ QIcon::fromTheme("virtual-desktops")
715​ };
716​ }
717​ modelData << OptionsModel::Data{ NET::OnAllDesktops, i18n("All Desktops"), QIcon::fromTheme("window-pin") };
718​ return modelData;
719​}
720​
721​
722​QList<OptionsModel::Data> RulesModel::activitiesModelData() const
723​{
724​#ifdef KWIN_BUILD_ACTIVITIES
725​ QList<OptionsModel::Data> modelData;
726​
727​ // NULL_ID from kactivities/src/lib/core/consumer.cpp
728​ modelData << OptionsModel::Data{
729​ QString::fromLatin1("00000000-0000-0000-0000-000000000000"),
730​ i18n("All Activities"),
731​ QIcon::fromTheme("activities")
732​ };
733​
734​ const auto activities = m_activities->activities(KActivities::Info::Running);
735​ if (m_activities->serviceStatus() == KActivities::Consumer::Running) {
736​ for (const QString &activityId : activities) {
737​ const KActivities::Info info(activityId);
738​ modelData << OptionsModel::Data{ activityId, info.name(), QIcon::fromTheme(info.icon()) };
739​ }
740​ }
741​
742​ return modelData;
743​#else
744​ return {};
745​#endif
746​}
747​
748​QList<OptionsModel::Data> RulesModel::placementModelData() const
749​{
750​ // From "placement.h" : Placement rule is stored as a string, not the enum value
751​ static const auto modelData = QList<OptionsModel::Data> {
752​ { Placement::policyToString(Placement::Default), i18n("Default") },
753​ { Placement::policyToString(Placement::NoPlacement), i18n("No Placement") },
754​ { Placement::policyToString(Placement::Smart), i18n("Minimal Overlapping") },
755​ { Placement::policyToString(Placement::Maximizing), i18n("Maximized") },
756​ { Placement::policyToString(Placement::Cascade), i18n("Cascaded") },
757​ { Placement::policyToString(Placement::Centered), i18n("Centered") },
758​ { Placement::policyToString(Placement::Random), i18n("Random") },
759​ { Placement::policyToString(Placement::ZeroCornered), i18n("In Top-Left Corner") },
760​ { Placement::policyToString(Placement::UnderMouse), i18n("Under Mouse") },
761​ { Placement::policyToString(Placement::OnMainWindow), i18n("On Main Window") }
762​ };
763​ return modelData;
764​}
765​
766​QList<OptionsModel::Data> RulesModel::focusModelData() const
767​{
768​ static const auto modelData = QList<OptionsModel::Data> {
769​ { 0, i18n("None") },
770​ { 1, i18n("Low") },
771​ { 2, i18n("Normal") },
772​ { 3, i18n("High") },
773​ { 4, i18n("Extreme") }
774​ };
broulikUnsubmitted
Done

There's now a KSortFilterProxyModel in KItemModels you can use from QML for simple filtering like this. Just a suggestion.

broulik: There's now a `KSortFilterProxyModel` in `KItemModels` you can use from QML for simple…
iasensioAuthorUnsubmitted
Done

That seems very handy. I've been trying to, but apparently there is no way to invalidateFilter() from the QML side, to add the showAll condition.
Maybe can it be extended on KItemModels?

iasensio: That seems very handy. I've been trying to, but apparently there is no way to `invalidateFilter…
775​ return modelData;
776​}
broulikUnsubmitted
Done

I probably would have stored the search string in a member and done the filtering fully custom here rather than relying on filterFixedString

broulik: I probably would have stored the search string in a member and done the filtering fully custom…
iasensioAuthorUnsubmitted
Done

Added fully custom search, to include name and value properties

iasensio: Added fully custom search, to include `name` and `value` properties
777​
778​QList<OptionsModel::Data> RulesModel::colorSchemesModelData() const
779​{
780​ QList<OptionsModel::Data> modelData;
781​
782​ KColorSchemeManager schemes;
783​ QAbstractItemModel *schemesModel = schemes.model();
784​
broulikUnsubmitted
Done

You can also do QModelIndex::data()

broulik: You can also do `QModelIndex::data()`
785​ // Skip row 0, which is Default scheme
786​ for (int r = 1; r < schemesModel->rowCount(); r++) {
787​ const QModelIndex index = schemesModel->index(r, 0);
788​ modelData << OptionsModel::Data{
789​ QFileInfo(index.data(Qt::UserRole).toString()).baseName(),
790​ index.data(Qt::DisplayRole).toString(),
791​ index.data(Qt::DecorationRole).value<QIcon>()
792​ };
793​ }
794​
795​ return modelData;
796​}
797​
798​void RulesModel::detectWindowProperties(int secs)
799​{
800​ QTimer::singleShot(secs*1000, this, &RulesModel::selectX11Window);
801​}
broulikUnsubmitted
Done

Generally we check if a property has actually changed before emitting a change:

if (m_showAll == showAll) {
    return;
}
...
broulik: Generally we check if a property has actually changed before emitting a change: ``` if…
802​
803​void RulesModel::selectX11Window()
804​{
805​ QDBusMessage message = QDBusMessage::createMethodCall(QStringLiteral("org.kde.KWin"),
806​ QStringLiteral("/KWin"),
807​ QStringLiteral("org.kde.KWin"),
808​ QStringLiteral("queryWindowInfo"));
809​
810​ QDBusPendingReply<QVariantMap> async = QDBusConnection::sessionBus().asyncCall(message);
811​
812​ QDBusPendingCallWatcher *callWatcher = new QDBusPendingCallWatcher(async, this);
813​ connect(callWatcher, &QDBusPendingCallWatcher::finished, this,
814​ [this](QDBusPendingCallWatcher *self) {
815​ QDBusPendingReply<QVariantMap> reply = *self;
816​ self->deleteLater();
817​ if (!reply.isValid()) {
818​ return;
819​ }
820​ const QVariantMap windowInfo = reply.value();
821​ //TODO: Improve UI to suggest/select the detected properties.
822​ // For now, just prefill unused rules
823​ prefillProperties(windowInfo);
824​ }
825​ );
826​}
827​
828​} //namespace