Delete image button

Signed-off-by: marcin mikołajczak <git@mkljczk.pl>

app/soapbox/features/compose/editor/index.tsx

@@ -152,7 +152,7 @@ const ComposeEditor = React.forwardRef<string, IComposeEditor>(({
           contentEditable={
             <div className='editor' ref={onRef} onFocus={onFocus} onPaste={handlePaste}>
               <ContentEditable
-                className={clsx('mr-4 outline-none transition-[min-height] motion-reduce:transition-none', {
+                className={clsx('mr-4 pb-8 outline-none transition-[min-height] motion-reduce:transition-none', {
                   'min-h-[40px]': condensed,
                   'min-h-[100px]': !condensed,
                 })}

app/soapbox/features/compose/editor/nodes/image-component.tsx

@@ -10,6 +10,7 @@
 import { useLexicalComposerContext } from '@lexical/react/LexicalComposerContext';
 import { useLexicalNodeSelection } from '@lexical/react/useLexicalNodeSelection';
 import { mergeRegister } from '@lexical/utils';
+import clsx from 'clsx';
 import {
   $getNodeByKey,
   $getSelection,
@@ -27,6 +28,8 @@ import {
 import * as React from 'react';
 import { Suspense, useCallback, useEffect, useRef, useState } from 'react';
 
+import { IconButton } from 'soapbox/components/ui';
+
 import { $isImageNode } from './image-node';
 
 import type {
@@ -94,15 +97,24 @@ export default function ImageComponent({
   >(null);
   const activeEditorRef = useRef<LexicalEditor | null>(null);
 
+  const deleteNode = useCallback(
+    () => {
+      editor.update(() => {
+        const node = $getNodeByKey(nodeKey);
+        if ($isImageNode(node)) {
+          node.remove();
+        }
+      });
+    },
+    [nodeKey],
+  );
+
   const onDelete = useCallback(
     (payload: KeyboardEvent) => {
       if (isSelected && $isNodeSelection($getSelection())) {
         const event: KeyboardEvent = payload;
         event.preventDefault();
-        const node = $getNodeByKey(nodeKey);
+        deleteNode();
-        if ($isImageNode(node)) {
-          node.remove();
-        }
       }
       return false;
     },
@@ -235,12 +247,20 @@ export default function ImageComponent({
   return (
     <Suspense fallback={null}>
       <>
-        <div draggable={draggable}>
+        <div className='relative' draggable={draggable}>
+          <IconButton
+            onClick={deleteNode}
+            src={require('@tabler/icons/x.svg')}
+            theme='dark'
+            className='absolute right-2 top-2 z-10 hover:scale-105 hover:bg-gray-900'
+            iconClassName='h-5 w-5'
+          />
           <LazyImage
             className={
-              isFocused
+              clsx('cursor-default', {
-                ? `focused ${$isNodeSelection(selection) ? 'draggable' : ''}`
+                'select-none': isFocused,
-                : null
+                'cursor-grab active:cursor-grabbing': isFocused && $isNodeSelection(selection),
+              })
             }
             src={src}
             altText={altText}

app/soapbox/features/compose/editor/nodes/image-node.tsx

@@ -29,14 +29,14 @@ interface ImagePayload {
   src: string
 }
 
-function convertImageElement(domNode: Node): null | DOMConversionOutput {
+const convertImageElement = (domNode: Node): null | DOMConversionOutput => {
   if (domNode instanceof HTMLImageElement) {
     const { alt: altText, src } = domNode;
     const node = $createImageNode({ altText, src });
     return { node };
   }
   return null;
-}
+};
 
 type SerializedImageNode = Spread<
   {
@@ -146,11 +146,11 @@ class ImageNode extends DecoratorNode<JSX.Element> {
 
 }
 
-function $createImageNode({
+const $createImageNode = ({
   altText = '',
   src,
   key,
-}: ImagePayload): ImageNode {
+}: ImagePayload): ImageNode => {
   return $applyNodeReplacement(
     new ImageNode(
       src,
@@ -158,7 +158,7 @@ function $createImageNode({
       key,
     ),
   );
-}
+};
 
 const $isImageNode = (
   node: LexicalNode | null | undefined,

app/soapbox/features/compose/editor/plugins/floating-block-type-toolbar-plugin.tsx

@@ -183,7 +183,7 @@ const BlockTypeFloatingToolbar = ({
       const selection = $getSelection();
       if ($isRangeSelection(selection) || DEPRECATED_$isGridSelection(selection)) {
         const selectionNode = selection.anchor.getNode();
-        selectionNode.replace($createImageNode({ src }));
+        selectionNode.replace($createImageNode({ altText: '', src }));
       }
     });
   };