From 19951c6dd5c7a8cf3f546b2019a88c230e78629a Mon Sep 17 00:00:00 2001
From: David Capello <davidcapello@gmail.com>
Date: Tue, 25 Aug 2015 13:59:15 -0300
Subject: [PATCH] New Keyboard Shortcuts dialog layout (fix #777)

---
 data/widgets/keyboard_shortcuts.xml         | 28 ++++++++++-----------
 src/app/commands/cmd_keyboard_shortcuts.cpp | 11 ++++----
 2 files changed, 20 insertions(+), 19 deletions(-)

diff --git a/data/widgets/keyboard_shortcuts.xml b/data/widgets/keyboard_shortcuts.xml
index d1727299f..ed4ecc991 100644
--- a/data/widgets/keyboard_shortcuts.xml
+++ b/data/widgets/keyboard_shortcuts.xml
@@ -1,13 +1,22 @@
 <!-- Aseprite -->
-<!-- Copyright (C) 2001-2014 by David Capello -->
+<!-- Copyright (C) 2001-2015 by David Capello -->
 <gui>
   <window id="keyboard_shortcuts" text="Keyboard Shortcuts">
     <hbox>
+      <vbox>
+        <view expansive="true" width="80">
+          <listbox id="section" expansive="true" />
+        </view>
+        <separator horizontal="true" />
+        <button text="&amp;OK" closewindow="true" id="ok" magnet="true" width="60" />
+        <button text="&amp;Cancel" closewindow="true" />
+        <separator horizontal="true" />
+        <button text="&amp;Import" id="import_button" />
+        <button text="&amp;Export" id="export_button" />
+        <separator horizontal="true" />
+        <button text="&amp;Reset" id="reset_button" />
+      </vbox>
       <vbox expansive="true">
-        <hbox>
-          <label text="Section:" />
-          <combobox id="section" expansive="true" />
-        </hbox>
         <view id="menus_view" expansive="true">
           <listbox id="menus" />
         </view>
@@ -21,15 +30,6 @@
           <listbox id="actions" />
         </view>
       </vbox>
-      <vbox>
-        <button text="&amp;OK" closewindow="true" id="ok" magnet="true" width="60" />
-        <button text="&amp;Cancel" closewindow="true" />
-        <separator horizontal="true" />
-        <button text="&amp;Import" id="import_button" />
-        <button text="&amp;Export" id="export_button" />
-        <separator horizontal="true" />
-        <button text="&amp;Reset" id="reset_button" />
-      </vbox>
     </hbox>
   </window>
 </gui>
diff --git a/src/app/commands/cmd_keyboard_shortcuts.cpp b/src/app/commands/cmd_keyboard_shortcuts.cpp
index 3e99250e3..b2f1d4f07 100644
--- a/src/app/commands/cmd_keyboard_shortcuts.cpp
+++ b/src/app/commands/cmd_keyboard_shortcuts.cpp
@@ -283,10 +283,10 @@ public:
   KeyboardShortcutsWindow() {
     setAutoRemap(false);
 
-    section()->addItem("Menus");
-    section()->addItem("Commands");
-    section()->addItem("Tools");
-    section()->addItem("Action Modifiers");
+    section()->addChild(new ListItem("Menus"));
+    section()->addChild(new ListItem("Commands"));
+    section()->addChild(new ListItem("Tools"));
+    section()->addChild(new ListItem("Action Modifiers"));
 
     section()->Change.connect(Bind<void>(&KeyboardShortcutsWindow::onSectionChange, this));
     importButton()->Click.connect(Bind<void>(&KeyboardShortcutsWindow::onImport, this));
@@ -365,11 +365,12 @@ private:
     this->tools()->sortItems();
     this->actions()->sortItems();
 
+    section()->selectIndex(0);
     onSectionChange();
   }
 
   void onSectionChange() {
-    int section = this->section()->getSelectedItemIndex();
+    int section = this->section()->getSelectedIndex();
 
     menusView()->setVisible(section == 0);
     commandsView()->setVisible(section == 1);