Skip to content

Various small improvements to desktop and json files

Christoph Wolk requested to merge cwo/kdeconnect-kde:desktop-file-fix into master

Full commit logs below.

Some visuals in context:

current new
kdeconnect_plasmoid_old kdeconnect_plasmoid_new
kdeconnect_desktop_old kdeconnect_desktop_new

I can't make a screenshot for the settings app .desktop outside plasma (I use plasma, and there's so many different environments that may handle the exact display of .desktop files differently).

Involves string changes, so not for 24.08.

Let me know if you want the commits autosquashed.

BUG: 466256


kcm: improve description field in json

The kcm json currently lists the description as "Connect and sync your
devices". This is hard to distinguish from the KDE Connect application
at a glance, and does not follow the general pattern in kcm descriptions
to use either a configuration-based verb or noun (e.g. configure, edit,
settings).

This change sets the description to "Configure device synchronization".

BUG: 466256

plasmoid: improve description field in json

The plasmoid currently has the decription "Show notifications from your
devices using KDE Connect. This is wordy and misleading as
the plasmoid does much more than display notifications; it's used to
show the  devices and their current state (e.g. battery) and provides
access to some of the broader KDE Connect functionality, such as the
find device function or file sharing.

This change sets the description to "Show and control devices".

settings: improve .desktop GenericName and Comment

The .desktop file for the non-Plasma settings app currently has a
sentence as the GenericName, which is supposed to be the term of a
class of applications, and does not have a Comment field, which is
suitable for descriptions in the form of a sentence.

This change sets the GenericName to "Device Synchronization Settings",
and adds a Domment field that matches the description for the kcm.

smsapp: improve .desktop GenericName and Description

The KDE Connect SMS app currently has the GenericName "SMS" which is the
name of a protocol, not the name for a class of applications, and the
Comment "Text Messaging", which is correct but very short for a field
that is intended to give a short but more exhaustive description of an
application's functions.

This change sets the GenericName To "Text Messaging" and the Comment to
"Read and send SMS messages on connected devices".

Merge request reports