diff --git a/.rubocop_todo/layout/line_length.yml b/.rubocop_todo/layout/line_length.yml index 669ab00c976b288fb5f0365acb684a5e2b1cdf86..7e96fb47e13277f955850f228bc5eaf22e501c10 100644 --- a/.rubocop_todo/layout/line_length.yml +++ b/.rubocop_todo/layout/line_length.yml @@ -2399,7 +2399,6 @@ Layout/LineLength: - 'spec/features/boards/board_filters_spec.rb' - 'spec/features/boards/boards_spec.rb' - 'spec/features/boards/issue_ordering_spec.rb' - - 'spec/features/boards/multi_select_spec.rb' - 'spec/features/boards/sidebar_assignee_spec.rb' - 'spec/features/boards/sidebar_labels_spec.rb' - 'spec/features/boards/user_visits_board_spec.rb' diff --git a/.rubocop_todo/rspec/pending_without_reason.yml b/.rubocop_todo/rspec/pending_without_reason.yml index 8fa2271e9bd6d8f1dada58dfbcdcd5c5794311ab..0d7b9ab3d385e9a859c3e759b27c2be9ac092289 100644 --- a/.rubocop_todo/rspec/pending_without_reason.yml +++ b/.rubocop_todo/rspec/pending_without_reason.yml @@ -12,7 +12,6 @@ RSpec/PendingWithoutReason: - 'ee/spec/support/shared_examples/models/vulnerabilities/cvss_vector_validator_shared_examples.rb' - 'qa/qa/specs/features/api/8_monitor/metrics_spec.rb' - 'spec/controllers/projects/issues_controller_spec.rb' - - 'spec/features/boards/multi_select_spec.rb' - 'spec/features/issues/create_issue_for_single_discussion_in_merge_request_spec.rb' - 'spec/features/merge_request/user_merges_merge_request_spec.rb' - 'spec/features/merge_request/user_resolves_conflicts_spec.rb' diff --git a/app/assets/javascripts/boards/components/board_card.vue b/app/assets/javascripts/boards/components/board_card.vue index 36532c31e7ba227f82ca14d2b60a531b178acb01..ce990e85dd8d76d557e9e59ad4d0758132b21c3d 100644 --- a/app/assets/javascripts/boards/components/board_card.vue +++ b/app/assets/javascripts/boards/components/board_card.vue @@ -4,9 +4,6 @@ import { getIdFromGraphQLId } from '~/graphql_shared/utils'; import { visitUrl } from '~/lib/utils/url_utility'; import { sprintf, __ } from '~/locale'; import { WORK_ITEM_TYPE_ENUM_INCIDENT } from '~/work_items/constants'; -import setSelectedBoardItemsMutation from '~/boards/graphql/client/set_selected_board_items.mutation.graphql'; -import unsetSelectedBoardItemsMutation from '~/boards/graphql/client/unset_selected_board_items.mutation.graphql'; -import selectedBoardItemsQuery from '~/boards/graphql/client/selected_board_items.query.graphql'; import setActiveBoardItemMutation from 'ee_else_ce/boards/graphql/client/set_active_board_item.mutation.graphql'; import activeBoardItemQuery from 'ee_else_ce/boards/graphql/client/active_board_item.query.graphql'; import BoardCardInner from './board_card_inner.vue'; @@ -65,10 +62,6 @@ export default { }; }, }, - // eslint-disable-next-line @gitlab/vue-no-undef-apollo-properties - selectedBoardItems: { - query: selectedBoardItemsQuery, - }, }, computed: { activeItemId() { @@ -77,9 +70,6 @@ export default { isActive() { return this.item.id === this.activeItemId; }, - multiSelectVisible() { - return !this.activeItemId && this.selectedBoardItems?.includes(this.item.id); - }, isDisabled() { return this.disabled || !this.item.id || this.item.isLoading || !this.canAdmin; }, @@ -100,7 +90,7 @@ export default { }; }, showFocusBackground() { - return !this.isActive && !this.multiSelectVisible; + return !this.isActive; }, itemPrefix() { return this.isEpicBoard ? '&' : '#'; @@ -123,9 +113,8 @@ export default { // Allow Ctrl/Cmd+click to open link in new tab const isMetaKey = e.ctrlKey || e.metaKey; - const boardMultiSelectEnabled = gon?.features?.boardMultiSelect; - if (isMetaKey && !boardMultiSelectEnabled) { + if (isMetaKey) { // Let the browser handle the new tab open return; } @@ -139,18 +128,11 @@ export default { return; } - if (isMetaKey && boardMultiSelectEnabled) { - this.toggleBoardItemMultiSelection(this.item); - } else { - this.$el.querySelector('.board-card-button')?.focus(); - this.toggleItem(); - this.track('click_card', { label: 'right_sidebar' }); - } + this.$el.querySelector('.board-card-button')?.focus(); + this.toggleItem(); + this.track('click_card', { label: 'right_sidebar' }); }, async toggleItem() { - await this.$apollo.mutate({ - mutation: unsetSelectedBoardItemsMutation, - }); this.$apollo.mutate({ mutation: setActiveBoardItemMutation, variables: { @@ -160,26 +142,6 @@ export default { }, }); }, - async toggleBoardItemMultiSelection(item) { - if (this.activeItemId) { - await this.$apollo.mutate({ - mutation: setSelectedBoardItemsMutation, - variables: { - itemId: this.activeItemId, - }, - }); - await this.$apollo.mutate({ - mutation: setActiveBoardItemMutation, - variables: { boardItem: null, listId: null }, - }); - } - this.$apollo.mutate({ - mutation: setSelectedBoardItemsMutation, - variables: { - itemId: item.id, - }, - }); - }, changeFocusInColumn(currentCard, i) { // Building a list using data-col-index instead of just traversing the ul is necessary for swimlanes const columnCards = [ @@ -240,7 +202,6 @@ export default {