Cleanup

Author: TatianaFomina

src/components/inline-tools/inline-tool-link.ts

@@ -128,37 +128,19 @@ export default class LinkInlineTool implements InlineTool {
     return this.nodes.button;
   }
 
-  // /**
-  //  * Create button for Inline Toolbar
-  //  */
-  // public render(): any {
-  //   return {
-  //     icon: IconLink,
-  //     title: 'Link',
-  //     onActivate: () => {
-  //       // this.surround(this.selection.get());
-  //     },
-  //   };
-  // }
-
   /**
    * Input for the link
    */
   public renderActions(): HTMLElement {
     this.nodes.input = document.createElement('input') as HTMLInputElement;
     this.nodes.input.placeholder = this.i18n.t('Add a link');
     this.nodes.input.classList.add(this.CSS.input);
-    // this.nodes.input.classList.add(this.CSS.inputShowed);
     this.nodes.input.addEventListener('keydown', (event: KeyboardEvent) => {
       if (event.keyCode === this.ENTER_KEY) {
         this.enterPressed(event);
       }
     });
 
-    // setTimeout(() => {
-    //   this.nodes.input.focus();
-    // }, 1000);
-
     return this.nodes.input;
   }
 
@@ -168,8 +150,6 @@ export default class LinkInlineTool implements InlineTool {
    * @param {Range} range - range to wrap with link
    */
   public surround(range: Range): void {
-    debugger;
-    console.log('surround');
     /**
      * Range will be null when user makes second click on the 'link icon' to close opened input
      */
@@ -251,7 +231,6 @@ export default class LinkInlineTool implements InlineTool {
    * Show/close link input
    */
   private toggleActions(): void {
-    debugger;
     if (!this.inputOpened) {
       this.openActions(true);
     } else {

src/components/modules/index.ts

@@ -18,10 +18,8 @@ import UiAPI from './api/ui';
 
 /** ./toolbar */
 import BlockSettings from './toolbar/blockSettings';
-import ConversionToolbar from './toolbar/conversion';
 import Toolbar from './toolbar/index';
-// import InlineToolbar from './toolbar/inline';
-import InlineToolbar from './toolbar/inline2';
+import InlineToolbar from './toolbar/inline';
 
 /** . */
 import BlockEvents from './blockEvents';
@@ -60,7 +58,6 @@ export default {
 
   // Toolbar Modules
   BlockSettings,
-  ConversionToolbar,
   Toolbar,
   InlineToolbar,
 

src/components/modules/toolbar/blockSettings.ts

@@ -7,13 +7,12 @@ import { I18nInternalNS } from '../../i18n/namespace-internal';
 import Flipper from '../../flipper';
 import { TunesMenuConfigItem } from '../../../../types/tools';
 import { resolveAliases } from '../../utils/resolve-aliases';
-import { type Popover, PopoverDesktop, PopoverMobile, PopoverItemParams, PopoverItemDefaultParams, PopoverItemType } from '../../utils/popover';
+import { type Popover, PopoverDesktop, PopoverMobile, PopoverItemParams, PopoverItemType } from '../../utils/popover';
 import { PopoverEvent } from '../../utils/popover/popover.types';
 import { isMobileScreen } from '../../utils';
 import { EditorMobileLayoutToggled } from '../../events';
-import * as _ from '../../utils';
 import { IconReplace } from '@codexteam/icons';
-import { getConvertToItems, isSameBlockData } from '../../utils/blocks';
+import { getConvertToItems } from '../../utils/blocks';
 
 /**
  * HTML Elements that used for BlockSettings
@@ -234,77 +233,6 @@ export default class BlockSettings extends Module<BlockSettingsNodes> {
     return items.map(tune => this.resolveTuneAliases(tune));
   }
 
-  /**
-   * Returns list of all available conversion menu items
-   *
-   * @param currentBlock - block we are about to open block tunes for
-   */
-  private async getConvertToItems(currentBlock: Block): Promise<PopoverItemDefaultParams[]> {
-    const conversionEntries = Array.from(this.Editor.Tools.blockTools.entries());
-
-    const resultItems: PopoverItemDefaultParams[] = [];
-
-    const blockData = await currentBlock.data;
-
-    conversionEntries.forEach(([toolName, tool]) => {
-      const conversionConfig = tool.conversionConfig;
-
-      /**
-       * Skip tools without «import» rule specified
-       */
-      if (!conversionConfig || !conversionConfig.import) {
-        return;
-      }
-
-      tool.toolbox?.forEach((toolboxItem) => {
-        /**
-         * Skip tools that don't pass 'toolbox' property
-         */
-        if (_.isEmpty(toolboxItem) || !toolboxItem.icon) {
-          return;
-        }
-
-        let shouldSkip = false;
-
-        if (toolboxItem.data !== undefined) {
-          /**
-           * When a tool has several toolbox entries, we need to make sure we do not add
-           * toolbox item with the same data to the resulting array. This helps exclude duplicates
-           */
-          const hasSameData = isSameBlockData(toolboxItem.data, blockData);
-
-          shouldSkip = hasSameData;
-        } else {
-          shouldSkip = toolName === currentBlock.name;
-        }
-
-
-        if (shouldSkip) {
-          return;
-        }
-
-        resultItems.push({
-          icon: toolboxItem.icon,
-          title: toolboxItem.title,
-          name: toolName,
-          onActivate: async () => {
-            const { BlockManager, BlockSelection, Caret } = this.Editor;
-
-            const newBlock = await BlockManager.convert(this.Editor.BlockManager.currentBlock, toolName, toolboxItem.data);
-
-            BlockSelection.clearSelection();
-
-            this.close();
-
-            Caret.setToBlock(newBlock, Caret.positions.END);
-          },
-        });
-      });
-    });
-
-    return resultItems;
-  }
-
   /**
    * Handles popover close event
    */