@gitlab/ui 32.44.0 → 32.47.0

package/documentation/documented_stories.js

@@ -100,6 +100,13 @@ export const setupStorybookReadme = () =>
         'GlAvatar',
         'GlAvatarLink',
         'GlPagination',
+        'GlSkeletonLoader',
+        'GlLabel',
+        'GlStackedColumnChart',
+        'GlDiscreteScatterChart',
+        'GlFormCombobox',
+        'GlChartTooltip',
+        'GlInputGroupText',
       ],
       components: {
         GlComponentDocumentation,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gitlab/ui",
-  "version": "32.44.0",
+  "version": "32.47.0",
   "description": "GitLab UI Components",
   "license": "MIT",
   "main": "dist/index.js",

package/src/components/base/form/form_combobox/form_combobox.documentation.js

@@ -1,14 +1,6 @@
-import examples from './examples';
 import * as description from './form_combobox.md';
 
 export default {
   followsDesignSystem: true,
   description,
-  examples,
-  events: [
-    {
-      event: 'value-selected',
-      description: 'Emitted when a value is selected.',
-    },
-  ],
 };

package/src/components/base/form/form_combobox/form_combobox.md

@@ -1,5 +1,3 @@
-# GlFormCombobox
-
 Use this component to add a [`FormInput`](/?path=/story/base-form-form-input--default) component
 with synchronous autocomplete dropdown. It behaves as follows:
 

package/src/components/base/form/form_combobox/form_combobox.stories.js

@@ -1,41 +1,41 @@
-import { withKnobs, object, text } from '@storybook/addon-knobs';
-import { documentedStoriesOf } from '../../../../../documentation/documented_stories';
 import { tokenList, labelText } from './constants';
 import readme from './form_combobox.md';
 import GlFormCombobox from './form_combobox.vue';
 
-const components = {
-  GlFormCombobox,
-};
-
-const getProps = () => {
-  return {
-    tokenList: {
-      type: Array,
-      default: object('tokens', tokenList),
-    },
-    labelText: {
-      type: String,
-      default: text('label text', labelText),
-    },
-  };
-};
+const getProps = () => ({
+  tokenList,
+  labelText,
+});
 
-documentedStoriesOf('base/form/form-combobox', readme)
-  .addDecorator(withKnobs)
-  .add('default', () => ({
-    components,
-    props: getProps(),
-    data: () => {
-      return {
-        value: '',
-      };
-    },
-    template: `
+const Template = (args) => ({
+  components: { GlFormCombobox },
+  data: () => {
+    return {
+      value: '',
+    };
+  },
+  props: Object.keys(args),
+  template: `
       <gl-form-combobox
         v-model="value"
         :token-list="tokenList"
         :labelText="labelText"
       />
     `,
-  }));
+});
+
+export const Default = Template.bind({});
+Default.args = getProps();
+
+export default {
+  title: 'base/form/form-combobox',
+  component: GlFormCombobox,
+  parameters: {
+    knobs: { disable: true },
+    docs: {
+      description: {
+        component: readme,
+      },
+    },
+  },
+};

package/src/components/base/form/form_combobox/form_combobox.vue

@@ -128,6 +128,10 @@ export default {
     selectToken(value) {
       this.$emit('input', value);
       this.closeSuggestions();
+      /**
+       * Emitted when a value is selected.
+       * @event value-selected
+       */
       this.$emit('value-selected', value);
     },
   },

package/src/components/base/form/form_group/form_group.scss

@@ -13,6 +13,10 @@
     }
   }
 
+  .optional-label {
+    @include gl-font-weight-normal;
+  }
+
   .invalid-feedback,
   .text-gl-muted {
     @include gl-font-base;

package/src/components/base/form/form_group/form_group.spec.js

@@ -1,17 +1,17 @@
-import { shallowMount, mount } from '@vue/test-utils';
+import { shallowMount } from '@vue/test-utils';
 import { BFormGroup } from 'bootstrap-vue';
-import GlFormText from '../form_text/form_text.vue';
 import GlFormGroup from './form_group.vue';
 
 describe('Form group component', () => {
   let wrapper;
 
-  const findDescription = () => {
-    return wrapper.find('[data-testid="label-description"]');
-  };
+  const findByTestId = (testId) => wrapper.find(`[data-testid="${testId}"]`);
+
+  const findLabelDescription = () => findByTestId('label-description');
+  const findOptionalLabel = () => findByTestId('optional-label');
 
-  const createComponent = (options, mountFn = shallowMount) => {
-    wrapper = mountFn(GlFormGroup, {
+  const createComponent = (options) => {
+    wrapper = shallowMount(GlFormGroup, {
       ...options,
     });
   };
@@ -35,73 +35,75 @@ describe('Form group component', () => {
     }
   );
 
-  describe('description slot', () => {
-    it('renders text when label-description prop is defined', () => {
-      const labelDescription = 'description via props';
-
-      createComponent(
-        {
-          propsData: {
-            labelDescription,
-          },
-        },
-        mount
-      );
-
-      expect(findDescription().text()).toBe(labelDescription);
-    });
-
-    it('renders a user provided label-description slot', () => {
-      const labelDescription = 'description via slot';
-
-      createComponent(
-        {
-          slots: {
-            'label-description': 'description via slot',
-          },
-          stubs: {
-            GlFormText,
-          },
-        },
-        mount
-      );
-
-      expect(findDescription().text()).toBe(labelDescription);
-    });
+  describe('label-description slot', () => {
+    const labelDescriptionFromPropOptions = {
+      propsData: { labelDescription: 'label-description from props' },
+    };
+    const labelDescriptionFromSlotOptions = {
+      slots: { 'label-description': 'label-description from slots' },
+    };
+
+    it.each`
+      labelDescriptionFrom | options
+      ${'prop'}            | ${labelDescriptionFromPropOptions}
+      ${'prop'}            | ${labelDescriptionFromPropOptions}
+      ${'slot'}            | ${labelDescriptionFromSlotOptions}
+      ${'slot'}            | ${labelDescriptionFromSlotOptions}
+    `(
+      'when label-description is provided from $labelDescriptionFrom',
+      ({ labelDescriptionFrom, options }) => {
+        createComponent({
+          ...options,
+        });
+
+        expect(findLabelDescription().text()).toContain(
+          `label-description from ${labelDescriptionFrom}`
+        );
+      }
+    );
   });
 
   describe('label slot', () => {
-    it('renders text when label prop is defined', () => {
-      const label = 'label via props';
-
-      createComponent(
-        {
+    const labelFromPropOptions = {
+      propsData: { label: 'label from props' },
+    };
+    const labelFromSlotOptions = {
+      slots: { label: 'label from slots' },
+    };
+
+    it.each`
+      labelFrom | options                 | optional | expectedOptional
+      ${'prop'} | ${labelFromPropOptions} | ${true}  | ${true}
+      ${'prop'} | ${labelFromPropOptions} | ${false} | ${false}
+      ${'slot'} | ${labelFromSlotOptions} | ${true}  | ${false}
+      ${'slot'} | ${labelFromSlotOptions} | ${false} | ${false}
+    `(
+      'when label is provided from $labelFrom, `optional` is $optional',
+      ({ labelFrom, options, optional, expectedOptional }) => {
+        createComponent({
+          ...options,
           propsData: {
-            label,
+            ...options.propsData,
+            optional,
           },
-        },
-        mount
-      );
+        });
 
-      expect(wrapper.text()).toBe(label);
-    });
+        expect(wrapper.text()).toContain(`label from ${labelFrom}`);
+        expect(findOptionalLabel().exists()).toBe(expectedOptional);
+      }
+    );
 
-    it('renders a user provided label slot', () => {
-      const label = 'label via slot';
+    it('renders optionalText when `optional` is true', () => {
+      const optionalText = '(not required)';
 
-      createComponent(
-        {
-          slots: {
-            label: 'label via slot',
-          },
-          stubs: {
-            GlFormText,
-          },
+      createComponent({
+        propsData: {
+          optional: true,
+          optionalText,
         },
-        mount
-      );
+      });
 
-      expect(wrapper.text()).toBe(label);
+      expect(findOptionalLabel().text()).toBe(optionalText);
     });
   });
 });

package/src/components/base/form/form_group/form_group.stories.js

@@ -13,6 +13,8 @@ function generateProps({
   label = 'Label Name',
   description = 'form group description',
   labelDescription = 'form label description',
+  optional = GlFormGroup.props.optional.default,
+  optionalText = GlFormGroup.props.optionalText.default,
   horizontal = false,
 } = {}) {
   return {
@@ -36,6 +38,14 @@ function generateProps({
       type: String,
       default: text('label-description', labelDescription),
     },
+    optional: {
+      type: Boolean,
+      default: boolean('optional', optional),
+    },
+    optionalText: {
+      type: String,
+      default: text('optional-text', optionalText),
+    },
     horizontal: {
       type: Boolean,
       default: boolean('horizontal', horizontal),
@@ -53,6 +63,8 @@ documentedStoriesOf('base/form/form-group', readme)
         :id="id"
         :label="label"
         :label-size="labelSize"
+        :optional="optional"
+        :optional-text="optionalText"
         :description="description"
         :horizontal="horizontal"
         label-for="label1"
@@ -69,6 +81,8 @@ documentedStoriesOf('base/form/form-group', readme)
         id="group-id"
         label="Label Name"
         label-size="sm"
+        :optional="optional"
+        :optional-text="optionalText"
         description="This feature is disabled"
         label-for="input1"
       >
@@ -77,14 +91,21 @@ documentedStoriesOf('base/form/form-group', readme)
     `,
   }))
   .add('with textarea', () => ({
+    props: generateProps({ optional: true }),
     template: `
-      <gl-form-group id="group-id-textarea2" label="Label Name" label-for="textarea2">
+      <gl-form-group
+        id="group-id-textarea2"
+        label="Label Name"
+        label-for="textarea2"
+        :optional="optional"
+        :optional-text="optionalText"
+      >
         <gl-form-textarea id="textarea2" placeholder="Enter something" />
       </gl-form-group>
     `,
   }))
   .add('with label description', () => ({
-    props: generateProps(),
+    props: generateProps({ optional: true }),
     components,
     template: `
       <gl-form-group
@@ -93,6 +114,8 @@ documentedStoriesOf('base/form/form-group', readme)
         :label-size="labelSize"
         :description="description"
         :label-description="labelDescription"
+        :optional="optional"
+        :optional-text="optionalText"
         :horizontal="horizontal"
         label-for="label1"
       >
@@ -125,16 +148,16 @@ documentedStoriesOf('base/form/form-group', readme)
       };
     },
     template: `
-    <gl-form-group
-      :id="id"
-      :label="label"
-      :label-size="labelSize"
-      :description="description"
-      :invalid-feedback="invalidFeedback"
-      :state="state"
-      label-for="label1"
-    >
-      <gl-form-input id="input1" :state="state" v-model.trim="name" />
-    </gl-form-group>
+      <gl-form-group
+        :id="id"
+        :label="label"
+        :label-size="labelSize"
+        :description="description"
+        :invalid-feedback="invalidFeedback"
+        :state="state"
+        label-for="label1"
+      >
+        <gl-form-input id="input1" :state="state" v-model.trim="name" />
+      </gl-form-group>
     `,
   }));

package/src/components/base/form/form_group/form_group.vue

@@ -20,6 +20,16 @@ export default {
       required: false,
       default: '',
     },
+    optional: {
+      type: Boolean,
+      required: false,
+      default: false,
+    },
+    optionalText: {
+      type: String,
+      required: false,
+      default: '(optional)',
+    },
   },
   computed: {
     actualLabelClass() {
@@ -45,11 +55,14 @@ export default {
 </script>
 <template>
   <b-form-group v-bind="$attrs" class="gl-form-group" :label-class="actualLabelClass">
-    <template v-if="hasLabelDescription" #label>
+    <template #label>
       <slot name="label">
         {{ $attrs.label }}
+        <span v-if="optional" class="optional-label" data-testid="optional-label">{{
+          optionalText
+        }}</span>
       </slot>
-      <gl-form-text data-testid="label-description">
+      <gl-form-text v-if="hasLabelDescription" data-testid="label-description">
         <slot name="label-description">{{ labelDescription }}</slot>
       </gl-form-text>
     </template>

package/src/components/base/form/input_group_text/input_group_text.documentation.js

@@ -1,8 +1,5 @@
-import examples from './examples';
 import * as description from './input_group_text.md';
 
 export default {
   description,
-  examples,
-  propsInfo: {},
 };

package/src/components/base/form/input_group_text/input_group_text.stories.js

@@ -1,25 +1,31 @@
-import { documentedStoriesOf } from '../../../../../documentation/documented_stories';
 import readme from './input_group_text.md';
 import GlInputGroupText from './input_group_text.vue';
 
-const components = {
-  GlInputGroupText,
-};
-
-const defaultProps = {};
-
-function generateProps(props = {}) {
-  return {
-    ...defaultProps,
-    ...props,
-  };
-}
+const generateProps = (props = {}) => ({
+  ...props,
+});
 
-documentedStoriesOf('base/form/input-group-text', readme).add('default', () => ({
-  components,
-  props: generateProps(),
-  data: () => ({}),
+const Template = (args) => ({
+  components: { GlInputGroupText },
+  props: Object.keys(args),
   template: `
       <gl-input-group-text>Some text</gl-input-group-text>
     `,
-}));
+});
+
+export const Default = Template.bind({});
+Default.args = generateProps();
+
+export default {
+  title: 'base/form/input-group-text',
+  component: GlInputGroupText,
+  parameters: {
+    controls: { disable: true },
+    knobs: { disable: true },
+    docs: {
+      description: {
+        component: readme,
+      },
+    },
+  },
+};

package/src/components/base/label/label.documentation.js

@@ -1,16 +1,6 @@
-import examples from './examples';
+import description from './label.vue';
 
 export default {
   followsDesignSystem: true,
-  examples,
-  events: [
-    {
-      event: 'click',
-      description: 'Emitted when label is clicked',
-    },
-    {
-      event: 'close',
-      description: 'Emitted when x is clicked',
-    },
-  ],
+  description,
 };

package/src/components/base/label/label.md

@@ -1,15 +1,10 @@
-# Label
-
-<!-- STORY -->
-## Usage
-
 Labels are editable objects that allow users to manually categorize other objects, like issues,
 merge requests, and epics. They have a name, description, and a customizable background color.
 They provide a quick way to recognize which categories the labeled object belongs to.
 
 ## Using the component
 
-~~~js
+```js
 <gl-label
   background-color="#D9C2EE"
   title="Label content"
@@ -18,4 +13,4 @@ They provide a quick way to recognize which categories the labeled object belong
   tooltipPlacement="top"
   target="http://some.url"
 />
-~~~
+```