Bug 361650

Summary:	Add a Plasma notification once a screenshot has been taken
Product:	[Applications] Spectacle	Reporter:	Sudhir Khanger <sudhir>
Component:	General	Assignee:	Boudhayan Gupta <me>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Fedora RPMs
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	Android 6.0 Screenshot notifications

Description Sudhir Khanger 2016-04-12 03:40:58 UTC

When I take a screenshot via shortcut the image gets saved in the designated folder. I have to then browse to the folder location to open the file. Instead Spectacle should create a Plasma notification that a screenshot has been taken and an option to open it.

Reproducible: Always

Steps to Reproduce:
1. Take a screenshot via shortcut
2.
3.

Actual Results:  
Notification is not persistent and there is no way to open image from the notification

Expected Results:  
1. The bubble notification should have an open button to open the screenshot.
2. Spectacle should create a persistent notification.

Just to clarify myself I would like to see following changes.

1. Notification is more descriptive. For example, "Active window screenshot was captured and saved to xyz/folder."
2. Bubble notification has a button to open the screenshot.
3. The notification stays in the Status & Notifications unless dismissed.

Comment 1 Boudhayan Gupta 2016-04-25 20:46:23 UTC

We'll implement 2 - it seems very useful - but 1 and 3 won't happen.

3 is just annoying for a vast majority of users, and for 1 the code that generates the notification doesn't know what kind of screenshot was taken. Passing that information down the chain will require way too many modifications to the code - this effort at the moment is unjustified.

Patches for 1 are welcome, of course.

Comment 2 Boudhayan Gupta 2016-04-25 22:43:06 UTC

Apparently Kai already implemented 1 and 2, so we're done :-)

Comment 3 Sudhir Khanger 2016-04-26 02:16:12 UTC

Thanks.

> 3 is just annoying for a vast majority of users

With all due respect I am not convinced that your statement has any backings other than your personal feelings. Android with its massive user penetration also has exactly the same behavior. When you take snapshot it places a notification in the notification drawer which has to be dismissed by user. Without this notification a user has to go at great lengths to open file manager and navigate to the folder which contains the screenshot.

Comment 4 Sudhir Khanger 2016-04-26 02:19:24 UTC

Created attachment 98599 [details]
Android 6.0 Screenshot notifications

Here is how Android 6.0 shows screenshot notifications.

If at some point of time you want this feature in Spectacle or you want someone else to submit patch please change the the bug reports status.

Thank you very to you and Kai for fixing 1 & 2.

Comment 5 Boudhayan Gupta 2016-04-26 05:26:58 UTC

This is not a personal opinion. In Plasma, unlike in Android, persistent notifications aren't stored away in a drawer, but stay on the corner of the screen until you specifically take action to close them.

Also, to handle the "Open" action in a persistent notification, Spectacle would have to keep running in the background until the notification was either closed or the image opened, which could be a while.