Adding drag-n-drop behavior

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

src/common/hotbar-store.ts

@@ -158,6 +158,20 @@ export class HotbarStore extends BaseStore<HotbarStoreModel> {
     hotbar.items[index] = null;
   }
 
+  swapItems(from: number, to: number): void {
+    const { items } = this.getActive();
+
+    if (from < 0 || to < 0 || from >= items.length || to >= items.length || isNaN(from) || isNaN(to)) {
+      throw new Error("Invalid 'from' or 'to' arguments");
+    }
+
+    if (from != to) {
+      const source = items.splice(from, 1);
+
+      items.splice(to, 0, source[0]);
+    }
+  }
+
   switchToPrevious() {
     const hotbarStore = HotbarStore.getInstance();
     let index = hotbarStore.activeHotbarIndex - 1;

src/renderer/components/hotbar/hotbar-menu.tsx

@@ -6,9 +6,9 @@ import { observer } from "mobx-react";
 import { HotbarIcon } from "./hotbar-icon";
 import { cssNames, IClassName } from "../../utils";
 import { catalogEntityRegistry } from "../../api/catalog-entity-registry";
-import { HotbarItem, HotbarStore } from "../../../common/hotbar-store";
+import { defaultHotbarCells, HotbarItem, HotbarStore } from "../../../common/hotbar-store";
 import { CatalogEntity, catalogEntityRunContext } from "../../api/catalog-entity";
-import { DragDropContext, Draggable, DraggableProvided, Droppable, DroppableProvided, DropResult } from "react-beautiful-dnd";
+import { DragDropContext, Draggable, Droppable, DropResult } from "react-beautiful-dnd";
 import { HotbarSelector } from "./hotbar-selector";
 
 interface Props {
@@ -35,8 +35,17 @@ export class HotbarMenu extends React.Component<Props> {
     return item ? catalogEntityRegistry.items.find((entity) => entity.metadata.uid === item.entity.uid) : null;
   }
 
-  onDragEnd() {
+  onDragEnd(result: DropResult) {
-    console.log("drag end")
+    const { source, destination } = result;
+
+    if (!destination) {  // Dropped outside of the list
+      return;
+    }
+
+    const from = parseInt(source.droppableId);
+    const to = parseInt(destination.droppableId);
+
+    HotbarStore.getInstance().swapItems(from, to);
   }
 
   renderGrid() {
@@ -44,33 +53,46 @@ export class HotbarMenu extends React.Component<Props> {
       const entity = this.getEntity(item);
 
       return (
-        <Droppable droppableId={`droppable-${index}`} key={index} isCombineEnabled>
+        <Droppable droppableId={`${index}`} key={index}>
           {(provided, snapshot) => (
             <HotbarCell
               index={index}
-              ref={provided.innerRef}
+              key={entity ? entity.getId() : `cell${index}`}
-              style={{ backgroundColor: snapshot.isDraggingOver ? 'blue' : 'grey' }}
+              innerRef={provided.innerRef}
+              className={cssNames({ isDraggingOver: snapshot.isDraggingOver })}
               {...provided.droppableProps}
             >
               {entity && (
-                <Draggable draggableId={`draggable-${index}`} index={0}>
+                <Draggable draggableId={item.entity.uid} key={item.entity.uid} index={0}>
-                  {(provided, snapshot) => (
+                  {(provided, snapshot) => {
-                    <div
+                    const style = {
-                      ref={provided.innerRef}
+                      zIndex: defaultHotbarCells - index,
-                      {...provided.draggableProps}
+                      position: "absolute",
-                      {...provided.dragHandleProps}
+                      ...provided.draggableProps.style,
-                    >
+                    } as React.CSSProperties;
-                      <HotbarIcon
+
-                        key={index}
+                    return (
-                        index={index}
+                      <div
-                        entity={entity}
+                        key={item.entity.uid}
-                        isActive={this.isActive(entity)}
+                        ref={provided.innerRef}
-                        onClick={() => entity.onRun(catalogEntityRunContext)}
+                        {...provided.draggableProps}
-                      />
+                        {...provided.dragHandleProps}
-                    </div>
+                        style={style}
-                  )}
+                      >
+                        <HotbarIcon
+                          key={index}
+                          index={index}
+                          entity={entity}
+                          isActive={this.isActive(entity)}
+                          onClick={() => entity.onRun(catalogEntityRunContext)}
+                          className={cssNames({ isDragging: snapshot.isDragging })}
+                        />
+                      </div>
+                    );
+                  }}
                 </Draggable>
               )}
+              {provided.placeholder}
             </HotbarCell>
           )}
         </Droppable>
@@ -99,21 +121,29 @@ export class HotbarMenu extends React.Component<Props> {
 interface HotbarCellProps extends HTMLAttributes<HTMLDivElement> {
   children?: ReactNode;
   index: number;
-  ref?: any;
+  innerRef?: React.LegacyRef<HTMLDivElement>;
 }
 
-function HotbarCell(props: HotbarCellProps) {
+function HotbarCell({ innerRef, children, className, ...rest }: HotbarCellProps) {
   const [animating, setAnimating] = useState(false);
   const onAnimationEnd = () => { setAnimating(false); };
-  const onClick = () => { setAnimating(true); };
+  const onClick = () => {
+    if (className.includes("isDraggingOver")) {
+      return;
+    }
+
+    setAnimating(true);
+  };
 
   return (
     <div
-      className={cssNames("HotbarCell", { animating })}
+      className={cssNames("HotbarCell", { animating }, className)}
       onAnimationEnd={onAnimationEnd}
       onClick={onClick}
+      ref={innerRef}
+      {...rest}
     >
-      {props.children}
+      {children}
     </div>
   );
 }

src/renderer/components/hotbar/hotbar-selector.tsx

@@ -11,9 +11,8 @@ interface Props {
   hotbar: Hotbar;
 }
 
-const store = HotbarStore.getInstance();
-
 export function HotbarSelector({ hotbar }: Props) {
+  const store = HotbarStore.getInstance();
   const activeIndexDisplay = store.activeHotbarIndex + 1;
 
   return (