Differential D4390

Set screenshot URL for notification also in stable branch
ClosedPublic
Actions

Authored by broulik on Feb 1 2017, 9:40 AM.

Details

Reviewers
davidedmundson
aacid
Group Reviewers
Spectacle
Plasma
Commits
R166:6ea1988af9e5: Set screenshot URL for notification also in stable branch
Summary

We made a huge fuzz about this feature in the 5.9 release announcement yet Spectacle that supports this isn't to be released until April.

Since it's just a one-liner let's add this feature to stable branch with an ifdef for KNotifications version (it was introduced in 5.29)

Test Plan

Applications/16.12 branch

Diff Detail

Repository
R166 Spectacle
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik updated this revision to Diff 10803.Feb 1 2017, 9:40 AM
broulik retitled this revision from to Set screenshot URL for notification also in stable branch.
broulik updated this object.
broulik edited the test plan for this revision. (Show Details)
broulik added reviewers: Plasma, Spectacle, aacid.
broulik set the repository for this revision to R166 Spectacle.
Restricted Application added a project: Plasma. · View Herald TranscriptFeb 1 2017, 9:40 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson accepted this revision.Feb 1 2017, 1:18 PM
davidedmundson added a reviewer: davidedmundson.
This revision is now accepted and ready to land.Feb 1 2017, 1:18 PM

Revision Contents
Changeset List

PathPackages
Msrc/SpectacleCore.cpp (5 lines)

Diff 10803

View Options

src/SpectacleCore.cpp

Loading...