[Spectacle] [Bug 361650] Add a Plasma notification once a screenshot has been taken

2016-04-25 Thread Boudhayan Gupta via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=361650

--- Comment #5 from Boudhayan Gupta  ---
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.

-- 
You are receiving this mail because:
You are watching all bug changes.


[Spectacle] [Bug 361650] Add a Plasma notification once a screenshot has been taken

2016-04-25 Thread Sudhir Khanger via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=361650

--- Comment #4 from Sudhir Khanger  ---
Created attachment 98599
  --> https://bugs.kde.org/attachment.cgi?id=98599&action=edit
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.

-- 
You are receiving this mail because:
You are watching all bug changes.


[Spectacle] [Bug 361650] Add a Plasma notification once a screenshot has been taken

2016-04-25 Thread Sudhir Khanger via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=361650

--- Comment #3 from Sudhir Khanger  ---
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.

-- 
You are receiving this mail because:
You are watching all bug changes.


[Spectacle] [Bug 361650] Add a Plasma notification once a screenshot has been taken

2016-04-25 Thread Boudhayan Gupta via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=361650

Boudhayan Gupta  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|CONFIRMED   |RESOLVED

--- Comment #2 from Boudhayan Gupta  ---
Apparently Kai already implemented 1 and 2, so we're done :-)

-- 
You are receiving this mail because:
You are watching all bug changes.


[Spectacle] [Bug 361650] Add a Plasma notification once a screenshot has been taken

2016-04-25 Thread Boudhayan Gupta via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=361650

Boudhayan Gupta  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|UNCONFIRMED |CONFIRMED

--- Comment #1 from Boudhayan Gupta  ---
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.

-- 
You are receiving this mail because:
You are watching all bug changes.