Differential D22901

Make alphabetic categorization of the "All Applications" section optional
ClosedPublic
Actions

Authored by hein on Aug 2 2019, 4:42 PM.

Details

Reviewers
davidedmundson
tcanabrava
Group Reviewers
Plasma
Commits
R120:656bb8469e52: Make alphabetic categorization of the "All Applications" section optional
R119:258545f996cf: Make alphabetic categorization of the "All Applications" section optional
Summary

Introduces a new prop and turns it off by default.

Also rewrites the initial "all applications" data gathering avoid
a crash uncovered while working on this (it didn't check EntryType)
and reduces assumptions about the model topology.

It also makes it collect root-level apps now, which was forgotten
to be added when support for root-level apps were added.

This supercedes D22762 and D22890. It does less duplicate work and fixes
more bugs.

Diff Detail

Repository
R119 Plasma Desktop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
hein created this revision.Aug 2 2019, 4:42 PM
Restricted Application added a project: Plasma. · View Herald TranscriptAug 2 2019, 4:42 PM
hein requested review of this revision.Aug 2 2019, 4:42 PM
davidedmundson accepted this revision.Aug 2 2019, 4:48 PM

That's much nicer.

This revision is now accepted and ready to land.Aug 2 2019, 4:48 PM
Closed by commit R119:258545f996cf: Make alphabetic categorization of the "All Applications" section optional (authored by hein). · Explain WhyAug 2 2019, 4:50 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 62992

View Options

applets/kicker/package/contents/ui/main.qml

Loading...
View Options

applets/kicker/plugin/rootmodel.h

Loading...
View Options

applets/kicker/plugin/rootmodel.cpp

Loading...