Merge branch 'gt-remove-tooltip-on-sidebar-text-buttons' into 'master'

Remove tooltip on sidebar text buttons

Closes #8089

See merge request gitlab-org/gitlab-ee!8021

app/assets/javascripts/sidebar/components/todo_toggle/todo.vue

@@ -48,6 +48,9 @@ export default {
     buttonLabel() {
       return this.isTodo ? MARK_TEXT : TODO_TEXT;
     },
+    buttonTooltip() {
+      return !this.collapsed ? undefined : this.buttonLabel;
+    },
     collapsedButtonIconClasses() {
       return this.isTodo ? 'todo-undone' : '';
     },
@@ -67,7 +70,7 @@ export default {
   <button
     v-tooltip
     :class="buttonClasses"
-    :title="buttonLabel"
+    :title="buttonTooltip"
     :aria-label="buttonLabel"
     :data-issuable-id="issuableId"
     :data-issuable-type="issuableType"

ee/changelogs/unreleased/gt-remove-tooltip-on-sidebar-text-buttons.yml

+---
+title: Remove tooltip on sidebar text buttons
+merge_request: 8021
+author: George Tsiolis
+type: other

spec/javascripts/sidebar/todo_spec.js

@@ -116,7 +116,7 @@ describe('SidebarTodo', () => {
       const dataAttributes = {
         issuableId: '1',
         issuableType: 'epic',
-        originalTitle: 'Mark todo as done',
+        originalTitle: '',
         placement: 'left',
         container: 'body',
         boundary: 'viewport',
@@ -130,6 +130,10 @@ describe('SidebarTodo', () => {
       });
     });
 
+    it('check button label computed property', () => {
+      expect(vm.buttonLabel).toEqual('Mark todo as done');
+    });
+
     it('renders button label element when `collapsed` prop is `false`', () => {
       const buttonLabelEl = vm.$el.querySelector('span.issuable-todo-inner');