Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
G
gitlab-ce
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
1
Merge Requests
1
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
nexedi
gitlab-ce
Commits
02e74971
Commit
02e74971
authored
Nov 05, 2018
by
George Tsiolis
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Replace tooltip directive with gl-tooltip directive in badges, cycle analytics, and diffs
parent
62e0877d
Changes
7
Hide whitespace changes
Inline
Side-by-side
Showing
7 changed files
with
24 additions
and
27 deletions
+24
-27
app/assets/javascripts/badges/components/badge.vue
app/assets/javascripts/badges/components/badge.vue
+3
-4
app/assets/javascripts/cycle_analytics/components/limit_warning_component.vue
...ts/cycle_analytics/components/limit_warning_component.vue
+3
-4
app/assets/javascripts/diffs/components/commit_item.vue
app/assets/javascripts/diffs/components/commit_item.vue
+0
-4
app/assets/javascripts/diffs/components/compare_versions.vue
app/assets/javascripts/diffs/components/compare_versions.vue
+3
-3
app/assets/javascripts/diffs/components/diff_file_header.vue
app/assets/javascripts/diffs/components/diff_file_header.vue
+7
-8
app/assets/javascripts/vue_shared/components/time_ago_tooltip.vue
...ts/javascripts/vue_shared/components/time_ago_tooltip.vue
+3
-4
changelogs/unreleased/gt-use-gl-tooltip-directive.yml
changelogs/unreleased/gt-use-gl-tooltip-directive.yml
+5
-0
No files found.
app/assets/javascripts/badges/components/badge.vue
View file @
02e74971
<
script
>
import
Icon
from
'
~/vue_shared/components/icon.vue
'
;
import
Tooltip
from
'
~/vue_shared/directives/tooltip
'
;
import
{
GlLoadingIcon
}
from
'
@gitlab/
ui
'
;
import
{
GlLoadingIcon
,
GlTooltipDirective
}
from
'
@gitlab-org/gitlab-
ui
'
;
export
default
{
name
:
'
Badge
'
,
components
:
{
Icon
,
Tooltip
,
GlLoadingIcon
,
},
directives
:
{
Tooltip
,
GlTooltip
:
GlTooltipDirective
,
},
props
:
{
imageUrl
:
{
...
...
@@ -93,7 +92,7 @@ export default {
<button
v-show=
"hasError"
v-tooltip
v-
gl-
tooltip
:title=
"s__('Badges|Reload badge image')"
class=
"btn btn-transparent btn-sm text-primary"
type=
"button"
...
...
app/assets/javascripts/cycle_analytics/components/limit_warning_component.vue
View file @
02e74971
<
script
>
import
tooltip
from
'
../../vue_shared/directives/tooltip
'
;
import
{
GlTooltipDirective
}
from
'
@gitlab-org/gitlab-ui
'
;
export
default
{
directives
:
{
tooltip
,
GlTooltip
:
GlTooltipDirective
,
},
props
:
{
count
:
{
...
...
@@ -16,13 +16,12 @@ export default {
<
template
>
<span
v-if=
"count === 50"
class=
"events-info float-right"
>
<i
v-
toolti
p
v-
gl-tooltip
.
to
p
:title=
"
n__('Limited to showing %d event at most', 'Limited to showing %d events at most', 50)
"
class=
"fa fa-warning"
aria-hidden=
"true"
data-placement=
"top"
>
</i>
{{
n__
(
'
Showing %d event
'
,
'
Showing %d events
'
,
50
)
}}
...
...
app/assets/javascripts/diffs/components/commit_item.vue
View file @
02e74971
<
script
>
import
tooltip
from
'
~/vue_shared/directives/tooltip
'
;
import
UserAvatarLink
from
'
~/vue_shared/components/user_avatar/user_avatar_link.vue
'
;
import
Icon
from
'
~/vue_shared/components/icon.vue
'
;
import
ClipboardButton
from
'
~/vue_shared/components/clipboard_button.vue
'
;
...
...
@@ -21,9 +20,6 @@ import CommitPipelineStatus from '~/projects/tree/components/commit_pipeline_sta
*
*/
export
default
{
directives
:
{
tooltip
,
},
components
:
{
UserAvatarLink
,
Icon
,
...
...
app/assets/javascripts/diffs/components/compare_versions.vue
View file @
02e74971
<
script
>
import
{
mapActions
,
mapGetters
,
mapState
}
from
'
vuex
'
;
import
Tooltip
from
'
@gitlab/ui/dist/directives/tooltip
'
;
import
{
GlTooltipDirective
}
from
'
@gitlab-org/gitlab-ui
'
;
import
{
__
}
from
'
~/locale
'
;
import
{
getParameterValues
,
mergeUrlParams
}
from
'
~/lib/utils/url_utility
'
;
import
Icon
from
'
~/vue_shared/components/icon.vue
'
;
...
...
@@ -12,7 +12,7 @@ export default {
Icon
,
},
directives
:
{
Tooltip
,
GlTooltip
:
GlTooltipDirective
,
},
props
:
{
mergeRequestDiffs
:
{
...
...
@@ -75,7 +75,7 @@ export default {
<div
class=
"mr-version-controls"
>
<div
class=
"mr-version-menus-container content-block"
>
<button
v-tooltip
.
hover
v-
gl-
tooltip
.
hover
type=
"button"
class=
"btn btn-default append-right-8 js-toggle-tree-list"
:class=
"
{
...
...
app/assets/javascripts/diffs/components/diff_file_header.vue
View file @
02e74971
...
...
@@ -4,7 +4,7 @@ import { mapActions, mapGetters } from 'vuex';
import
ClipboardButton
from
'
~/vue_shared/components/clipboard_button.vue
'
;
import
Icon
from
'
~/vue_shared/components/icon.vue
'
;
import
FileIcon
from
'
~/vue_shared/components/file_icon.vue
'
;
import
Tooltip
from
'
~/vue_shared/directives/tooltip
'
;
import
{
GlTooltipDirective
}
from
'
@gitlab-org/gitlab-ui
'
;
import
{
truncateSha
}
from
'
~/lib/utils/text_utility
'
;
import
{
__
,
s__
,
sprintf
}
from
'
~/locale
'
;
import
EditButton
from
'
./edit_button.vue
'
;
...
...
@@ -17,7 +17,7 @@ export default {
FileIcon
,
},
directives
:
{
Tooltip
,
GlTooltip
:
GlTooltipDirective
,
},
props
:
{
discussionPath
:
{
...
...
@@ -161,23 +161,22 @@ export default {
/>
<span
v-if=
"diffFile.renamed_file"
>
<strong
v-tooltip
v-
gl-
tooltip
:title=
"diffFile.old_path"
class=
"file-title-name"
data-container=
"body"
v-html=
"diffFile.old_path_html"
></strong>
→
<strong
v-tooltip
v-
gl-
tooltip
:title=
"diffFile.new_path"
class=
"file-title-name"
data-container=
"body"
v-html=
"diffFile.new_path_html"
></strong>
</span>
<strong
v-else
v-tooltip
:title=
"filePath"
class=
"file-title-name"
data-container=
"body"
>
<strong
v-else
v-
gl-
tooltip
:title=
"filePath"
class=
"file-title-name"
data-container=
"body"
>
{{
filePath
}}
</strong>
</a>
...
...
@@ -233,7 +232,7 @@ export default {
<a
v-if=
"diffFile.external_url"
v-tooltip
v-
gl-
tooltip
:href=
"diffFile.external_url"
:title=
"`View on ${diffFile.formatted_external_url}`"
target=
"_blank"
...
...
app/assets/javascripts/vue_shared/components/time_ago_tooltip.vue
View file @
02e74971
<
script
>
import
tooltip
from
'
../directives/tooltip
'
;
import
{
GlTooltipDirective
}
from
'
@gitlab-org/gitlab-ui
'
;
import
timeagoMixin
from
'
../mixins/timeago
'
;
import
'
../../lib/utils/datetime_utility
'
;
...
...
@@ -9,7 +9,7 @@ import '../../lib/utils/datetime_utility';
export
default
{
directives
:
{
tooltip
,
GlTooltip
:
GlTooltipDirective
,
},
mixins
:
[
timeagoMixin
],
props
:
{
...
...
@@ -34,11 +34,10 @@ export default {
</
script
>
<
template
>
<time
v-
tooltip
v-
gl-tooltip=
"
{ placement: tooltipPlacement }"
:class="cssClass"
:title="tooltipTitle(time)"
:data-placement="tooltipPlacement"
data-container=
"body"
v-text="timeFormated(time)"
>
</time>
...
...
changelogs/unreleased/gt-use-gl-tooltip-directive.yml
0 → 100644
View file @
02e74971
---
title
:
Replace tooltip directive with gl-tooltip diretive in badges, cycle analytics, and diffs
merge_request
:
22770
author
:
George Tsiolis
type
:
performance
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment