From 63485449f616f69813d0b34eee699ca1fd2f7987 Mon Sep 17 00:00:00 2001
From: David Capello <davidcapello@gmail.com>
Date: Thu, 19 Mar 2015 10:17:40 -0300
Subject: [PATCH] Add an option in "Export Sprite Sheet" to open the generated
 sprite sheet

---
 data/pref.xml                                |  1 +
 data/widgets/export_sprite_sheet.xml         |  2 ++
 src/app/app.cpp                              |  2 +-
 src/app/commands/cmd_export_sprite_sheet.cpp | 17 ++++++++++++++++-
 src/app/document_exporter.h                  |  2 +-
 5 files changed, 21 insertions(+), 3 deletions(-)

diff --git a/data/pref.xml b/data/pref.xml
index 14854954b..a57971d9b 100644
--- a/data/pref.xml
+++ b/data/pref.xml
@@ -147,6 +147,7 @@
       <option id="best_fit" type="bool" default="false" />
       <option id="texture_filename" type="std::string" />
       <option id="data_filename" type="std::string" />
+      <option id="open_generated" type="bool" default="false" />
     </section>
   </document>
 
diff --git a/data/widgets/export_sprite_sheet.xml b/data/widgets/export_sprite_sheet.xml
index 5021b83ad..7e259df49 100644
--- a/data/widgets/export_sprite_sheet.xml
+++ b/data/widgets/export_sprite_sheet.xml
@@ -24,6 +24,8 @@
     <check id="data_enabled" text="JSON Data:" />
     <button id="data_filename" cell_hspan="3" />
 
+    <check id="open_generated" text="Open generated sprite sheet" cell_hspan="4" />
+
     <hbox cell_hspan="4">
       <boxfiller />
       <hbox homogeneous="true">
diff --git a/src/app/app.cpp b/src/app/app.cpp
index 7a7c9665d..09d44c1f3 100644
--- a/src/app/app.cpp
+++ b/src/app/app.cpp
@@ -434,7 +434,7 @@ void App::initialize(const AppOptions& options)
     if (trim)
       m_exporter->setTrimCels(true);
 
-    m_exporter->exportSheet();
+    base::UniquePtr<Document> spriteSheet(m_exporter->exportSheet());
     m_exporter.reset(NULL);
 
     PRINTF("Export sprite sheet: Done\n");
diff --git a/src/app/commands/cmd_export_sprite_sheet.cpp b/src/app/commands/cmd_export_sprite_sheet.cpp
index cad170132..0edd618aa 100644
--- a/src/app/commands/cmd_export_sprite_sheet.cpp
+++ b/src/app/commands/cmd_export_sprite_sheet.cpp
@@ -121,6 +121,8 @@ public:
     if (m_docPref.spriteSheet.type() != app::gen::SpriteSheetType::NONE)
       sheetType()->setSelectedItemIndex((int)m_docPref.spriteSheet.type()-1);
 
+    openGenerated()->setSelected(m_docPref.spriteSheet.openGenerated());
+
     for (int i=2; i<=8192; i*=2) {
       std::string value = base::convert_to<std::string>(i);
       if (i >= m_sprite->width()) fitWidth()->addItem(value);
@@ -210,6 +212,10 @@ public:
       return std::string();
   }
 
+  bool openGeneratedValue() {
+    return openGenerated()->isSelected();
+  }
+
 protected:
 
   void onSheetTypeChange() {
@@ -369,6 +375,7 @@ void ExportSpriteSheetCommand::onExecute(Context* context)
     docPref.spriteSheet.bestFit(window.bestFitValue());
     docPref.spriteSheet.textureFilename(window.filenameValue());
     docPref.spriteSheet.dataFilename(window.dataFilenameValue());
+    docPref.spriteSheet.openGenerated(window.openGeneratedValue());
   }
 
   m_type = docPref.spriteSheet.type();
@@ -418,11 +425,19 @@ void ExportSpriteSheetCommand::onExecute(Context* context)
   exporter.setTextureHeight(sheet_h);
   exporter.setTexturePack(true);
   exporter.addDocument(document);
-  exporter.exportSheet();
+
+  base::UniquePtr<Document> newDocument(exporter.exportSheet());
+  if (!newDocument)
+    return;
 
   StatusBar* statusbar = StatusBar::instance();
   if (statusbar)
     statusbar->showTip(1000, "Sprite Sheet Generated");
+
+  if (docPref.spriteSheet.openGenerated()) {
+    newDocument->setContext(context);
+    newDocument.release();
+  }
 }
 
 Command* CommandFactory::createExportSpriteSheetCommand()
diff --git a/src/app/document_exporter.h b/src/app/document_exporter.h
index 480f9d8da..b627f290c 100644
--- a/src/app/document_exporter.h
+++ b/src/app/document_exporter.h
@@ -95,7 +95,7 @@ namespace app {
       m_documents.push_back(Item(document, layer));
     }
 
-    void exportSheet();
+    Document* exportSheet();
 
   private:
     class Sample;