add support for drag and drop from/to shortcuts window.

5 files changed, 75 insertions, 43 deletions

diff --git a/src/gui/skilldialog.cpp b/src/gui/skilldialog.cpp
index d6fc3656c..0653a5e0d 100644
--- a/src/gui/skilldialog.cpp
+++ b/src/gui/skilldialog.cpp
@@ -168,6 +168,7 @@ class SkillListBox final : public ListBox
                         if (!skill)
                             return;
                         dragDrop.dragSkill(skill, DRAGDROP_SOURCE_SKILLS);
+                        dragDrop.setItem(skill->id + SKILL_MIN_ID);
                     }
                     ListBox::mouseDragged(event);
                 }
@@ -609,6 +610,11 @@ SkillInfo* SkillDialog::getSkill(const int id)
     return mSkills[id];
 }
 
+SkillInfo* SkillDialog::getSkillByItem(const int itemId)
+{
+    return mSkills[itemId - SKILL_MIN_ID];
+}
+
 void SkillDialog::widgetResized(const gcn::Event &event)
 {
     Window::widgetResized(event);
diff --git a/src/gui/skilldialog.h b/src/gui/skilldialog.h
index fd8728f2d..c6d491056 100644
--- a/src/gui/skilldialog.h
+++ b/src/gui/skilldialog.h
@@ -79,6 +79,8 @@ class SkillDialog final : public Window, public gcn::ActionListener
 
         SkillInfo* getSkill(const int id) A_WARN_UNUSED;
 
+        SkillInfo* getSkillByItem(const int itemId) A_WARN_UNUSED;
+
         bool hasSkills() const A_WARN_UNUSED
         { return !mSkills.empty(); }
 
diff --git a/src/gui/widgets/itemshortcutcontainer.cpp b/src/gui/widgets/itemshortcutcontainer.cpp
index 416f2092c..ec0776f36 100644
--- a/src/gui/widgets/itemshortcutcontainer.cpp
+++ b/src/gui/widgets/itemshortcutcontainer.cpp
@@ -24,6 +24,7 @@
 
 #include "client.h"
 #include "configuration.h"
+#include "dragdrop.h"
 #include "inventory.h"
 #include "inputmanager.h"
 #include "item.h"
@@ -50,7 +51,6 @@
 ItemShortcutContainer::ItemShortcutContainer(const unsigned number) :
     ShortcutContainer(),
     mItemClicked(false),
-    mItemMoved(nullptr),
     mNumber(number),
     mItemPopup(new ItemPopup),
     mSpellPopup(new SpellPopup),
@@ -222,23 +222,6 @@ void ItemShortcutContainer::draw(gcn::Graphics *graphics)
             }
         }
     }
-
-    if (mItemMoved)
-    {
-        // Draw the item image being dragged by the cursor.
-        const Image *const image = mItemMoved->getImage();
-        if (image)
-        {
-            const int tPosX = mCursorPosX - (image->mBounds.w / 2);
-            const int tPosY = mCursorPosY - (image->mBounds.h / 2);
-            const std::string str = toString(mItemMoved->getQuantity());
-
-            g->drawImage(image, tPosX, tPosY);
-            font->drawString(g, str,
-                tPosX + (mBoxWidth - font->getWidth(str)) / 2,
-                tPosY + mBoxHeight - 14);
-        }
-    }
     BLOCK_END("ItemShortcutContainer::draw")
 }
 
@@ -250,8 +233,10 @@ void ItemShortcutContainer::mouseDragged(gcn::MouseEvent &event)
 
     if (event.getButton() == gcn::MouseEvent::LEFT)
     {
-        if (!mItemMoved && mItemClicked)
+        if (dragDrop.isEmpty() && mItemClicked)
         {
+            mItemClicked = false;
+
             const int index = getIndexFromGrid(event.getX(), event.getY());
             if (index == -1)
                 return;
@@ -262,37 +247,69 @@ void ItemShortcutContainer::mouseDragged(gcn::MouseEvent &event)
             if (itemId < 0)
                 return;
 
+            event.consume();
             if (itemId < SPELL_MIN_ID)
-            {
+            {   // items
                 if (!PlayerInfo::getInventory())
                     return;
 
-                Item *const item = PlayerInfo::getInventory()->findItem(
+                const Item *const item = PlayerInfo::getInventory()->findItem(
                     itemId, itemColor);
 
                 if (item)
                 {
-                    mItemMoved = item;
                     selShortcut->removeItem(index);
+                    dragDrop.dragItem(item, DRAGDROP_SOURCE_SHORTCUTS, index);
+                }
+                else
+                {
+                    dragDrop.clear();
                 }
             }
-            else if (itemId < SKILL_MIN_ID && spellManager)
-            {
+            else if (itemId < SKILL_MIN_ID)
+            {   // spells/commands
+                if (!spellManager)
+                {
+                    dragDrop.clear();
+                    return;
+                }
+
                 const TextCommand *const spell = spellManager->getSpellByItem(
                     itemId);
                 if (spell)
+                {
                     selShortcut->removeItem(index);
+                    dragDrop.dragCommand(spell,
+                        DRAGDROP_SOURCE_SHORTCUTS, index);
+                    dragDrop.setItem(itemId);
+                }
+                else
+                {
+                    dragDrop.clear();
+                }
             }
             else
-            {
-                selShortcut->removeItem(index);
+            {   // skills
+                if (!skillDialog)
+                {
+                    dragDrop.clear();
+                    return;
+                }
+                const SkillInfo *const skill
+                    = skillDialog->getSkillByItem(itemId);
+                if (skill)
+                {
+                    selShortcut->removeItem(index);
+                    dragDrop.dragSkill(skill,
+                        DRAGDROP_SOURCE_SHORTCUTS, index);
+                    dragDrop.setItem(itemId);
+                }
+                else
+                {
+                    dragDrop.clear();
+                }
             }
         }
-        if (mItemMoved)
-        {
-            mCursorPosX = event.getX();
-            mCursorPosY = event.getY();
-        }
     }
 }
 
@@ -348,23 +365,30 @@ void ItemShortcutContainer::mouseReleased(gcn::MouseEvent &event)
 
         const int index = getIndexFromGrid(event.getX(), event.getY());
         if (index == -1)
-        {
-            mItemMoved = nullptr;
             return;
-        }
-        if (mItemMoved)
+
+        if (dragDrop.isEmpty())
         {
-            selShortcut->setItems(index, mItemMoved->getId(),
-                mItemMoved->getColor());
-            mItemMoved = nullptr;
+            if (selShortcut->getItem(index) && mItemClicked)
+                selShortcut->useItem(index);
         }
-        else if (selShortcut->getItem(index) && mItemClicked)
+        else
         {
-            selShortcut->useItem(index);
+            if (dragDrop.getSource() == DRAGDROP_SOURCE_SHORTCUTS)
+            {
+                const int oldIndex = dragDrop.getTag();
+                selShortcut->setItem(oldIndex, dragDrop.getItem(),
+                    dragDrop.getItemColor());
+                selShortcut->swap(oldIndex, index);
+            }
+            else
+            {
+                selShortcut->setItem(index, dragDrop.getItem(),
+                    dragDrop.getItemColor());
+            }
         }
 
-        if (mItemClicked)
-            mItemClicked = false;
+        mItemClicked = false;
     }
 }
 
diff --git a/src/gui/widgets/itemshortcutcontainer.h b/src/gui/widgets/itemshortcutcontainer.h
index afd421b40..5f08cdd56 100644
--- a/src/gui/widgets/itemshortcutcontainer.h
+++ b/src/gui/widgets/itemshortcutcontainer.h
@@ -82,7 +82,6 @@ class ItemShortcutContainer final : public ShortcutContainer
 
     private:
         bool mItemClicked;
-        Item *mItemMoved;
         unsigned mNumber;
 
         ItemPopup *mItemPopup;
diff --git a/src/gui/widgets/spellshortcutcontainer.cpp b/src/gui/widgets/spellshortcutcontainer.cpp
index df9f421b8..dad7f5dcf 100644
--- a/src/gui/widgets/spellshortcutcontainer.cpp
+++ b/src/gui/widgets/spellshortcutcontainer.cpp
@@ -170,6 +170,7 @@ void SpellShortcutContainer::mouseDragged(gcn::MouseEvent &event)
             if (spell)
             {
                 dragDrop.dragCommand(spell, DRAGDROP_SOURCE_SPELLS, index);
+                dragDrop.setItem(spell->getId() + SPELL_MIN_ID);
             }
             else
             {