github/greenshot
1
0
Fork 0
mirror of https://github.com/greenshot/greenshot synced 2025-08-14 10:47:02 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions

Changed the picker behavior to show again if the export failed or was "canceled"

Browse source 
git-svn-id: http://svn.code.sf.net/p/greenshot/code/trunk@1913 7dccd23d-a4a3-4e1f-8c07-b4c1b4018ab4
This commit is contained in:
RKrom 2012-06-08 13:52:55 +00:00
commit 6f3c928792
2 changed files with 28 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

27
Greenshot/Destinations/PickerDestination.cs
View file

@ -78,15 +78,26 @@ namespace Greenshot.Destinations {
if (clickedDestination == null) {
return;
}
// Make sure the menu is closed
menu.Close();
bool isEditor = EditorDestination.DESIGNATION.Equals(clickedDestination.Designation);
// Make sure the menu is invisible, don't close it
menu.Hide();
// Export
bool result = clickedDestination.ExportCapture(true, surface, captureDetails);
// TODO: Find some better way to detect that we exported to the editor
if (!EditorDestination.DESIGNATION.Equals(clickedDestination.Designation) || result == false) {
LOG.DebugFormat("Disposing as Destination was {0} and result {1}", clickedDestination.Description, result);
// Cleanup surface
surface.Dispose();
}
LOG.InfoFormat("Destination was {0} and result {1}", clickedDestination.Designation, result);
if (result == true) {
LOG.Info("Export success, closing menu");
// close menu if the destination wasn't the editor
menu.Close();
// Cleanup surface, only if the destination wasn't the editor
if (!isEditor) {
surface.Dispose();
}
} else {
LOG.Info("Export failed, showing menu again");
menu.Show();
}
}
);
if (item != null) {