From 82d9711c393c8d31a35aa3b43de30f0665f0c43c Mon Sep 17 00:00:00 2001 From: Himanshu Kapoor <hkapoor@gitlab.com> Date: Wed, 20 Apr 2022 21:24:19 +0200 Subject: [PATCH] Create a bubble_menus directory in content editor Since we anticipate 2 more bubble menus, a minor refactor to put the current 2 bubble menus in a separate bubble_menus directory --- .../code_block.vue} | 10 +++++----- .../formatting.vue} | 14 +++++++------- .../content_editor/components/content_editor.vue | 4 ++-- .../code_block_spec.js} | 6 +++--- .../formatting_spec.js} | 6 +++--- .../components/content_editor_spec.js | 2 +- 6 files changed, 21 insertions(+), 21 deletions(-) rename app/assets/javascripts/content_editor/components/{code_block_bubble_menu.vue => bubble_menus/code_block.vue} (93%) rename app/assets/javascripts/content_editor/components/{formatting_bubble_menu.vue => bubble_menus/formatting.vue} (83%) rename spec/frontend/content_editor/components/{code_block_bubble_menu_spec.js => bubble_menus/code_block_spec.js} (95%) rename spec/frontend/content_editor/components/{formatting_bubble_menu_spec.js => bubble_menus/formatting_spec.js} (92%) diff --git a/app/assets/javascripts/content_editor/components/code_block_bubble_menu.vue b/app/assets/javascripts/content_editor/components/bubble_menus/code_block.vue similarity index 93% rename from app/assets/javascripts/content_editor/components/code_block_bubble_menu.vue rename to app/assets/javascripts/content_editor/components/bubble_menus/code_block.vue index 51a84d80a65c..5347f7c87121 100644 --- a/app/assets/javascripts/content_editor/components/code_block_bubble_menu.vue +++ b/app/assets/javascripts/content_editor/components/bubble_menus/code_block.vue @@ -9,11 +9,11 @@ import { } from '@gitlab/ui'; import { BubbleMenu } from '@tiptap/vue-2'; import { getParentByTagName } from '~/lib/utils/dom_utils'; -import codeBlockLanguageLoader from '../services/code_block_language_loader'; -import CodeBlockHighlight from '../extensions/code_block_highlight'; -import Diagram from '../extensions/diagram'; -import Frontmatter from '../extensions/frontmatter'; -import EditorStateObserver from './editor_state_observer.vue'; +import codeBlockLanguageLoader from '../../services/code_block_language_loader'; +import CodeBlockHighlight from '../../extensions/code_block_highlight'; +import Diagram from '../../extensions/diagram'; +import Frontmatter from '../../extensions/frontmatter'; +import EditorStateObserver from '../editor_state_observer.vue'; const CODE_BLOCK_NODE_TYPES = [CodeBlockHighlight.name, Diagram.name, Frontmatter.name]; diff --git a/app/assets/javascripts/content_editor/components/formatting_bubble_menu.vue b/app/assets/javascripts/content_editor/components/bubble_menus/formatting.vue similarity index 83% rename from app/assets/javascripts/content_editor/components/formatting_bubble_menu.vue rename to app/assets/javascripts/content_editor/components/bubble_menus/formatting.vue index 103079534bcc..34e5caeca947 100644 --- a/app/assets/javascripts/content_editor/components/formatting_bubble_menu.vue +++ b/app/assets/javascripts/content_editor/components/bubble_menus/formatting.vue @@ -1,13 +1,13 @@ <script> import { GlButtonGroup } from '@gitlab/ui'; import { BubbleMenu } from '@tiptap/vue-2'; -import { BUBBLE_MENU_TRACKING_ACTION } from '../constants'; -import trackUIControl from '../services/track_ui_control'; -import Code from '../extensions/code'; -import CodeBlockHighlight from '../extensions/code_block_highlight'; -import Diagram from '../extensions/diagram'; -import Frontmatter from '../extensions/frontmatter'; -import ToolbarButton from './toolbar_button.vue'; +import { BUBBLE_MENU_TRACKING_ACTION } from '../../constants'; +import trackUIControl from '../../services/track_ui_control'; +import Code from '../../extensions/code'; +import CodeBlockHighlight from '../../extensions/code_block_highlight'; +import Diagram from '../../extensions/diagram'; +import Frontmatter from '../../extensions/frontmatter'; +import ToolbarButton from '../toolbar_button.vue'; export default { components: { diff --git a/app/assets/javascripts/content_editor/components/content_editor.vue b/app/assets/javascripts/content_editor/components/content_editor.vue index 5b3f4f4ddf26..585d100dfc6a 100644 --- a/app/assets/javascripts/content_editor/components/content_editor.vue +++ b/app/assets/javascripts/content_editor/components/content_editor.vue @@ -4,8 +4,8 @@ import { createContentEditor } from '../services/create_content_editor'; import ContentEditorAlert from './content_editor_alert.vue'; import ContentEditorProvider from './content_editor_provider.vue'; import EditorStateObserver from './editor_state_observer.vue'; -import FormattingBubbleMenu from './formatting_bubble_menu.vue'; -import CodeBlockBubbleMenu from './code_block_bubble_menu.vue'; +import FormattingBubbleMenu from './bubble_menus/formatting.vue'; +import CodeBlockBubbleMenu from './bubble_menus/code_block.vue'; import TopToolbar from './top_toolbar.vue'; import LoadingIndicator from './loading_indicator.vue'; diff --git a/spec/frontend/content_editor/components/code_block_bubble_menu_spec.js b/spec/frontend/content_editor/components/bubble_menus/code_block_spec.js similarity index 95% rename from spec/frontend/content_editor/components/code_block_bubble_menu_spec.js rename to spec/frontend/content_editor/components/bubble_menus/code_block_spec.js index d3ab5bd762a3..2323410992ad 100644 --- a/spec/frontend/content_editor/components/code_block_bubble_menu_spec.js +++ b/spec/frontend/content_editor/components/bubble_menus/code_block_spec.js @@ -2,13 +2,13 @@ import { BubbleMenu } from '@tiptap/vue-2'; import { GlDropdown, GlDropdownItem, GlSearchBoxByType } from '@gitlab/ui'; import Vue from 'vue'; import { mountExtended } from 'helpers/vue_test_utils_helper'; -import CodeBlockBubbleMenu from '~/content_editor/components/code_block_bubble_menu.vue'; +import CodeBlockBubbleMenu from '~/content_editor/components/bubble_menus/code_block.vue'; import eventHubFactory from '~/helpers/event_hub_factory'; import CodeBlockHighlight from '~/content_editor/extensions/code_block_highlight'; import codeBlockLanguageLoader from '~/content_editor/services/code_block_language_loader'; -import { createTestEditor, emitEditorEvent } from '../test_utils'; +import { createTestEditor, emitEditorEvent } from '../../test_utils'; -describe('content_editor/components/code_block_bubble_menu', () => { +describe('content_editor/components/bubble_menus/code_block', () => { let wrapper; let tiptapEditor; let bubbleMenu; diff --git a/spec/frontend/content_editor/components/formatting_bubble_menu_spec.js b/spec/frontend/content_editor/components/bubble_menus/formatting_spec.js similarity index 92% rename from spec/frontend/content_editor/components/formatting_bubble_menu_spec.js rename to spec/frontend/content_editor/components/bubble_menus/formatting_spec.js index 192ddee78c6c..889acd63be39 100644 --- a/spec/frontend/content_editor/components/formatting_bubble_menu_spec.js +++ b/spec/frontend/content_editor/components/bubble_menus/formatting_spec.js @@ -1,15 +1,15 @@ import { BubbleMenu } from '@tiptap/vue-2'; import { mockTracking } from 'helpers/tracking_helper'; import { shallowMountExtended } from 'helpers/vue_test_utils_helper'; -import FormattingBubbleMenu from '~/content_editor/components/formatting_bubble_menu.vue'; +import FormattingBubbleMenu from '~/content_editor/components/bubble_menus/formatting.vue'; import { BUBBLE_MENU_TRACKING_ACTION, CONTENT_EDITOR_TRACKING_LABEL, } from '~/content_editor/constants'; -import { createTestEditor } from '../test_utils'; +import { createTestEditor } from '../../test_utils'; -describe('content_editor/components/formatting_bubble_menu', () => { +describe('content_editor/components/bubble_menus/formatting', () => { let wrapper; let trackingSpy; let tiptapEditor; diff --git a/spec/frontend/content_editor/components/content_editor_spec.js b/spec/frontend/content_editor/components/content_editor_spec.js index 73fcfeab8bcd..9ee3b0178317 100644 --- a/spec/frontend/content_editor/components/content_editor_spec.js +++ b/spec/frontend/content_editor/components/content_editor_spec.js @@ -4,7 +4,7 @@ import ContentEditor from '~/content_editor/components/content_editor.vue'; import ContentEditorAlert from '~/content_editor/components/content_editor_alert.vue'; import ContentEditorProvider from '~/content_editor/components/content_editor_provider.vue'; import EditorStateObserver from '~/content_editor/components/editor_state_observer.vue'; -import FormattingBubbleMenu from '~/content_editor/components/formatting_bubble_menu.vue'; +import FormattingBubbleMenu from '~/content_editor/components/bubble_menus/formatting.vue'; import TopToolbar from '~/content_editor/components/top_toolbar.vue'; import LoadingIndicator from '~/content_editor/components/loading_indicator.vue'; import { emitEditorEvent } from '../test_utils'; -- GitLab