https://bugs.kde.org/show_bug.cgi?id=394182
Nate Graham <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Latest Commit| |https://commits.kde.org/spe | |ctacle/74a2df918f87d81eba16 | |117bb6ae242f6ab07937 Status|CONFIRMED |RESOLVED --- Comment #2 from Nate Graham <[email protected]> --- Git commit 74a2df918f87d81eba16117bb6ae242f6ab07937 by Nate Graham, on behalf of Kyle Utecht. Committed on 15/10/2018 at 19:52. Pushed by ngraham into branch 'master'. clarify logic for openScreenshotsFolder, refactor preferences access, and clarify method names Summary: New openScreenshotsFolder logic is to highlight recently saved file, then default save location (save mode) or last saved file's folder (save as mode) with failsafe as default save location. SpectacleConfig now stores the last saved files and can determine the last saved file location from the file name through separate method. Removed extraneous methods and variables from ExportManager. ExportManager now uses SpectacleConfig to access save location preferences. Reviewers: #spectacle, ngraham Reviewed By: #spectacle, ngraham Subscribers: ngraham, #spectacle Tags: #spectacle Differential Revision: https://phabricator.kde.org/D15908 M +8 -26 src/ExportManager.cpp M +1 -6 src/ExportManager.h M +21 -14 src/Gui/KSMainWindow.cpp M +2 -2 src/Gui/SettingsDialog/SaveOptionsPage.cpp M +30 -6 src/SpectacleConfig.cpp M +11 -5 src/SpectacleConfig.h https://commits.kde.org/spectacle/74a2df918f87d81eba16117bb6ae242f6ab07937 -- You are receiving this mail because: You are watching all bug changes.
