From 6cb575d612cd2470ee4224159fa16458f5426f73 Mon Sep 17 00:00:00 2001 From: Eric Eastwood Date: Tue, 6 Jun 2017 15:46:31 -0500 Subject: Add support for multiple tooltips in the same Vue component Fix https://gitlab.com/gitlab-org/gitlab-ce/issues/33223 --- doc/development/fe_guide/style_guide_js.md | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) (limited to 'doc') diff --git a/doc/development/fe_guide/style_guide_js.md b/doc/development/fe_guide/style_guide_js.md index d2d89517241..ae844fa1051 100644 --- a/doc/development/fe_guide/style_guide_js.md +++ b/doc/development/fe_guide/style_guide_js.md @@ -463,20 +463,24 @@ A forEach will cause side effects, it will be mutating the array being iterated. 1. `destroyed` #### Vue and Boostrap -1. Tooltips: Do not rely on `has-tooltip` class name for vue components +1. Tooltips: Do not rely on `has-tooltip` class name for Vue components ```javascript // bad - + Text // good - + Text ``` -1. Tooltips: When using a tooltip, include the tooltip mixin +1. Tooltips: When using a tooltip, include the tooltip directive, `./app/assets/javascripts/vue_shared/directives/tooltip.js` 1. Don't change `data-original-title`. ```javascript -- cgit v1.2.3