Commit ba2bd1bf authored by Kushal Pandya's avatar Kushal Pandya

Merge branch '247845-remove-epic-icon-from-epic-swimlanes' into 'master'

Remove epic icons on Swimlanes

See merge request gitlab-org/gitlab!43369
parents 09e608d8 c6f0389a
...@@ -56,15 +56,6 @@ export default { ...@@ -56,15 +56,6 @@ export default {
chevronIcon() { chevronIcon() {
return this.isExpanded ? 'chevron-down' : 'chevron-right'; return this.isExpanded ? 'chevron-down' : 'chevron-right';
}, },
stateText() {
return this.isOpen ? __('Opened') : __('Closed');
},
epicIcon() {
return this.isOpen ? 'epic' : 'epic-closed';
},
stateIconClass() {
return this.isOpen ? 'gl-text-green-500' : 'gl-text-blue-500';
},
issuesCount() { issuesCount() {
return this.lists.reduce( return this.lists.reduce(
(total, list) => total + this.getIssuesByEpic(list.id, this.epic.id).length, (total, list) => total + this.getIssuesByEpic(list.id, this.epic.id).length,
...@@ -126,12 +117,6 @@ export default { ...@@ -126,12 +117,6 @@ export default {
data-testid="epic-lane-chevron" data-testid="epic-lane-chevron"
@click="toggleExpanded" @click="toggleExpanded"
/> />
<gl-icon
class="gl-mr-2 gl-flex-shrink-0"
:class="stateIconClass"
:name="epicIcon"
:aria-label="stateText"
/>
<h4 <h4
ref="epicTitle" ref="epicTitle"
class="gl-mr-3 gl-font-weight-bold gl-font-base gl-white-space-nowrap gl-text-overflow-ellipsis gl-overflow-hidden" class="gl-mr-3 gl-font-weight-bold gl-font-base gl-white-space-nowrap gl-text-overflow-ellipsis gl-overflow-hidden"
......
...@@ -58,21 +58,12 @@ describe('EpicLane', () => { ...@@ -58,21 +58,12 @@ describe('EpicLane', () => {
createComponent(); createComponent();
}); });
it('icon aria label is Opened when epic is opened', () => {
expect(wrapper.find(GlIcon).attributes('aria-label')).toEqual('Opened');
});
it('icon aria label is Closed when epic is closed', () => {
createComponent({ props: { epic: { ...mockEpic, state: 'closed' } } });
expect(wrapper.find(GlIcon).attributes('aria-label')).toEqual('Closed');
});
it('displays count of issues in epic which belong to board', () => { it('displays count of issues in epic which belong to board', () => {
expect(findByTestId('epic-lane-issue-count').text()).toContain(2); expect(findByTestId('epic-lane-issue-count').text()).toContain(2);
}); });
it('displays 2 icons', () => { it('displays 1 icon', () => {
expect(wrapper.findAll(GlIcon)).toHaveLength(2); expect(wrapper.findAll(GlIcon)).toHaveLength(1);
}); });
it('displays epic title', () => { it('displays epic title', () => {
......
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