Commit f3943823 authored by Alexander Turinske's avatar Alexander Turinske

Rename popover trigger id computed value

- `iconId` => `popoverTriggerId`
- it is more readable
parent 2109cef2
...@@ -30,7 +30,7 @@ export default { ...@@ -30,7 +30,7 @@ export default {
}, },
}, },
computed: { computed: {
iconId() { popoverTriggerId() {
return `reportInfo-${this.title}`; return `reportInfo-${this.title}`;
}, },
}, },
...@@ -40,14 +40,14 @@ export default { ...@@ -40,14 +40,14 @@ export default {
<template> <template>
<span class="vertical-align-middle text-muted js-help ml-2"> <span class="vertical-align-middle text-muted js-help ml-2">
<gl-icon <gl-icon
:id="iconId" :id="popoverTriggerId"
name="question" name="question"
class="author-link suggestion-help-hover" class="author-link suggestion-help-hover"
:aria-label="__('help')" :aria-label="__('help')"
:size="14" :size="14"
data-testid="icon2" data-testid="icon2"
/> />
<gl-popover :target="iconId" placement="top" :title="title"> <gl-popover :target="popoverTriggerId" placement="top" :title="title">
<div class="mb-2">{{ text }}</div> <div class="mb-2">{{ text }}</div>
<gl-link v-if="documentationLink" target="_blank" :href="documentationLink"> <gl-link v-if="documentationLink" target="_blank" :href="documentationLink">
<span class="vertical-align-middle">{{ documentationText }}</span> <span class="vertical-align-middle">{{ documentationText }}</span>
......
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