https://bugs.kde.org/show_bug.cgi?id=399395

Nate Graham <n...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Version Fixed In|                            |18.12.0
         Resolution|---                         |FIXED
      Latest Commit|                            |https://commits.kde.org/spe
                   |                            |ctacle/e5fd391d2416afa14fd2
                   |                            |c53ffdd58dddb04dc53c
             Status|CONFIRMED                   |RESOLVED

--- Comment #3 from Nate Graham <n...@kde.org> ---
Git commit e5fd391d2416afa14fd2c53ffdd58dddb04dc53c by Nate Graham.
Committed on 19/10/2018 at 13:26.
Pushed by ngraham into branch 'master'.

Don't save to a temp file if exporting to another app

Summary:
When exporting a screenshot to a local app for further processing, the image is
saved to `/tmp/`, which causes issues in the other app, not least of which is
that the file is not writable, so {nav Save} operations fail.

This patch makes Spectacle save the file to the default location when exporting
to another app.
FIXED-IN: 18.12.0

Test Plan:
1. {nav Export > [some app or Other Application...]}
2. Observe that the screenshot is saved at the default location, and then
opened in the app of your choice
3. Edit the image in that app and do {nav File > Save}
4. Observe that it works!

Reviewers: #spectacle, #kde_applications, broulik, utecht, alexeymin, shubham

Reviewed By: #spectacle, alexeymin

Subscribers: pino

Tags: #spectacle

Differential Revision: https://phabricator.kde.org/D16191

M  +6    -2    src/Gui/ExportMenu.cpp

https://commits.kde.org/spectacle/e5fd391d2416afa14fd2c53ffdd58dddb04dc53c

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

Reply via email to