Rewrite model to use QAbstractListModel instead of QStandardItemModel

Jan Blackquill authored Sep 07, 2020 and Nate Graham committed Sep 17, 2020

The writing subroutine no longer:

- Leaves old groups behind when they don't exist in the new color scheme
- Properly handles writing [Colors:*][Inactive] subgroups

Summary:
This brings numerous advantages such as:

- easier admin configuration with drop-ins, overrides and multiple
hooks throughout (and only having to learn one tool)

- session cleanup on exit, avoiding that occasional part where shutdown
hangs

 - startup that actually knows when things are up

 - race free autostart and DBus activation at once

 - logs that rotate are split by service and usable

 - resource management through slices and cgroups (the part I want)

Over the past 2 years I've been trying to tidy up and encapsulate the
relevant parts of startup into the binary plasma-session so that we can
just runtime swap out that one part and supporting both paths will be
easy.

Support is toggleable via cmake flag, as it seems like it should be a
distro decision, especially as we will require a specific systemd with
the xdg-generator.

KDED/kwin/other services are attached to the relevant repo.

Task T11914

Differential Revision: https://phabricator.kde.org/D28305

squash

This existed before QQuickWindow::sceneGraphError which ShellCorona
currently connects to.

This code is now redundant

GIT_SILENT

GIT_SILENT

GIT_SILENT

In case of conflict in i18n, keep the version of the branch "ours"
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"

Eugene Popov authored Sep 16, 2020 and Nate Graham committed Sep 16, 2020

At the moment, in the battery widget, the slider displays the brightness of the screen, but
it doesn't have text labels, so it's quite difficult to determine the brightness of the
screen. Adding a label over the slider that will display the current screen brightness as a
percentage will make this applet more descriptive and handy.

The connection to the lambda persists, so when it's called a second time,
the lambda is also called twice, eventually resulting in a nullptr deref as
m_restoreSessionCall is empty.

Commit 19c06e72 removed setting dpy,
but it is still used in the NewKDM and OldKDM code branches.

So set it again, and add a check that it's not empty where relevant.

this frees the internal per-thread inotify instance used to watch
database changes meaning runner applications no longer claim N inotify
instances (N=size of threadweaver pool).
the database is still reloaded by ksyscoca internally if it finds the
backing file has changed.

CCBUG: 423818

Some users reported this button becoming too small after KF5.74 was released, but I could not reproduce it myself. However, this code did look suspicious. Hopefully this fixes the issue.

In case of conflict in i18n, keep the version of the branch "ours"
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"

Ahmad Samir authored Sep 02, 2020 and David Edmundson committed Sep 14, 2020

Drive-by changes: less implicit casting from ASCII

Ahmad Samir authored Sep 02, 2020 and David Edmundson committed Sep 14, 2020

- Use range-for/for_each algorithim
- Less implicit casting from ASCII
- Use string comparision instead of QRegExp, the former is usually faster
- Replace one letter var names with slightly longer ones

David Edmundson authored Sep 10, 2020 and Aleix Pol Gonzalez committed Sep 14, 2020

This should be safe from a migration discussion
as a user has already opted into a setting called "Platform". This is
effectively the same.

David Edmundson authored Aug 20, 2020 and Aleix Pol Gonzalez committed Sep 14, 2020

ShellCorona is a messy class doing mulitple functions.

This patch splits out all telemetry code into a new class, meaning it's
easier to extend. Code remains the same.

kcms are handy but not nearly as handy as an actual application. this is
particular notable with the test case where the user could type 'virt'
but that wouldn't have pushed virtualization software to the top but kcms
since kcms are kde things so they always get a .09 bump to
relevance.

the revised scoring excludes kcms from the bump and further reduces the
relvance by the tiniest fraction. this results in all applications that
match at least as well as the KCM to outscore the KCM

When we moved fonts kcm to this repository this find_package call was
missing appearantly since this was not in top-level KCM(!).

Add this find_package call here, so it finds it correctly and fonts kcm
and kfontinst gets built correctly

In case of conflict in i18n, keep the version of the branch "ours"
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"

In case of conflict in i18n, keep the version of the branch "ours"
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"

This code has been commented out for 6 years and was not missed.

In case of conflict in i18n, keep the version of the branch "ours"
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"

Currently when we invoke KRunner and it is already running we will start binary,
KDBusService will check if it is already registered and then call
org.freedesktop.Application.Activate. Cutting the middle out should improve the
time that it takes to show KRunner and make it less likely that key strokes are
lost. If KRunner is not running we, can rely on dbus activiation to start KRunner.
Unfortunately we cannnot call Activate with dbus-send because it doesn't support
it's argument type. Instead we can just use the existing display method.
CCBUG: 416145

In case of conflict in i18n, keep the version of the branch "ours"
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"

Piotr Henryk Dabrowski authored Mar 02, 2020 and Nate Graham committed Sep 09, 2020

Aggregate text for KRunner in DesktopView to prevent eating fast-typed characters.

CCBUG: 416145
partial solution: resolved for DesktopView, not resolved for running KRunner from shortcut

Test Plan:
When DesktopView is focused, typed characters show KRunner query.
All fast-typed characters should now appear correctly in KRunner window,
instead of only first typed character.

Simplifies the logic of the busy indicator since now it will already
fade out when it's not running.
Also addresses the logic of the timer which was off.

This made krunner freak out on Wayland for some reason to investigate,
both opacity animations were triggering a repaint and we tried to render
too often making the process block. With this change it works fine
again.

Otherwise we have to reevaluate a bunch of expressions to update with
the new value. Fixes several warnings too.

BUG: 421589

In case of conflict in i18n, keep the version of the branch "ours"
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"