Commit 97b4aa4f authored by Filipa Lacerda's avatar Filipa Lacerda

Merge branch 'revert-55767af4' into 'master'

Revert "Merge branch '60855-mr-popover-is-not-attached-in-system-notes' into 'master'"

See merge request gitlab-org/gitlab-ce!27727
parents 6066b61a 94547f81
...@@ -22,7 +22,6 @@ import noteHeader from '~/notes/components/note_header.vue'; ...@@ -22,7 +22,6 @@ import noteHeader from '~/notes/components/note_header.vue';
import Icon from '~/vue_shared/components/icon.vue'; import Icon from '~/vue_shared/components/icon.vue';
import TimelineEntryItem from './timeline_entry_item.vue'; import TimelineEntryItem from './timeline_entry_item.vue';
import { spriteIcon } from '../../../lib/utils/common_utils'; import { spriteIcon } from '../../../lib/utils/common_utils';
import initMRPopovers from '~/mr_popover/';
const MAX_VISIBLE_COMMIT_LIST_COUNT = 3; const MAX_VISIBLE_COMMIT_LIST_COUNT = 3;
...@@ -72,9 +71,6 @@ export default { ...@@ -72,9 +71,6 @@ export default {
); );
}, },
}, },
mounted() {
initMRPopovers(this.$el.querySelectorAll('.gfm-merge_request'));
},
}; };
</script> </script>
......
---
title: Fix bug where system note MR has no popover
merge_request: 27589
author:
type: fixed
...@@ -5,10 +5,8 @@ import createStore from '~/notes/stores'; ...@@ -5,10 +5,8 @@ import createStore from '~/notes/stores';
describe('system note component', () => { describe('system note component', () => {
let vm; let vm;
let props; let props;
let initMRPopoversSpy;
beforeEach(() => { beforeEach(() => {
initMRPopoversSpy = spyOnDependency(issueSystemNote, 'initMRPopovers');
props = { props = {
note: { note: {
id: '1424', id: '1424',
...@@ -58,8 +56,4 @@ describe('system note component', () => { ...@@ -58,8 +56,4 @@ describe('system note component', () => {
it('removes wrapping paragraph from note HTML', () => { it('removes wrapping paragraph from note HTML', () => {
expect(vm.$el.querySelector('.system-note-message').innerHTML).toEqual('<span>closed</span>'); expect(vm.$el.querySelector('.system-note-message').innerHTML).toEqual('<span>closed</span>');
}); });
it('should initMRPopovers onMount', () => {
expect(initMRPopoversSpy).toHaveBeenCalled();
});
}); });
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