Commit d15d8261 authored by mfluharty's avatar mfluharty

Limit popover max width, trigger on hover

parent 6bbf1dcb
...@@ -63,9 +63,9 @@ export default { ...@@ -63,9 +63,9 @@ export default {
class="gl-hover-cursor-pointer codequality-severity-icon" class="gl-hover-cursor-pointer codequality-severity-icon"
/> />
<gl-popover <gl-popover
triggers="click blur" triggers="hover focus"
placement="topright" placement="topright"
:css-classes="['gl-max-w-none', 'gl-w-half']" :css-classes="['gl-max-w-max-content', 'gl-w-half']"
:target="`codequality-${filePath}:${line}`" :target="`codequality-${filePath}:${line}`"
:title="$options.i18n.popoverTitle" :title="$options.i18n.popoverTitle"
> >
......
...@@ -73,10 +73,10 @@ describe('EE CodeQualityGutterIcon', () => { ...@@ -73,10 +73,10 @@ describe('EE CodeQualityGutterIcon', () => {
createComponent(); createComponent();
}); });
it('opens a popover on click', () => { it('shows a popover on hover', () => {
const popoverTarget = 'codequality-index.js:3'; const popoverTarget = 'codequality-index.js:3';
wrapper.findComponent(GlIcon).trigger('click'); wrapper.findComponent(GlIcon).trigger('mouseover');
expect(wrapper.find(GlPopover).props().target).toBe(popoverTarget); expect(wrapper.find(GlPopover).props().target).toBe(popoverTarget);
}); });
......
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