[Differential] [Accepted] D3376: [effects] Show a notification when a screenshot got saved to file

[mart (Marco Martin)]

mart accepted this revision.
mart added a reviewer: mart.
mart added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> screenshot.cpp:249
> +                        i18nc("Notification caption that a screenshot got saved to file", "Screenshot"),
> +                        i18nc("Notification with path to screenshot file", "Screenshot saved to %1", temp.fileName()),
> +                        QStringLiteral("spectacle"));

I think for this thing, spectacle is the right icon, as it's the only thing that is presented to the user "this makes a screenshot"

REPOSITORY
  rKWIN KWin

BRANCH
  screenshot-show-notification

REVISION DETAIL
  https://phabricator.kde.org/D3376

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma, #vdg, mart
Cc: mart, andreask, plasma-devel, kwin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161116/b357289c/attachment-0001.html>