Commit 6f2d7fa9 authored by George Tsiolis's avatar George Tsiolis

Remove tooltip on sidebar text buttons

parent 82bd2a0b
......@@ -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"
......
---
title: Remove tooltip on sidebar text buttons
merge_request: 8021
author: George Tsiolis
type: other
......@@ -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');
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment