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
