From 32bdee9df845788abfd4fe0e4486202ae23709ac Mon Sep 17 00:00:00 2001 From: Mark Florian Date: Mon, 28 Jun 2021 20:44:44 +0100 Subject: [PATCH] chore(deps): update dependency @gitlab/eslint-plugin to v9 This change does not fix the various chart types, as they are more complicated, and their test coverage isn't sufficient to be confident about changing them. They can be fixed later. Also, there's a bug in the interaction between our GlPopover wrapper component and the underlying BPopover component: naively replacing `slot="title"` syntax with `#title` breaks title rendering. To work around this, we need to only set up the title slot in GlPopover if the consumer of GlPopover is passing that slot. We do this by iterating over `$scopedSlots`/`$slots`, rather than statically declaring it. The same fix could be achieved with something like: diff --git a/src/components/base/form/form_group/form_group.vue b/src/components/base/form/form_group/form_group.vue index e0d1d048c2..3ebd6c2784 100644 --- a/src/components/base/form/form_group/form_group.vue +++ b/src/components/base/form/form_group/form_group.vue @@ -54,6 +54,8 @@ export default { - + diff --git a/src/components/base/form/form_select/examples/form_select.mixed_options.example.vue b/src/components/base/form/form_select/examples/form_select.mixed_options.example.vue index f0e8dd8e60..f18f84aafe 100644 --- a/src/components/base/form/form_select/examples/form_select.mixed_options.example.vue +++ b/src/components/base/form/form_select/examples/form_select.mixed_options.example.vue @@ -14,7 +14,7 @@ export default {