Differential D16644

KateSessionManager: Limit the number of referenced jump list actions
ClosedPublic
Actions

Authored by loh.tar on Nov 3 2018, 5:47 PM.

Details

Reviewers
cullmann
Group Reviewers
Kate
Commits
R40:dd4760bcbb1a: KateSessionManager: Limit the number of referenced jump list actions
Summary
...to the number of defined actions

This fix the grumble...

  The desktop file "foobar/org.kde.kate.desktop" references the action
  "Session <checksum>" but doesn't define it

...when there are more than 10 sessions available
Test Plan
  • Ensure not to run a kate instance without this patch or you will see anymay the grumble
  • Create more than 10 sessions
  • Switch sessions and watch

Diff Detail

Repository
R40 Kate
Lint
Lint Skipped
Unit
Unit Tests Skipped
loh.tar created this revision.Nov 3 2018, 5:47 PM
Restricted Application added a subscriber: kwrite-devel. · View Herald TranscriptNov 3 2018, 5:47 PM
loh.tar requested review of this revision.Nov 3 2018, 5:47 PM
cullmann accepted this revision.Nov 4 2018, 11:58 AM
cullmann added a subscriber: cullmann.

Looks reasonable, thanks.
Makes no sense to create them but later limit how many to use.

This revision is now accepted and ready to land.Nov 4 2018, 11:58 AM
Closed by commit R40:dd4760bcbb1a: KateSessionManager: Limit the number of referenced jump list actions (authored by loh.tar, committed by cullmann). · Explain WhyNov 11 2018, 2:44 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mkate/session/katesessionmanager.cpp (13 lines)

Diff 44788

View Options

kate/session/katesessionmanager.cpp

Loading...