sddm-theme: Set needsFullUserModel=false
ClosedPublic

Authored by fvogt on Oct 1 2019, 8:37 AM.

Details

Summary

Otherwise SDDM has to load all users even if they're not shown.
See https://github.com/sddm/sddm/pull/1017

Test Plan

Using this for several releases downstream to support >1500 users.

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
fvogt created this revision.Oct 1 2019, 8:37 AM
Restricted Application added a project: Plasma. · View Herald TranscriptOct 1 2019, 8:37 AM
fvogt requested review of this revision.Oct 1 2019, 8:37 AM
fvogt updated this revision to Diff 67111.Oct 1 2019, 8:38 AM

Maybe get rid of the unrelated change?

fvogt updated this revision to Diff 67112.Oct 1 2019, 8:46 AM

Maybe now?

I don't think I follow.

How does the regular non-textfield version of the login screen still work? Surely containsAllUsers will always be false.

fvogt added a comment.Oct 2 2019, 5:47 PM

I don't think I follow.

How does the regular non-textfield version of the login screen still work? Surely containsAllUsers will always be false.

It's only false if there are more users in the system than the threshold configured in sddm.

davidedmundson accepted this revision.Oct 7 2019, 8:28 AM
This revision is now accepted and ready to land.Oct 7 2019, 8:28 AM
This revision was automatically updated to reflect the committed changes.