Bug 361650 - Add a Plasma notification once a screenshot has been taken
Summary: Add a Plasma notification once a screenshot has been taken
Status: RESOLVED FIXED
Alias: None
Product: Spectacle
Classification: Applications
Component: General (other bugs)
Version First Reported In: unspecified
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: Boudhayan Gupta
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-04-12 03:40 UTC by Sudhir Khanger
Modified: 2016-04-26 05:26 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Android 6.0 Screenshot notifications (35.88 KB, image/jpeg)
2016-04-26 02:19 UTC, Sudhir Khanger
Details

Note You need to log in before you can comment on or make changes to this bug.
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.