@gitlab/ui 114.3.0 → 114.5.0

package/dist/utils/is_slot_empty.js

@@ -28,8 +28,6 @@ function isVnodeEmpty(vnode) {
 function isSlotEmpty(vueInstance, slot, slotArgs) {
   var _vueInstance$$scopedS, _vueInstance$$scopedS2;
   const slotContent = (_vueInstance$$scopedS = (_vueInstance$$scopedS2 = vueInstance.$scopedSlots)[slot]) === null || _vueInstance$$scopedS === void 0 ? void 0 : _vueInstance$$scopedS.call(_vueInstance$$scopedS2, slotArgs);
-
-  // eslint-disable-next-line unicorn/no-array-callback-reference
   return isVnodeEmpty(slotContent);
 }
 

package/dist/utils/use_mock_intersection_observer.js

@@ -24,8 +24,6 @@ class MockObserver {
     this.$_observers = [];
   }
   takeRecords() {}
-
-  // eslint-disable-next-line camelcase
   $_triggerObserve(nodeParam) {
     let {
       entry = {},
@@ -41,8 +39,6 @@ class MockObserver {
       }
     });
   }
-
-  // eslint-disable-next-line camelcase
   $_hasObserver(node) {
     let options = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
     return this.$_observers.some(_ref => {

package/dist/vendor/bootstrap-vue/src/components/table/helpers/mixin-table-renderer.js

@@ -66,7 +66,7 @@ const tableRendererMixin = extend({
       const {
         isResponsive
       } = this;
-      return [this.isStickyHeader ? 'b-table-sticky-header' : '', isResponsive === true ? 'table-responsive' : isResponsive ? `table-responsive-${this.responsive}` : ''].filter(identity);
+      return [this.isStickyHeader ? 'b-table-sticky-header' : '', isResponsive === true ? 'table-responsive' : isResponsive ? `table-responsive table-responsive-${this.responsive}` : ''].filter(identity);
     },
     wrapperStyles() {
       const {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gitlab/ui",
-  "version": "114.3.0",
+  "version": "114.5.0",
   "description": "GitLab UI Components",
   "license": "MIT",
   "main": "dist/index.js",
@@ -66,7 +66,7 @@
     "test:visual:internal": "NODE_ENV=test IS_VISUAL_TEST=true start-test storybook:run http-get://${STORYBOOK_HOST:-localhost}:${STORYBOOK_PORT:-9001}/iframe.html",
     "prettier": "prettier --check '**/*.{js,mjs,ts,vue}'",
     "prettier:fix": "prettier --write '**/*.{js,mjs,ts,vue}'",
-    "eslint": "eslint --max-warnings 0 --ext .js,.vue .",
+    "eslint": "eslint --max-warnings 0 .",
     "eslint:fix": "yarn eslint --fix",
     "stylelint": "stylelint 'src/**/*.scss'",
     "stylelint:fix": "yarn stylelint --fix",
@@ -102,7 +102,8 @@
     "jackspeak": "2.1.1",
     "postcss": "8.5.3",
     "json5": "2.2.3",
-    "rollup-plugin-vue/@vue/component-compiler/postcss-modules-sync/generic-names/loader-utils": "1.4.2"
+    "rollup-plugin-vue/@vue/component-compiler/postcss-modules-sync/generic-names/loader-utils": "1.4.2",
+    "eslint-plugin-import": "2.31.0"
   },
   "devDependencies": {
     "@babel/core": "^7.27.1",
@@ -111,6 +112,8 @@
     "@babel/preset-env": "^7.27.2",
     "@babel/preset-react": "^7.27.1",
     "@cypress/grep": "^4.0.1",
+    "@eslint/eslintrc": "^3.3.1",
+    "@eslint/js": "^9.28.0",
     "@gitlab/eslint-plugin": "20.7.1",
     "@gitlab/fonts": "^1.3.0",
     "@gitlab/stylelint-config": "6.2.2",
@@ -154,10 +157,10 @@
     "emoji-regex": "^10.0.0",
     "entities": "^4.5.0",
     "esbuild": "^0.18.0",
-    "eslint": "8.57.1",
-    "eslint-formatter-gitlab": "^5.1.0",
+    "eslint": "9.28.0",
+    "eslint-formatter-gitlab": "^6.0.0",
     "eslint-import-resolver-jest": "3.0.2",
-    "eslint-plugin-cypress": "3.6.0",
+    "eslint-plugin-cypress": "5.0.1",
     "eslint-plugin-storybook": "0.12.0",
     "fuse.js": "^7.0.0",
     "gitlab-api-async-iterator": "^1.3.1",

package/src/components/base/drawer/drawer.scss

@@ -51,7 +51,7 @@ $gl-drawer-scrim-gradient: linear-gradient(
   .gl-drawer-body > * {
     @apply gl-border-b;
     @apply gl-mx-5;
-  
+
     &:last-child {
       @apply gl-border-none;
     }
@@ -112,6 +112,7 @@ $gl-drawer-scrim-gradient: linear-gradient(
 
 .gl-drawer-body {
   @apply gl-grow;
+  min-height: 0;
   // prevent safari bug where box shadow is visible
   // above the drawer when hovering interactive elements
   // see https://gitlab.com/gitlab-org/gitlab/-/issues/366558

package/src/components/base/toggle/toggle.vue

@@ -92,7 +92,6 @@ export default {
   },
   computed: {
     shouldRenderDescription() {
-      // eslint-disable-next-line @gitlab/vue-prefer-dollar-scopedslots
       return Boolean(this.$scopedSlots.description || this.description) && this.isVerticalLayout;
     },
     shouldRenderHelp() {

package/src/config.js

@@ -29,7 +29,7 @@ try {
   if (glTooltipDelay) {
     tooltipGlobalConfig.delay = JSON.parse(glTooltipDelay);
   }
-} catch (e) {
+} catch {
   // localStorage doesn't exist (or the value is not properly formatted)
 }
 

package/src/directives/safe_link/mock_data.js

@@ -14,7 +14,7 @@ export const javascriptUrls = [
   'javascript:alert("XSS")',
   'jav\tascript:alert("XSS");',
 ];
-/* eslint-disable no-script-url */
+/* eslint-enable no-script-url */
 
 export const encodedJavaScriptUrls = [
   '&#0000106&#0000097&#0000118&#0000097&#0000115&#0000099&#0000114&#0000105&#0000112&#0000116&#0000058&#0000097&#0000108&#0000101&#0000114&#0000116&#0000040&#0000039&#0000088&#0000083&#0000083&#0000039&#0000041',

package/src/directives/safe_link/safe_link.js

@@ -29,7 +29,7 @@ const isSafeURL = (url) => {
   try {
     const parsedURL = new URL(url, getBaseURL());
     return ['http:', 'https:', 'mailto:', 'ftp:'].includes(parsedURL.protocol);
-  } catch (e) {
+  } catch {
     return false;
   }
 };

package/src/utils/is_slot_empty.js

@@ -30,6 +30,5 @@ export function isVnodeEmpty(vnode) {
 export function isSlotEmpty(vueInstance, slot, slotArgs) {
   const slotContent = vueInstance.$scopedSlots[slot]?.(slotArgs);
 
-  // eslint-disable-next-line unicorn/no-array-callback-reference
   return isVnodeEmpty(slotContent);
 }

package/src/utils/use_mock_intersection_observer.js

@@ -1,5 +1,5 @@
 /* global jest, beforeEach, afterEach */
-/* eslint-disable class-methods-use-this, max-classes-per-file, camelcase */
+/* eslint-disable class-methods-use-this, max-classes-per-file */
 import isMatch from 'lodash/isMatch';
 
 /**
@@ -27,7 +27,6 @@ class MockObserver {
 
   takeRecords() {}
 
-  // eslint-disable-next-line camelcase
   $_triggerObserve(nodeParam, { entry = {}, options = {} } = {}) {
     const nodes = this.$_getNodesFromParam(nodeParam);
 
@@ -38,7 +37,6 @@ class MockObserver {
     });
   }
 
-  // eslint-disable-next-line camelcase
   $_hasObserver(node, options = {}) {
     return this.$_observers.some(
       ([obvNode, obvOptions]) => node === obvNode && isMatch(options, obvOptions)

package/src/vendor/bootstrap-vue/src/components/table/_table.scss

@@ -93,8 +93,7 @@
 
 @if $bv-enable-table-sticky {
   .b-table-sticky-header,
-  .table-responsive,
-  [class*="table-responsive-"] {
+  .table-responsive {
     // Move the table bottom margin to the wrapper
     margin-bottom: $spacer;
 
@@ -137,8 +136,7 @@
     // Sticky columns only work when table has sticky
     // headers and/or is responsive
     .b-table-sticky-header,
-    .table-responsive,
-    [class*="table-responsive-"] {
+    .table-responsive {
       > .table.b-table {
         > thead,
         > tbody,

package/src/vendor/bootstrap-vue/src/components/table/helpers/mixin-table-renderer.js

@@ -70,7 +70,7 @@ export const tableRendererMixin = extend({
         isResponsive === true
           ? 'table-responsive'
           : isResponsive
-            ? `table-responsive-${this.responsive}`
+            ? `table-responsive table-responsive-${this.responsive}`
             : ''
       ].filter(identity)
     },