Sync ("Wayland") appending to match SDDM
ClosedPublic

Authored by davidedmundson on Thu, Feb 13, 9:42 AM.

Details

Summary

For historical reasons some .desktop files in wayland-sessions suffix
their .desktop files, some expect the display manager to do it.

SDDM compares the suffix and only adds if needed. Our config UI should
match exactly.

The obvious comment is this only works in English, but for some reason
both SDDM and this code only read the Name entry in English anyway.

BUG: 417499

Test Plan

Opened KCM

Diff Detail

Repository
R123 SDDM Configuration Panel (KCM)
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
davidedmundson created this revision.Thu, Feb 13, 9:42 AM
Restricted Application added a project: Plasma. · View Herald TranscriptThu, Feb 13, 9:42 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Thu, Feb 13, 9:42 AM
ngraham accepted this revision.Thu, Feb 13, 9:44 PM
ngraham added a subscriber: ngraham.

Wasn't this something that SDDM was going to stop doing in its next release?

This revision is now accepted and ready to land.Thu, Feb 13, 9:44 PM

Given we have that .endsWith("Wayland") check, there isn't any rush to remove it

This revision was automatically updated to reflect the committed changes.