From 76edbda15cb5bb4a2acaac2af9f2692c20fedd0e Mon Sep 17 00:00:00 2001
From: David Capello <davidcapello@gmail.com>
Date: Fri, 16 Dec 2016 11:07:43 -0300
Subject: [PATCH 1/6] Fix "new version available" button size (fix #1354)

Regression introduced in 12726fedf2be5f697cffae86d77337bd16d9a738
---
 data/skins/default/skin.xml    |  5 +++++
 src/app/ui/browser_view.cpp    |  9 ++++++++-
 src/app/ui/skin/skin_theme.cpp | 10 ++++------
 3 files changed, 17 insertions(+), 7 deletions(-)

diff --git a/data/skins/default/skin.xml b/data/skins/default/skin.xml
index bebf0e2e9..5f56c588d 100644
--- a/data/skins/default/skin.xml
+++ b/data/skins/default/skin.xml
@@ -436,6 +436,11 @@
       <text color="link_hover" />
     </style>
 
+    <!-- browser_link -->
+    <style id="browser_link" base="link">
+      <text padding-top="1" />
+    </style>
+
     <!-- view -->
     <style id="view">
       <background part="sunken_normal" />
diff --git a/src/app/ui/browser_view.cpp b/src/app/ui/browser_view.cpp
index cc13dfc01..fecabeb54 100644
--- a/src/app/ui/browser_view.cpp
+++ b/src/app/ui/browser_view.cpp
@@ -447,8 +447,12 @@ private:
         Label* label;
 
         if (word.size() > 4 &&
-            std::strncmp(word.c_str(), "http", 4) == 0)
+            std::strncmp(word.c_str(), "http", 4) == 0) {
           label = new LinkLabel(word);
+          label->setProperty(
+            SkinStylePropertyPtr(
+              new SkinStyleProperty(SkinTheme::instance()->styles.browserLink())));
+        }
         else
           label = new Label(word);
 
@@ -474,6 +478,9 @@ private:
 
   void addLink(const std::string& url, const std::string& text) {
     auto label = new LinkLabel(url, text);
+    label->setProperty(
+      SkinStylePropertyPtr(
+        new SkinStyleProperty(SkinTheme::instance()->styles.browserLink())));
 
     if (url.find(':') == std::string::npos) {
       label->setUrl("");
diff --git a/src/app/ui/skin/skin_theme.cpp b/src/app/ui/skin/skin_theme.cpp
index 4f2f310c4..d405635ea 100644
--- a/src/app/ui/skin/skin_theme.cpp
+++ b/src/app/ui/skin/skin_theme.cpp
@@ -1050,17 +1050,15 @@ void SkinTheme::paintLinkLabel(PaintEvent& ev)
   if (styleProp)
     style = styleProp->getStyle();
 
-  if (!is_transparent(bg))
-    g->fillRect(bg, rc);
-  rc.shrink(widget->border());
-
   Style::State state;
   if (widget->hasMouseOver()) state += Style::hover();
   if (widget->isSelected()) state += Style::clicked();
   if (!widget->isEnabled()) state += Style::disabled();
 
-  widget->getTextIconInfo(nullptr, &text);
-  style->paint(g, text, widget->text().c_str(), state);
+  if (!is_transparent(bg))
+    g->fillRect(bg, rc);
+
+  style->paint(g, rc, widget->text().c_str(), state);
 }
 
 void SkinTheme::paintListBox(PaintEvent& ev)

From 447b2701e49360e7195fbeb1461fd06bae656b5c Mon Sep 17 00:00:00 2001
From: David Capello <davidcapello@gmail.com>
Date: Fri, 16 Dec 2016 11:35:56 -0300
Subject: [PATCH 2/6] Update version to 1.1.12-dev

---
 data/gui.xml                | 2 +-
 src/config.h                | 2 +-
 src/main/resources_win32.rc | 8 ++++----
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/data/gui.xml b/data/gui.xml
index d5507c7b9..19d5de131 100644
--- a/data/gui.xml
+++ b/data/gui.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!-- Aseprite -->
 <!-- Copyright (C) 2001-2016 by David Capello -->
-<gui version="1.1.10-dev">
+<gui version="1.1.12-dev">
   <!-- Keyboard shortcuts -->
   <keyboard version="1">
 
diff --git a/src/config.h b/src/config.h
index 534e655cb..0f69b9f30 100644
--- a/src/config.h
+++ b/src/config.h
@@ -23,7 +23,7 @@
 
 // General information
 #define PACKAGE "Aseprite"
-#define VERSION "1.1.10-dev"
+#define VERSION "1.1.12-dev"
 
 #ifdef CUSTOM_WEBSITE_URL
 #define WEBSITE                 CUSTOM_WEBSITE_URL // To test web server
diff --git a/src/main/resources_win32.rc b/src/main/resources_win32.rc
index 4a22c5b67..81cde6137 100644
--- a/src/main/resources_win32.rc
+++ b/src/main/resources_win32.rc
@@ -3,8 +3,8 @@ allegro_icon ICON data/icons/ase.ico
 1 ICON data/icons/doc.ico
 
 1 VERSIONINFO
-  FILEVERSION 1,1,10,0
-  PRODUCTVERSION 1,1,10,0
+  FILEVERSION 1,1,12,0
+  PRODUCTVERSION 1,1,12,0
   FILEFLAGSMASK 0x3fL
 #ifdef _DEBUG
   FILEFLAGS 0x1L
@@ -21,12 +21,12 @@ BEGIN
         BEGIN
             VALUE "Comments", "http://www.aseprite.org/"
             VALUE "FileDescription", "Aseprite - Animated sprites editor & pixel art tool"
-            VALUE "FileVersion", "1,1,10,0"
+            VALUE "FileVersion", "1,1,12,0"
             VALUE "InternalName", "aseprite"
             VALUE "LegalCopyright", "Copyright (C) 2001-2016 by David Capello"
             VALUE "OriginalFilename", "aseprite.exe"
             VALUE "ProductName", "ASEPRITE"
-            VALUE "ProductVersion", "1,1,10,0"
+            VALUE "ProductVersion", "1,1,12,0"
         END
     END
     BLOCK "VarFileInfo"

From f8874f4f7b41a73db16c647aeaa7f95306c360ec Mon Sep 17 00:00:00 2001
From: David Capello <davidcapello@gmail.com>
Date: Fri, 16 Dec 2016 14:53:26 -0300
Subject: [PATCH 3/6] Fix zoom entry so we can insert numbers with 4 digits

---
 src/app/ui/zoom_entry.cpp | 3 ++-
 src/ui/int_entry.cpp      | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/app/ui/zoom_entry.cpp b/src/app/ui/zoom_entry.cpp
index 2fd5f48e2..8f82cc37b 100644
--- a/src/app/ui/zoom_entry.cpp
+++ b/src/app/ui/zoom_entry.cpp
@@ -31,7 +31,8 @@ using namespace gfx;
 using namespace ui;
 
 ZoomEntry::ZoomEntry()
-  : IntEntry(0, render::Zoom::linearValues()-1, this)
+  : IntEntry(
+    0, render::Zoom::fromLinearScale(render::Zoom::linearValues()-1).scale()*100, this)
   , m_locked(false)
 {
   setSuffix("%");
diff --git a/src/ui/int_entry.cpp b/src/ui/int_entry.cpp
index c7e80c2c8..9d3114d73 100644
--- a/src/ui/int_entry.cpp
+++ b/src/ui/int_entry.cpp
@@ -29,7 +29,7 @@ namespace ui {
 using namespace gfx;
 
 IntEntry::IntEntry(int min, int max, SliderDelegate* sliderDelegate)
-  : Entry(int(std::ceil(std::log10((double)max)))+1, "")
+  : Entry(int(std::floor(std::log10(double(max))))+1, "")
   , m_min(min)
   , m_max(max)
   , m_slider(m_min, m_max, m_min, sliderDelegate)

From 7ae63156c5eb459754b855671b59af4127424527 Mon Sep 17 00:00:00 2001
From: David Capello <davidcapello@gmail.com>
Date: Sun, 18 Dec 2016 14:44:39 -0300
Subject: [PATCH 4/6] Add option to disable Home tab (fix #1368)

---
 data/pref.xml                    | 1 +
 data/widgets/options.xml         | 1 +
 src/app/commands/cmd_options.cpp | 6 ++++++
 src/app/ui/main_window.cpp       | 6 ++++++
 4 files changed, 14 insertions(+)

diff --git a/data/pref.xml b/data/pref.xml
index ccd0a8003..6cdd800fd 100644
--- a/data/pref.xml
+++ b/data/pref.xml
@@ -95,6 +95,7 @@
       <option id="screen_scale" type="int" default="0" />
       <option id="ui_scale" type="int" default="1" migrate="experimental.ui_scale" />
       <option id="gpu_acceleration" type="bool" default="false" />
+      <option id="show_home" type="bool" default="true" />
       <option id="visible_timeline" type="bool" default="false" />
       <option id="autoshow_timeline" type="bool" default="true" migrate="Options.AutoShowTimeline" />
       <option id="rewind_on_stop" type="bool" default="false" />
diff --git a/data/widgets/options.xml b/data/widgets/options.xml
index 8b684b41b..dfefba14d 100644
--- a/data/widgets/options.xml
+++ b/data/widgets/options.xml
@@ -40,6 +40,7 @@
             </grid>
           </hbox>
           <check text="GPU acceleration" id="gpu_acceleration" tooltip="Check this option to enable hardware acceleration" />
+          <check text="Show Home tab when Aseprite is started" id="show_home" />
           <check text="Expand menu bar items on mouseover" id="expand_menubar_on_mouseover" tooltip="Check this option to get&#10;this old menus behavior." />
           <hbox>
             <check text="Automatically save recovery data every" id="enable_data_recovery" tooltip="With this option you can recover your documents&#10;if the program finalizes unexpectedly." />
diff --git a/src/app/commands/cmd_options.cpp b/src/app/commands/cmd_options.cpp
index 4ebcc0fb9..9be49b7e8 100644
--- a/src/app/commands/cmd_options.cpp
+++ b/src/app/commands/cmd_options.cpp
@@ -194,6 +194,8 @@ public:
       gpuAcceleration()->setVisible(false);
     }
 
+    showHome()->setSelected(m_pref.general.showHome());
+
     // Right-click
 
     static_assert(int(app::gen::RightClickMode::PAINT_BGCOLOR) == 0, "");
@@ -353,6 +355,10 @@ public:
       reset_screen = true;
     }
 
+    bool newShowHome = showHome()->isSelected();
+    if (newShowHome != m_pref.general.showHome())
+      m_pref.general.showHome(newShowHome);
+
     m_pref.save();
 
     if (!warnings.empty()) {
diff --git a/src/app/ui/main_window.cpp b/src/app/ui/main_window.cpp
index 98c63ebf5..472eeb17d 100644
--- a/src/app/ui/main_window.cpp
+++ b/src/app/ui/main_window.cpp
@@ -229,6 +229,12 @@ void MainWindow::showNotification(INotificationDelegate* del)
 
 void MainWindow::showHomeOnOpen()
 {
+  // Don't open Home tab
+  if (!Preferences::instance().general.showHome()) {
+    configureWorkspaceLayout();
+    return;
+  }
+
   if (!getHomeView()->parent()) {
     TabView* selectedTab = m_tabsBar->getSelectedTab();
 

From 52fbbe844c82dbc5ad28ff9d2bd0e0fa16aa57d7 Mon Sep 17 00:00:00 2001
From: "Azamat H. Hackimov" <azamat.hackimov@gmail.com>
Date: Mon, 19 Dec 2016 22:19:30 +0500
Subject: [PATCH 5/6] Control for undo tests

---
 CMakeLists.txt | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 68d4f0fc0..0abc55fef 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -451,6 +451,8 @@ if(NOT USE_STATIC_LIBC)
 endif()
 
 set(LAF_WITH_TESTS ${ENABLE_TESTS} CACHE BOOL "Enable LAF tests")
+set(UNDO_TESTS     ${ENABLE_TESTS} CACHE BOOL "Enable undo tests")
+
 add_subdirectory(laf)
 
 add_subdirectory(src)

From 809a90ba3b3fabceec0834e2b68db6d4f430d27c Mon Sep 17 00:00:00 2001
From: David Capello <davidcapello@gmail.com>
Date: Tue, 20 Dec 2016 16:59:07 -0300
Subject: [PATCH 6/6] Fix crash using focus-movement keys in strange UI state

There were some crash reports in these days on the focus movement
code. It looks like we could arrive into an invalid/strange UI state
double-clicking palette buttons and showing menus two consecutive
times. After that, pressing Tab key would crash the focus-movement
logic.
---
 src/ui/manager.cpp |  3 +-
 src/ui/menu.cpp    | 85 ++++++++++++++++++++++++++++++----------------
 src/ui/menu.h      |  3 ++
 3 files changed, 60 insertions(+), 31 deletions(-)

diff --git a/src/ui/manager.cpp b/src/ui/manager.cpp
index 060c9b368..f70328f87 100644
--- a/src/ui/manager.cpp
+++ b/src/ui/manager.cpp
@@ -1671,7 +1671,8 @@ static Widget* next_widget(Widget* widget)
   if (!widget->children().empty())
     return UI_FIRST_WIDGET(widget->children());
 
-  while (widget->parent()->type() != kManagerWidget) {
+  while (widget->parent() &&
+         widget->parent()->type() != kManagerWidget) {
     WidgetsList::const_iterator begin = widget->parent()->children().begin();
     WidgetsList::const_iterator end = widget->parent()->children().end();
     WidgetsList::const_iterator it = std::find(begin, end, widget);
diff --git a/src/ui/menu.cpp b/src/ui/menu.cpp
index 0cd4f3e81..9bb233040 100644
--- a/src/ui/menu.cpp
+++ b/src/ui/menu.cpp
@@ -64,8 +64,7 @@ private:
 };
 
 // Data for the main jmenubar or the first popuped-jmenubox
-struct MenuBaseData
-{
+struct MenuBaseData {
   // True when the menu-items must be opened with the cursor movement
   bool was_clicked;
 
@@ -79,8 +78,7 @@ struct MenuBaseData
 
   bool close_all;
 
-  MenuBaseData()
-  {
+  MenuBaseData() {
     was_clicked = false;
     is_filtering = false;
     is_processing = false;
@@ -158,11 +156,7 @@ MenuBox::MenuBox(WidgetType type)
 
 MenuBox::~MenuBox()
 {
-  if (m_base && m_base->is_filtering) {
-    m_base->is_filtering = false;
-    Manager::getDefault()->removeMessageFilter(kMouseDownMessage, this);
-  }
-
+  stopFilteringMouseDown();
   delete m_base;
 }
 
@@ -219,7 +213,7 @@ Menu* MenuBox::getMenu()
 MenuBaseData* MenuBox::createBase()
 {
   delete m_base;
-  m_base = new MenuBaseData();
+  m_base = new MenuBaseData;
   return m_base;
 }
 
@@ -269,18 +263,29 @@ bool MenuItem::hasSubmenu() const
 
 void Menu::showPopup(const gfx::Point& pos)
 {
+  // Generally, when we call showPopup() the menu shouldn't contain a
+  // parent menu-box, because we're filtering kMouseDownMessage to
+  // close the popup automatically when we click outside the menubox.
+  // Anyway there is one specific case were a clicked widget might
+  // call showPopup() when it's clicked the first time, and a second
+  // click could generate a kDoubleClickMessage which is then
+  // converted to kMouseDownMessage to finally call showPopup() again.
+  // In this case, the menu is already in a menubox.
+  if (parent()) {
+    static_cast<MenuBox*>(parent())->cancelMenuLoop();
+    return;
+  }
+
   // New window and new menu-box
-  Window* window = new Window(Window::WithoutTitleBar);
+  base::UniquePtr<Window> window(new Window(Window::WithoutTitleBar));
   MenuBox* menubox = new MenuBox();
   MenuBaseData* base = menubox->createBase();
   base->was_clicked = true;
-  base->is_filtering = true;
-  Manager::getDefault()->addMessageFilter(kMouseDownMessage, menubox);
-
   window->setMoveable(false);   // Can't move the window
 
   // Set children
   menubox->setMenu(this);
+  menubox->startFilteringMouseDown();
   window->addChild(menubox);
 
   window->remapWindow();
@@ -301,10 +306,9 @@ void Menu::showPopup(const gfx::Point& pos)
   Manager::getDefault()->freeFocus();
 
   // Fetch the "menu" so it isn't destroyed
-  menubox->setMenu(NULL);
+  menubox->setMenu(nullptr);
+  menubox->stopFilteringMouseDown();
 
-  // Destroy the window
-  delete window;
 }
 
 void Menu::onPaint(PaintEvent& ev)
@@ -374,7 +378,10 @@ bool MenuBox::onProcessMessage(Message* msg)
       // Fall through
 
     case kMouseDownMessage:
+    case kDoubleClickMessage:
       if (menu) {
+        ASSERT(menu->parent() == this);
+
         if (get_base(this)->is_processing)
           break;
 
@@ -650,7 +657,7 @@ bool MenuBox::onProcessMessage(Message* msg)
 
     default:
       if (msg->type() == kClosePopupMessage) {
-        manager()->_closeWindow(window(), true);
+        window()->closeWindow(nullptr);
       }
       break;
 
@@ -829,9 +836,9 @@ bool MenuItem::onProcessMessage(Message* msg)
         else
           manager()->setFocus(this->parent()->parent());
 
-        // It is not necessary to delete this window because it's
-        // automatically destroyed by the manager
-        // ... delete window;
+        // Do not call "delete window" here, because it
+        // (CustomizedWindowForMenuBox) will be deferDelete()d on
+        // kCloseMessage.
 
         if (last_of_close_chain) {
           base->close_all = false;
@@ -928,7 +935,11 @@ static MenuBox* get_base_menubox(Widget* widget)
 static MenuBaseData* get_base(Widget* widget)
 {
   MenuBox* menubox = get_base_menubox(widget);
-  return menubox->getBase();
+  ASSERT(menubox);
+  if (menubox)
+    return menubox->getBase();
+  else
+    return nullptr;
 }
 
 MenuItem* Menu::getHighlightedItem()
@@ -1048,10 +1059,9 @@ void MenuItem::openSubmenu(bool select_first)
   // clicks outside the menu (and close all the hierarchy in that
   // case); the widget to intercept messages is the base menu-bar or
   // popuped menu-box
-  if (!base->is_filtering) {
-    base->is_filtering = true;
-    Manager::getDefault()->addMessageFilter(kMouseDownMessage, get_base_menubox(this));
-  }
+  MenuBox* base_menubox = get_base_menubox(this);
+  if (base_menubox)
+    base_menubox->startFilteringMouseDown();
 }
 
 void MenuItem::closeSubmenu(bool last_of_close_chain)
@@ -1128,10 +1138,7 @@ void Menu::closeAll()
 
   base->close_all = true;
   base->was_clicked = false;
-  if (base->is_filtering) {
-    base->is_filtering = false;
-    Manager::getDefault()->removeMessageFilter(kMouseDownMessage, base_menubox);
-  }
+  base_menubox->stopFilteringMouseDown();
 
   menu->unhighlightItem();
 
@@ -1162,6 +1169,24 @@ void MenuBox::closePopup()
   Manager::getDefault()->enqueueMessage(msg);
 }
 
+void MenuBox::startFilteringMouseDown()
+{
+  if (m_base && !m_base->is_filtering) {
+    m_base->is_filtering = true;
+    Manager::getDefault()->addMessageFilter(kMouseDownMessage, this);
+    Manager::getDefault()->addMessageFilter(kDoubleClickMessage, this);
+  }
+}
+
+void MenuBox::stopFilteringMouseDown()
+{
+  if (m_base && m_base->is_filtering) {
+    m_base->is_filtering = false;
+    Manager::getDefault()->removeMessageFilter(kMouseDownMessage, this);
+    Manager::getDefault()->removeMessageFilter(kDoubleClickMessage, this);
+  }
+}
+
 void MenuBox::cancelMenuLoop()
 {
   Menu* menu = getMenu();
diff --git a/src/ui/menu.h b/src/ui/menu.h
index 01ec0d2b6..eed627d8d 100644
--- a/src/ui/menu.h
+++ b/src/ui/menu.h
@@ -78,10 +78,13 @@ namespace ui {
 
   private:
     void closePopup();
+    void startFilteringMouseDown();
+    void stopFilteringMouseDown();
 
     MenuBaseData* m_base;
 
     friend class Menu;
+    friend class MenuItem;
   };
 
   class MenuBar : public MenuBox {