Remove scoped labels documentation link

Remove docs link and icon in scoped labels

app/assets/javascripts/boards/components/board_form.vue

@@ -58,11 +58,6 @@ export default {
       required: false,
       default: false,
     },
-    scopedLabelsDocumentationLink: {
-      type: String,
-      required: false,
-      default: '#',
-    },
   },
   data() {
     return {
@@ -208,7 +203,6 @@ export default {
           :can-admin-board="canAdminBoard"
           :milestone-path="milestonePath"
           :labels-path="labelsPath"
-          :scoped-labels-documentation-link="scopedLabelsDocumentationLink"
           :enable-scoped-labels="enableScopedLabels"
           :project-id="projectId"
           :group-id="groupId"

app/assets/javascripts/boards/components/boards_selector.vue

@@ -86,11 +86,6 @@ export default {
       required: false,
       default: false,
     },
-    scopedLabelsDocumentationLink: {
-      type: String,
-      required: false,
-      default: '#',
-    },
   },
   data() {
     return {
@@ -348,7 +343,6 @@ export default {
         :scoped-issue-board-feature-enabled="scopedIssueBoardFeatureEnabled"
         :weights="weights"
         :enable-scoped-labels="enabledScopedLabels"
-        :scoped-labels-documentation-link="scopedLabelsDocumentationLink"
       />
     </span>
   </div>

app/assets/javascripts/boards/components/issue_card_inner.vue

@@ -102,9 +102,6 @@ export default {
     orderedLabels() {
       return sortBy(this.issue.labels.filter(this.isNonListLabel), 'title');
     },
-    helpLink() {
-      return boardsStore.scopedLabels.helpLink;
-    },
   },
   methods: {
     isIndexLessThanlimit(index) {
@@ -181,7 +178,6 @@ export default {
           :description="label.description"
           size="sm"
           :scoped="showScopedLabel(label)"
-          :scoped-labels-documentation-link="helpLink"
           @click="filterByLabel(label)"
         />
       </template>

app/assets/javascripts/labels_select.js

@@ -55,7 +55,6 @@ export default class LabelsSelect {
         })
         .get();
       const scopedLabels = $dropdown.data('scopedLabels');
-      const scopedLabelsDocumentationLink = $dropdown.data('scopedLabelsDocumentationLink');
       const { handleClick } = options;
       $sidebarLabelTooltip.tooltip();
 
@@ -104,7 +103,6 @@ export default class LabelsSelect {
                 labels: sortBy(data.labels, 'title'),
                 issueUpdateURL,
                 enableScopedLabels: scopedLabels,
-                scopedLabelsDocumentationLink,
               });
               labelCount = data.labels.length;
 
@@ -497,14 +495,6 @@ export default class LabelsSelect {
       return escapeStr(label.text_color === '#FFFFFF' ? label.color : label.text_color);
     };
 
-    const infoIconTemplate = template(
-      [
-        '<a href="<%= scopedLabelsDocumentationLink %>" class="gl-link gl-label-icon" target="_blank" rel="noopener">',
-        '<i class="fa fa-question-circle"></i>',
-        '</a>',
-      ].join(''),
-    );
-
     const scopedLabelTemplate = template(
       [
         '<span class="gl-label gl-label-scoped" style="color: <%= escapeStr(label.color) %>;">',
@@ -516,7 +506,6 @@ export default class LabelsSelect {
         '<%- label.title.slice(label.title.lastIndexOf("::") + 2) %>',
         '</span>',
         '</a>',
-        '<%= infoIconTemplate({ label, scopedLabelsDocumentationLink, escapeStr }) %>',
         '</span>',
       ].join(''),
     );
@@ -538,7 +527,7 @@ export default class LabelsSelect {
         '<% labels.forEach(function(label){ %>',
         '<% if (isScopedLabel(label) && enableScopedLabels) { %>',
         '<span class="d-inline-block position-relative scoped-label-wrapper">',
-        '<%= scopedLabelTemplate({ label, issueUpdateURL, isScopedLabel, enableScopedLabels, rightLabelTextColor, infoIconTemplate, scopedLabelsDocumentationLink, tooltipTitleTemplate, escapeStr, linkAttrs: \'data-html="true"\' }) %>',
+        '<%= scopedLabelTemplate({ label, issueUpdateURL, isScopedLabel, enableScopedLabels, rightLabelTextColor, tooltipTitleTemplate, escapeStr, linkAttrs: \'data-html="true"\' }) %>',
         '</span>',
         '<% } else { %>',
         '<%= labelTemplate({ label, issueUpdateURL, isScopedLabel, enableScopedLabels, tooltipTitleTemplate, escapeStr, linkAttrs: "" }) %>',
@@ -551,7 +540,6 @@ export default class LabelsSelect {
       ...tplData,
       labelTemplate,
       rightLabelTextColor,
-      infoIconTemplate,
       scopedLabelTemplate,
       tooltipTitleTemplate,
       isScopedLabel,

app/assets/javascripts/vue_shared/components/sidebar/labels_select/base.vue

@@ -80,11 +80,6 @@ export default {
       required: false,
       default: false,
     },
-    scopedLabelsDocumentationLink: {
-      type: String,
-      required: false,
-      default: '#',
-    },
   },
   computed: {
     hiddenInputName() {
@@ -136,7 +131,6 @@ export default {
     <dropdown-value
       :labels="context.labels"
       :label-filter-base-path="labelFilterBasePath"
-      :scoped-labels-documentation-link="scopedLabelsDocumentationLink"
       :enable-scoped-labels="enableScopedLabels"
     >
       <slot></slot>
@@ -157,7 +151,6 @@ export default {
           :namespace="namespace"
           :labels="context.labels"
           :show-extra-options="!showCreate"
-          :scoped-labels-documentation-link="scopedLabelsDocumentationLink"
           :enable-scoped-labels="enableScopedLabels"
         />
         <div

app/assets/javascripts/vue_shared/components/sidebar/labels_select/dropdown_button.vue

@@ -36,11 +36,6 @@ export default {
       required: false,
       default: false,
     },
-    scopedLabelsDocumentationLink: {
-      type: String,
-      required: false,
-      default: '#',
-    },
   },
   computed: {
     dropdownToggleText() {
@@ -72,7 +67,6 @@ export default {
     :data-namespace-path="namespace"
     :data-show-any="showExtraOptions"
     :data-scoped-labels="enableScopedLabels"
-    :data-scoped-labels-documentation-link="scopedLabelsDocumentationLink"
     type="button"
     class="dropdown-menu-toggle wide js-label-select js-multiselect js-context-config-modal"
     data-toggle="dropdown"

app/assets/javascripts/vue_shared/components/sidebar/labels_select/dropdown_value.vue

@@ -20,11 +20,6 @@ export default {
       required: false,
       default: false,
     },
-    scopedLabelsDocumentationLink: {
-      type: String,
-      required: false,
-      default: '#',
-    },
   },
   computed: {
     isEmpty() {
@@ -64,7 +59,6 @@ export default {
         :title="label.title"
         :description="label.description"
         :scoped="showScopedLabels(label)"
-        :scoped-labels-documentation-link="scopedLabelsDocumentationLink"
       />
     </template>
   </div>

app/assets/javascripts/vue_shared/components/sidebar/labels_select_vue/dropdown_value.vue

@@ -45,7 +45,6 @@ export default {
         :background-color="label.color"
         :target="labelFilterUrl(label)"
         :scoped="scopedLabel(label)"
-        :scoped-labels-documentation-link="scopedLabelsDocumentationPath"
         tooltip-placement="top"
       />
     </template>

app/assets/stylesheets/pages/issuable.scss

@@ -167,10 +167,6 @@
     }
   }
 
-  a.gl-label-icon {
-    color: $gray-500;
-  }
-
   .gl-label .gl-label-link:hover {
     text-decoration: none;
     color: inherit;
@@ -180,11 +176,6 @@
     }
   }
 
-  .gl-label .gl-label-icon:hover {
-    text-decoration: none;
-    color: $gray-500;
-  }
-
   .btn-link {
     color: inherit;
   }
@@ -826,10 +817,6 @@
           }
         }
       }
-
-      .gl-label-icon {
-        color: $gray-500;
-      }
     }
 
     @media(max-width: map-get($grid-breakpoints, lg)-1) {

app/assets/stylesheets/pages/labels.scss

@@ -460,8 +460,7 @@
 // Label inside title of Delete Label Modal
 .modal-header .page-title {
   .scoped-label-wrapper {
-    .scoped-label,
-    .gl-label-icon {
+    .scoped-label {
       line-height: 20px;
     }
 

app/assets/stylesheets/pages/milestone.scss

@@ -68,10 +68,6 @@ $status-box-line-height: 26px;
       .gl-label-link {
         color: inherit;
       }
-
-      .gl-label-icon {
-        color: $gray-500;
-      }
     }
   }
 

app/assets/stylesheets/pages/notes.scss

@@ -688,8 +688,7 @@ $note-form-margin-left: 72px;
     text-decoration: underline;
   }
 
-  .gl-label-link:hover,
-  .gl-label-icon:hover {
+  .gl-label-link:hover {
     text-decoration: none;
     color: inherit;
 

app/views/shared/boards/components/_board.html.haml

@@ -38,8 +38,7 @@
             ":description" => "list.label.description",
             "tooltipPlacement" => "bottom",
             ":size" => '(!list.isExpanded ? "sm" : "")',
-            ":scoped" => "showScopedLabels(list.label)",
-            ":scoped-labels-documentation-link" => "helpLink" }
+            ":scoped" => "showScopedLabels(list.label)" }
 
         - if can?(current_user, :admin_list, current_board_parent)
           %board-delete{ "inline-template" => true,

app/views/shared/boards/components/sidebar/_labels.html.haml

@@ -12,8 +12,7 @@
         ":background-color" => "label.color",
         ":title" => "label.title",
         ":description" => "label.description",
-        ":scoped" => "showScopedLabels(label)",
-        ":scoped-labels-documentation-link" => "helpLink" }
+        ":scoped" => "showScopedLabels(label)" }
 
   - if can_admin_issue?
     .selectbox

ee/app/assets/javascripts/boards/components/board_scope.vue

@@ -35,11 +35,6 @@ export default {
       type: String,
       required: true,
     },
-    scopedLabelsDocumentationLink: {
-      type: String,
-      required: false,
-      default: '#',
-    },
     enableScopedLabels: {
       type: Boolean,
       required: false,
@@ -119,7 +114,6 @@ export default {
         :context="board"
         :labels-path="labelsPath"
         :can-edit="canAdminBoard"
-        :scoped-labels-documentation-link="scopedLabelsDocumentationLink"
         :enable-scoped-labels="enableScopedLabels"
         ability-name="issue"
         @onLabelClick="handleLabelClick"

ee/app/assets/javascripts/boards/stores/boards_store_ee.js

@@ -39,7 +39,6 @@ class BoardsStoreEE {
             boardWeight,
             weightFeatureAvailable,
             scopedLabels,
-            scopedLabelsDocumentationLink,
           },
         } = this.$boardApp;
         this.store.boardConfig = {
@@ -53,7 +52,6 @@ class BoardsStoreEE {
         this.store.weightFeatureAvailable = parseBoolean(weightFeatureAvailable);
         this.store.scopedLabels = {
           enabled: parseBoolean(scopedLabels),
-          helpLink: scopedLabelsDocumentationLink,
         };
         this.initBoardFilters();
       }

ee/app/assets/javascripts/epic/components/sidebar_items/sidebar_labels.vue

@@ -35,7 +35,6 @@ export default {
       'labelsWebUrl',
       'epicsWebUrl',
       'scopedLabels',
-      'scopedLabelsDocumentationLink',
       'epicLabelsSelectInProgress',
     ]),
     epicContext() {
@@ -118,7 +117,6 @@ export default {
     :labels-fetch-path="labelsPath"
     :labels-manage-path="labelsWebUrl"
     :labels-filter-base-path="epicsWebUrl"
-    :scoped-labels-documentation-path="scopedLabelsDocumentationLink"
     class="block labels js-labels-block"
     @updateSelectedLabels="handleUpdateSelectedLabels"
     @onDropdownClose="handleDropdownClose"

ee/app/helpers/ee/boards_helper.rb

@@ -26,7 +26,6 @@ module EE
         weight_feature_available: current_board_parent.feature_available?(:issue_weights).to_s,
         show_promotion: show_feature_promotion,
         scoped_labels: current_board_parent.feature_available?(:scoped_labels)&.to_s,
-        scoped_labels_documentation_link: help_page_path('user/project/labels.md', anchor: 'scoped-labels-premium')
       }
 
       super.merge(data)

ee/app/helpers/ee/labels_helper.rb

@@ -30,7 +30,7 @@ module EE
 
       <<~HTML.chomp.html_safe
         <span class="d-inline-block position-relative scoped-label-wrapper">
-          <span class="#{wrapper_classes.join(' ')}" style="color: #{label.color}">#{label_html + scoped_labels_doc_link}</span>
+          <span class="#{wrapper_classes.join(' ')}" style="color: #{label.color}">#{label_html}</span>
         </span>
       HTML
     end
@@ -45,7 +45,6 @@ module EE
     def label_dropdown_data(edit_context, opts = {})
       scoped_labels_fields = {
         scoped_labels: edit_context&.feature_available?(:scoped_labels)&.to_s,
-        scoped_labels_documentation_link: help_page_path('user/project/labels.md', anchor: 'scoped-labels-premium')
       }
 
       return super.merge(scoped_labels_fields) unless edit_context.is_a?(Group)
@@ -70,13 +69,5 @@ module EE
 
       super + ['epics']
     end
-
-    private
-
-    def scoped_labels_doc_link
-      help_url = ::Gitlab::Routing.url_helpers.help_page_url('user/project/labels.md', anchor: 'scoped-labels-premium')
-
-      %(<a href="#{help_url}" class="gl-link gl-label-icon" target="_blank" rel="noopener"><i class="fa fa-question-circle"></i></a>).html_safe
-    end
   end
 end

ee/app/presenters/epic_presenter.rb

@@ -79,7 +79,6 @@ class EpicPresenter < Gitlab::View::Presenter::Delegated
       toggle_subscription_path: toggle_subscription_group_epic_path(group, epic),
       labels_web_url: group_labels_path(group),
       epics_web_url: group_epics_path(group),
-      scoped_labels_documentation_link: help_page_path('user/project/labels.md', anchor: 'scoped-labels-premium')
     }
 
     paths[:todo_delete_path] = dashboard_todo_path(epic_pending_todo) if epic_pending_todo.present?

ee/spec/fixtures/api/schemas/epic_meta_data.json

@@ -3,7 +3,7 @@
   "type": "object",
   "required": ["epic_id", "created", "author", "ancestors", "todo_exists", "todo_path", "lock_version",
     "state", "namespace", "labels_path", "toggle_subscription_path", "labels_web_url", "epics_web_url",
-    "scoped_labels", "scoped_labels_documentation_link", "start_date", "start_date_is_fixed", "start_date_fixed",
+    "scoped_labels", "start_date", "start_date_is_fixed", "start_date_fixed",
     "start_date_from_milestones", "start_date_sourcing_milestone_title", "start_date_sourcing_milestone_dates",
     "due_date", "due_date_is_fixed", "due_date_fixed",
     "due_date_from_milestones", "due_date_sourcing_milestone_title", "due_date_sourcing_milestone_dates"],
@@ -103,9 +103,6 @@
     "scoped_labels": {
       "type": "boolean"
     },
-    "scoped_labels_documentation_link": {
-      "type": "string"
-    },
     "start_date": {
       "type": [
         "string",

ee/spec/frontend/boards/components/board_settings_sidebar_spec.js

@@ -67,7 +67,6 @@ describe('BoardSettingsSideBar', () => {
       removeList: bs.removeList,
       scopedLabels: {
         enabled: false,
-        scopedLabelsDocumentationLink: '',
       },
     };
 

ee/spec/helpers/ee/labels_helper_spec.rb

@@ -57,7 +57,6 @@ describe LabelsHelper do
         show_any: "true",
         default_label: "Labels",
         scoped_labels: "false",
-        scoped_labels_documentation_link: "/help/user/project/labels.md#scoped-labels-premium"
       }
     end
 

ee/spec/lib/banzai/filter/label_reference_filter_spec.rb

@@ -18,14 +18,12 @@ describe Banzai::Filter::LabelReferenceFilter do
       doc = reference_filter("See #{scoped_label.to_reference}")
 
       expect(doc.css('.gl-label-scoped .gl-label-text').map(&:text)).to eq([scoped_label.scoped_label_key, scoped_label.scoped_label_value])
-      expect(doc.at_css('a.gl-label-icon')['href']).to eq('http://localhost/help/user/project/labels.md#scoped-labels-premium')
     end
 
     it 'renders common label' do
       doc = reference_filter("See #{label.to_reference}")
 
       expect(doc.css('.gl-label .gl-label-text').map(&:text)).to eq([label.name])
-      expect(doc.at_css('a.gl-label-icon')).to be_nil
     end
   end
 
@@ -38,7 +36,6 @@ describe Banzai::Filter::LabelReferenceFilter do
       doc = reference_filter("See #{scoped_label.to_reference}")
 
       expect(doc.css('.gl-label .gl-label-text').map(&:text)).to eq([scoped_label.name])
-      expect(doc.at_css('a.gl-label-icon')).to be_nil
     end
   end
 end

spec/frontend/labels_select_spec.js

@@ -45,7 +45,6 @@ describe('LabelsSelect', () => {
             labels: mockLabels,
             issueUpdateURL: mockUrl,
             enableScopedLabels: true,
-            scopedLabelsDocumentationLink: 'docs-link',
           }),
         );
       });
@@ -71,10 +70,6 @@ describe('LabelsSelect', () => {
       it('generated label item has a gl-label-text class', () => {
         expect($labelEl.find('span').hasClass('gl-label-text')).toEqual(true);
       });
-
-      it('generated label item template does not have gl-label-icon class', () => {
-        expect($labelEl.find('.gl-label-icon')).toHaveLength(0);
-      });
     });
 
     describe('when scoped label is present', () => {
@@ -87,7 +82,6 @@ describe('LabelsSelect', () => {
             labels: mockScopedLabels,
             issueUpdateURL: mockUrl,
             enableScopedLabels: true,
-            scopedLabelsDocumentationLink: 'docs-link',
           }),
         );
       });
@@ -110,10 +104,6 @@ describe('LabelsSelect', () => {
         expect($labelEl.find('i.fa-question-circle')).toHaveLength(1);
       });
 
-      it('generated label item template has gl-label-icon class', () => {
-        expect($labelEl.find('.gl-label-icon')).toHaveLength(1);
-      });
-
       it('generated label item template has correct label styles', () => {
         expect($labelEl.find('span.gl-label-text').attr('style')).toBe(
           `background-color: ${label.color}; color: ${label.text_color};`,
@@ -141,7 +131,6 @@ describe('LabelsSelect', () => {
             labels: mockScopedLabels2,
             issueUpdateURL: mockUrl,
             enableScopedLabels: true,
-            scopedLabelsDocumentationLink: 'docs-link',
           }),
         );
       });