Add custom data to Sprite Properties Window (fix #1306)

data/pref.xml

@@ -177,6 +177,9 @@
     <section id="layers">
       <option id="user_data_visibility" type="bool" default="false" />
     </section>
+    <section id="sprite">
+      <option id="user_data_visibility" type="bool" default="false" />
+    </section>
     <section id="tags">
       <option id="user_data_visibility" type="bool" default="false" />
     </section>

data/widgets/sprite_properties.xml

@@ -1,22 +1,25 @@
 <!-- Aseprite -->
-<!-- Copyright (C) 2018  Igara Studio S.A. -->
+<!-- Copyright (C) 2018-2020  Igara Studio S.A. -->
 <!-- Copyright (C) 2001-2016  David Capello -->
 <gui>
 <window id="sprite_properties" text="@.title">
   <vbox>
-    <grid columns="2">
+    <grid id="properties_grid" columns="3">
       <label text="@.filename" />
       <entry text="" id="name" maxsize="256" minwidth="64" readonly="true" cell_align="horizontal" />
+      <button id="user_data" icon="icon_user_data" maxsize="32" tooltip="@general.user_data" />
 
       <label text="@.type" />
-      <label text="" id="type" />
+      <label text="" id="type" cell_hspan="2" />
 
       <label text="@.size" />
-      <label text="" id="size" />
+      <label text="" id="size" cell_hspan="2" />
 
       <label text="@.frames" />
-      <label text="" id="frames" />
+      <label text="" id="frames" cell_hspan="2" />
+    </grid>
 
+    <grid columns="2">
       <separator text="@.advanced" horizontal="true" cell_hspan="2" />
 
       <label text="@.transparent_color" />
@@ -39,7 +42,6 @@
           <button id="convert_color_profile" text="@.convert_color_profile">Convert</button>
         </hbox>
       </hbox>
-
     </grid>
     <separator horizontal="true" />
     <hbox>

src/app/commands/cmd_sprite_properties.cpp

@@ -12,6 +12,7 @@
 #include "app/cmd/assign_color_profile.h"
 #include "app/cmd/convert_color_profile.h"
 #include "app/cmd/set_pixel_ratio.h"
+#include "app/cmd/set_user_data.h"
 #include "app/color.h"
 #include "app/commands/command.h"
 #include "app/context_access.h"
@@ -21,11 +22,13 @@
 #include "app/pref/preferences.h"
 #include "app/tx.h"
 #include "app/ui/color_button.h"
+#include "app/ui/user_data_view.h"
 #include "app/util/pixel_ratio.h"
 #include "base/mem_utils.h"
 #include "doc/image.h"
 #include "doc/palette.h"
 #include "doc/sprite.h"
+#include "doc/user_data.h"
 #include "fmt/format.h"
 #include "os/color_space.h"
 #include "os/system.h"
@@ -37,6 +40,37 @@ namespace app {
 
 using namespace ui;
 
+class SpritePropertiesWindow : public app::gen::SpriteProperties {
+public:
+  SpritePropertiesWindow(Sprite* sprite)
+    : SpriteProperties()
+    , m_sprite(sprite)
+    , m_userDataView(new gen::UserData(), &Preferences::instance().sprite.userDataVisibility)
+  {
+    userData()->Click.connect([this]{ onToggleUserData(); });
+
+    ui::Grid* mainGrid = propertiesGrid();
+    m_userDataView.configureAndSet(m_sprite->userData(), mainGrid);
+
+    remapWindow();
+    centerWindow();
+    load_window_pos(this, "SpriteProperties");
+    manager()->invalidate();
+  }
+
+  const UserData& getUserData() const { return m_userDataView.userData(); }
+
+private:
+  void onToggleUserData() {
+    m_userDataView.toggleVisibility();
+    remapWindow();
+    manager()->invalidate();
+  }
+
+  Sprite* m_sprite;
+  UserDataView m_userDataView;
+};
+
 class SpritePropertiesCommand : public Command {
 public:
   SpritePropertiesCommand();
@@ -67,7 +101,8 @@ void SpritePropertiesCommand::onExecute(Context* context)
   os::instance()->listColorSpaces(colorSpaces);
 
   // Load the window widget
-  app::gen::SpriteProperties window;
+  SpritePropertiesWindow window(context->activeDocument()->sprite());
+
   int selectedColorProfile = -1;
 
   auto updateButtons =
@@ -208,8 +243,11 @@ void SpritePropertiesCommand::onExecute(Context* context)
     PixelRatio pixelRatio =
       base::convert_to<PixelRatio>(window.pixelRatio()->getValue());
 
+    const UserData newUserData = window.getUserData();
+
     if (index != sprite->transparentColor() ||
-        pixelRatio != sprite->pixelRatio()) {
+        pixelRatio != sprite->pixelRatio() ||
+        newUserData != sprite->userData()) {
       Tx tx(writer.context(), "Change Sprite Properties");
       DocApi api = writer.document()->getApi(tx);
 
@@ -219,6 +257,9 @@ void SpritePropertiesCommand::onExecute(Context* context)
       if (pixelRatio != sprite->pixelRatio())
         tx(new cmd::SetPixelRatio(sprite, pixelRatio));
 
+      if (newUserData != sprite->userData())
+        tx(new cmd::SetUserData(sprite, newUserData, static_cast<Doc*>(sprite->document())));
+
       tx.commit();
 
       update_screen_for_document(writer.document());

src/app/file/ase_format.cpp

@@ -378,6 +378,10 @@ bool AseFormat::onSave(FileOp* fop)
 
     // Write extra chunks in the first frame
     if (frame == fop->roi().fromFrame()) {
+      // Write sprite user data only if needed
+      if (!sprite->userData().isEmpty())
+        ase_file_write_user_data_chunk(f, &frame_header, &sprite->userData());
+
       // Write tilesets
       ase_file_write_tileset_chunks(f, fop, &frame_header, ext_files,
                                     sprite->tilesets());

src/app/ui/browser_view.cpp

@@ -560,6 +560,11 @@ TabIcon BrowserView::getTabIcon()
   return TabIcon::NONE;
 }
 
+gfx::Color BrowserView::getTabColor()
+{
+  return gfx::ColorNone;
+}
+
 WorkspaceView* BrowserView::cloneWorkspaceView()
 {
   return new BrowserView();

src/app/ui/browser_view.h

@@ -25,6 +25,7 @@ namespace app {
     // TabView implementation
     std::string getTabText() override;
     TabIcon getTabIcon() override;
+    gfx::Color getTabColor() override;
 
     // WorkspaceView implementation
     ui::Widget* getContentWidget() override { return this; }

src/app/ui/data_recovery_view.cpp

@@ -378,6 +378,11 @@ TabIcon DataRecoveryView::getTabIcon()
   return TabIcon::NONE;
 }
 
+gfx::Color DataRecoveryView::getTabColor()
+{
+  return gfx::ColorNone;
+}
+
 void DataRecoveryView::onWorkspaceViewSelected()
 {
   // Do nothing

src/app/ui/data_recovery_view.h

@@ -37,6 +37,7 @@ namespace app {
     // TabView implementation
     std::string getTabText() override;
     TabIcon getTabIcon() override;
+    gfx::Color getTabColor() override;
 
     // WorkspaceView implementation
     ui::Widget* getContentWidget() override { return this; }

src/app/ui/devconsole_view.cpp

@@ -113,6 +113,11 @@ TabIcon DevConsoleView::getTabIcon()
   return TabIcon::NONE;
 }
 
+gfx::Color DevConsoleView::getTabColor()
+{
+  return gfx::ColorNone;
+}
+
 WorkspaceView* DevConsoleView::cloneWorkspaceView()
 {
   return new DevConsoleView();

src/app/ui/devconsole_view.h

@@ -32,6 +32,7 @@ namespace app {
     // TabView implementation
     std::string getTabText() override;
     TabIcon getTabIcon() override;
+    gfx::Color getTabColor() override;
 
     // WorkspaceView implementation
     ui::Widget* getContentWidget() override { return this; }

src/app/ui/doc_view.cpp

@@ -38,6 +38,7 @@
 #include "app/util/clipboard.h"
 #include "app/util/range_utils.h"
 #include "base/fs.h"
+#include "doc/color.h"
 #include "doc/layer.h"
 #include "doc/sprite.h"
 #include "fmt/format.h"
@@ -234,6 +235,12 @@ TabIcon DocView::getTabIcon()
   return TabIcon::NONE;
 }
 
+gfx::Color DocView::getTabColor()
+{
+  color_t c = m_editor->sprite()->userData().color();
+  return gfx::rgba(doc::rgba_getr(c), doc::rgba_getg(c), doc::rgba_getb(c), doc::rgba_geta(c));
+}
+
 WorkspaceView* DocView::cloneWorkspaceView()
 {
   return new DocView(m_document, Normal, m_previewDelegate);

src/app/ui/doc_view.h

@@ -58,6 +58,7 @@ namespace app {
     // TabView implementation
     std::string getTabText() override;
     TabIcon getTabIcon() override;
+    gfx::Color getTabColor() override;
 
     // WorkspaceView implementation
     ui::Widget* getContentWidget() override { return this; }

src/app/ui/home_view.cpp

@@ -111,6 +111,11 @@ TabIcon HomeView::getTabIcon()
   return TabIcon::HOME;
 }
 
+gfx::Color HomeView::getTabColor()
+{
+  return gfx::ColorNone;
+}
+
 bool HomeView::onCloseView(Workspace* workspace, bool quitting)
 {
   workspace->removeView(this);

src/app/ui/home_view.h

@@ -49,6 +49,7 @@ namespace app {
     // TabView implementation
     std::string getTabText() override;
     TabIcon getTabIcon() override;
+    gfx::Color getTabColor() override;
 
     // WorkspaceView implementation
     ui::Widget* getContentWidget() override { return this; }

src/app/ui/tabs.cpp

@@ -174,6 +174,7 @@ void Tabs::updateTabs()
 
     tab->text = tab->view->getTabText();
     tab->icon = tab->view->getTabIcon();
+    tab->color = tab->view->getTabColor();
     tab->x = int(x);
     tab->width = int(x+tabWidth) - int(x);
     x += tabWidth;
@@ -626,6 +627,10 @@ void Tabs::drawTab(Graphics* g, const gfx::Rect& _box,
     // Tab with text + clipping the close button
     if (box.w > 8*ui::guiscale()) {
       info.text = &tab->text;
+      if (tab->color != gfx::ColorNone) {
+        g->fillRect(tab->color, gfx::Rect(box.x+dx+2, box.y+dy+3, box.w-dx-2, box.h-3));
+        g->fillRect(tab->color, gfx::Rect(box.x+dx+3, box.y+dy+2, box.w-dx-3, 1));
+      }
       theme->paintWidgetPart(
         g, theme->styles.tabText(),
         gfx::Rect(box.x+dx, box.y+dy, box.w-dx, box.h),
@@ -656,6 +661,11 @@ void Tabs::drawTab(Graphics* g, const gfx::Rect& _box,
       }
     }
 
+    if (tab->color != gfx::ColorNone) {
+      g->fillRect(tab->color, gfx::Rect(closeBox.x, closeBox.y+3, closeBox.w-3, closeBox.h-3));
+      g->fillRect(tab->color, gfx::Rect(closeBox.x, closeBox.y+2, closeBox.w-4, 1));
+    }
+
     info.styleFlags = 0;
     if (selected)
       info.styleFlags |= ui::Style::Layer::kFocus;

src/app/ui/tabs.h

@@ -41,6 +41,9 @@ namespace app {
 
     // Returns the icon to be shown in the tab
     virtual TabIcon getTabIcon() = 0;
+
+    // Returns the tab background color
+    virtual gfx::Color getTabColor() = 0;
   };
 
   enum class DropTabResult {
@@ -116,6 +119,7 @@ namespace app {
       TabView* view;
       std::string text;
       TabIcon icon;
+      gfx::Color color;
       int x, width;
       int oldX, oldWidth;
       bool modified;
@@ -124,6 +128,7 @@ namespace app {
         ASSERT(view);
         text = view->getTabText();
         icon = view->getTabIcon();
+        color = view->getTabColor();
 
         x = width = oldX = oldWidth =
 #if _DEBUG

src/dio/aseprite_decoder.cpp

@@ -82,7 +82,7 @@ bool AsepriteDecoder::decode()
 
   // Prepare variables for layer chunks
   doc::Layer* last_layer = sprite->root();
-  doc::WithUserData* last_object_with_user_data = nullptr;
+  doc::WithUserData* last_object_with_user_data = sprite.get();
   doc::Cel* last_cel = nullptr;
   auto tag_it = sprite->tags().begin();
   auto tag_end = sprite->tags().end();

src/doc/sprite.cpp

@@ -66,7 +66,7 @@ void Sprite::SetDefaultRgbMapAlgorithm(const RgbMapAlgorithm mapAlgo)
 
 Sprite::Sprite(const ImageSpec& spec,
                int ncolors)
-  : Object(ObjectType::Sprite)
+  : WithUserData(ObjectType::Sprite)
   , m_document(nullptr)
   , m_spec(spec)
   , m_pixelRatio(1, 1)

src/doc/sprite.h

@@ -24,6 +24,7 @@
 #include "doc/rgbmap_algorithm.h"
 #include "doc/slices.h"
 #include "doc/tags.h"
+#include "doc/with_user_data.h"
 #include "gfx/rect.h"
 
 #include <memory>
@@ -52,7 +53,7 @@ namespace doc {
   typedef std::vector<Palette*> PalettesList;
 
   // The main structure used in the whole program to handle a sprite.
-  class Sprite : public Object {
+  class Sprite : public WithUserData {
   public:
     enum class RgbMapFor {
       OpaqueLayer,