@gitlab/ui 74.9.0 → 74.9.2

package/dist/tokens/css/tokens.css

@@ -1,6 +1,6 @@
 /**
  * Do not edit directly
- * Generated on Mon, 19 Feb 2024 23:43:48 GMT
+ * Generated on Tue, 20 Feb 2024 13:40:26 GMT
  */
 
 :root {

package/dist/tokens/css/tokens.dark.css

@@ -1,6 +1,6 @@
 /**
  * Do not edit directly
- * Generated on Mon, 19 Feb 2024 23:43:48 GMT
+ * Generated on Tue, 20 Feb 2024 13:40:26 GMT
  */
 
 :root.gl-dark {

package/dist/tokens/js/tokens.dark.js

@@ -1,6 +1,6 @@
 /**
  * Do not edit directly
- * Generated on Mon, 19 Feb 2024 23:43:48 GMT
+ * Generated on Tue, 20 Feb 2024 13:40:26 GMT
  */
 
 export const DATA_VIZ_GREEN_50 = "#133a03";

package/dist/tokens/js/tokens.js

@@ -1,6 +1,6 @@
 /**
  * Do not edit directly
- * Generated on Mon, 19 Feb 2024 23:43:48 GMT
+ * Generated on Tue, 20 Feb 2024 13:40:26 GMT
  */
 
 export const DATA_VIZ_GREEN_50 = "#ddfab7";

package/dist/tokens/scss/_tokens.dark.scss

@@ -1,6 +1,6 @@
 
 // Do not edit directly
-// Generated on Mon, 19 Feb 2024 23:43:48 GMT
+// Generated on Tue, 20 Feb 2024 13:40:26 GMT
 
 $red-950: #fff4f3;
 $red-900: #fcf1ef;

package/dist/tokens/scss/_tokens.scss

@@ -1,6 +1,6 @@
 
 // Do not edit directly
-// Generated on Mon, 19 Feb 2024 23:43:48 GMT
+// Generated on Tue, 20 Feb 2024 13:40:26 GMT
 
 $gl-line-height-52: 3.25rem;
 $gl-line-height-44: 2.75rem;

package/package.json

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

package/src/components/base/new_dropdowns/base_dropdown/base_dropdown.spec.js

@@ -307,7 +307,7 @@ describe('base dropdown', () => {
       // close menu clicking toggle btn
       await toggle.trigger('click');
       expect(menu.classes('gl-display-block!')).toBe(false);
-      expect(toggle.attributes('aria-expanded')).toBeUndefined();
+      expect(toggle.attributes('aria-expanded')).toBe('false');
       expect(wrapper.emitted(GL_DROPDOWN_HIDDEN)).toHaveLength(1);
       expect(toggle.element).toHaveFocus();
     });
@@ -332,7 +332,7 @@ describe('base dropdown', () => {
       await menu.trigger('keydown.esc');
 
       expect(menu.classes('gl-display-block!')).toBe(false);
-      expect(toggle.attributes('aria-expanded')).toBeUndefined();
+      expect(toggle.attributes('aria-expanded')).toBe('false');
       expect(wrapper.emitted(GL_DROPDOWN_HIDDEN)).toHaveLength(1);
       expect(toggle.element).toHaveFocus();
     });
@@ -391,7 +391,7 @@ describe('base dropdown', () => {
       moveFocusWithinDropdown();
       await menu.trigger('keydown.esc');
       expect(menu.classes('gl-display-block!')).toBe(true);
-      expect(toggle.attributes('aria-expanded')).toBeDefined();
+      expect(toggle.attributes('aria-expanded')).toBe('true');
       expect(wrapper.emitted(GL_DROPDOWN_HIDDEN)).toBeUndefined();
       expect(toggle.element).not.toHaveFocus();
     });

package/src/components/base/new_dropdowns/base_dropdown/base_dropdown.vue

@@ -166,7 +166,7 @@ export default {
     ariaAttributes() {
       return {
         'aria-haspopup': this.ariaHaspopup,
-        'aria-expanded': this.visible,
+        'aria-expanded': String(this.visible),
         'aria-controls': this.baseDropdownId,
         'aria-labelledby': this.toggleLabelledBy,
       };

package/src/components/base/table/table.scss

@@ -110,4 +110,14 @@ table.gl-table {
       }
     }
   }
+
+  &.table-hover tbody tr:hover,
+  &.table-hover td.table-secondary:hover {
+    background-color: $gray-10;
+  }
+}
+
+.table.b-table > thead > tr > th,
+.table.b-table > tfoot > tr > th {
+  background-image: none !important;
 }

package/src/components/base/table/table.spec.js

@@ -1,7 +1,9 @@
-import { shallowMount } from '@vue/test-utils';
+import { nextTick } from 'vue';
+import { shallowMount, mount } from '@vue/test-utils';
 import { BTable } from 'bootstrap-vue';
 import { logWarning } from '../../../utils/utils';
 import { waitForAnimationFrame } from '../../../utils/test_utils';
+import Icon from '../icon/icon.vue';
 import { glTableLiteWarning } from './constants';
 import Table from './table.vue';
 
@@ -18,14 +20,15 @@ describe('GlTable', () => {
       <p>Placeholder empty text</p>`,
   };
 
-  const factory = ({ props = {}, scopedSlots = {} } = {}) => {
-    wrapper = shallowMount(Table, {
+  const factory = ({ mountFn = shallowMount, props = {}, scopedSlots = {} } = {}) => {
+    wrapper = mountFn(Table, {
       scopedSlots,
       propsData: props,
     });
   };
 
   const findBTable = () => wrapper.findComponent(BTable);
+  const findFirstColHeader = () => findBTable().find('thead').findAll('th').at(0);
 
   afterEach(() => {
     logWarning.mockClear();
@@ -76,4 +79,69 @@ describe('GlTable', () => {
     expect(wrapper.props('fields')).toEqual(fields);
     expect(findBTable().props('fields')).toEqual(fields);
   });
+
+  describe('sortable columns', () => {
+    const field = {
+      key: 'name',
+      label: 'Name column',
+      sortable: true,
+    };
+
+    describe('without custom slots', () => {
+      beforeEach(() => {
+        factory({ mountFn: mount, props: { fields: [field] } });
+      });
+
+      it('sets the correct column label', () => {
+        expect(findFirstColHeader().text()).toMatch(field.label);
+      });
+
+      it('renders the ascending sort icon', async () => {
+        findFirstColHeader().trigger('click');
+        await nextTick();
+
+        const icon = findFirstColHeader().findComponent(Icon);
+
+        expect(icon.exists()).toBe(true);
+        expect(icon.props('name')).toBe('arrow-up');
+      });
+
+      it('renders the descending sort icon', async () => {
+        findFirstColHeader().trigger('click');
+        findFirstColHeader().trigger('click');
+        await nextTick();
+        const icon = findFirstColHeader().findComponent(Icon);
+
+        expect(icon.exists()).toBe(true);
+        expect(icon.props('name')).toBe('arrow-down');
+      });
+    });
+
+    describe('when headers are customized via slots', () => {
+      const customSlotContent = 'customSlotContent';
+
+      beforeEach(() => {
+        factory({
+          mountFn: mount,
+          props: {
+            fields: [field],
+          },
+          scopedSlots: {
+            'head(name)': `<div>${customSlotContent}</div>`,
+          },
+        });
+      });
+
+      it('renders the ascending sort icon alongside the custom slot content', async () => {
+        findFirstColHeader().trigger('click');
+        await nextTick();
+
+        const icon = findFirstColHeader().findComponent(Icon);
+
+        expect(icon.exists()).toBe(true);
+        expect(icon.props('name')).toBe('arrow-up');
+        expect(findFirstColHeader().text()).toContain(customSlotContent);
+      });
+    });
+  });
 });

package/src/components/base/table/table.stories.js

@@ -44,6 +44,8 @@ export const Default = (args, { argTypes }) => ({
     :fixed="fixed"
     :stacked="stacked"
     :foot-clone="footClone"
+    sort-by="col_2"
+    sort-desc
     hover
     selectable
     selected-variant="primary"
@@ -58,11 +60,12 @@ export const Default = (args, { argTypes }) => ({
       key: 'column_one',
       label: 'Column One',
       variant: 'secondary',
-      sortable: false,
+      sortable: true,
       isRowHeader: false,
     },
     {
       key: 'col_2',
+      sortable: true,
       label: 'Column 2',
       formatter: (value) => value,
     },

package/src/components/base/table/table.vue

@@ -1,6 +1,7 @@
 <!-- eslint-disable vue/multi-word-component-names -->
 <script>
 import { BTable } from 'bootstrap-vue';
+import GlIcon from '../icon/icon.vue';
 import { logWarning, isDev } from '../../../utils/utils';
 import { tableFullSlots, tableFullProps, glTableLiteWarning } from './constants';
 
@@ -17,6 +18,7 @@ export default {
   name: 'GlTable',
   components: {
     BTable,
+    GlIcon,
   },
   inheritAttrs: false,
   props: {
@@ -31,6 +33,22 @@ export default {
       default: false,
       required: false,
     },
+    sortBy: {
+      type: String,
+      required: false,
+      default: undefined,
+    },
+    sortDesc: {
+      type: Boolean,
+      required: false,
+      default: false,
+    },
+  },
+  data() {
+    return {
+      localSortBy: this.sortBy,
+      localSortDesc: this.sortDesc,
+    };
   },
   computed: {
     stickyHeaderClass() {
@@ -39,6 +57,12 @@ export default {
     localTableClass() {
       return ['gl-table', this.tableClass, this.stickyHeaderClass];
     },
+    headSlots() {
+      return [
+        'head()',
+        ...Object.keys(this.$scopedSlots).filter((slotName) => slotName.startsWith('head(')),
+      ];
+    },
   },
   mounted() {
     // logWarning will call isDev before logging any message
@@ -47,13 +71,44 @@ export default {
       logWarning(glTableLiteWarning, this.$el);
     }
   },
+  methods: {
+    isSortable({ field }) {
+      return field?.sortable;
+    },
+    getSortingIcon({ field }) {
+      if (this.localSortBy !== field?.key) {
+        return null;
+      }
+      return this.localSortDesc ? 'arrow-down' : 'arrow-up';
+    },
+  },
 };
 </script>
 
 <template>
-  <b-table :table-class="localTableClass" :fields="fields" v-bind="$attrs" v-on="$listeners">
-    <template v-for="slot in Object.keys($scopedSlots)" #[slot]="scope">
-      <slot :name="slot" v-bind="scope"></slot>
+  <b-table
+    :table-class="localTableClass"
+    :fields="fields"
+    :sort-by.sync="localSortBy"
+    :sort-desc.sync="localSortDesc"
+    v-bind="$attrs"
+    v-on="$listeners"
+  >
+    <template v-for="slotName in Object.keys($scopedSlots)" #[slotName]="scope">
+      <slot :name="slotName" v-bind="scope"></slot>
+    </template>
+    <template v-for="headSlotName in headSlots" #[headSlotName]="scope">
+      <div :key="headSlotName" class="gl-display-flex gl-align-items-center">
+        <slot :name="headSlotName" v-bind="scope">{{ scope.label }}</slot
+        ><template v-if="isSortable(scope)">
+          <gl-icon
+            v-if="getSortingIcon(scope)"
+            :name="getSortingIcon(scope)"
+            class="gl-ml-3 gl-min-w-5 gl-text-gray-900"
+          />
+          <div v-else class="gl-display-inline-block gl-w-5 gl-h-5 gl-ml-3"></div>
+        </template>
+      </div>
     </template>
   </b-table>
 </template>