@knime/product-features 1.1.0 → 1.1.1

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @knime/product-features
 
+## 1.1.1
+
+### Patch Changes
+
+- 50d919c: Don't support `remove` action for upload/download items. Instead, items can only be cancelled while in_progress, or they don't have an action otherwise.
+
 ## 1.1.0
 
 ### Patch Changes

package/dist/dataTransfer.css

@@ -119,24 +119,24 @@
 }
 
 .progress-list {
-&[data-v-66f55049] {
+&[data-v-8f14a3cb] {
   max-height: var(--kds-dimension-component-height-20x);
   overflow-y: auto;
 }
-&[data-v-66f55049] .progress-wrapper:not(:first-child) {
+&[data-v-8f14a3cb] .progress-wrapper:not(:first-child) {
     border-top: var(--kds-border-base-subtle);
 }
 }
 
-.upload-panel[data-v-7d4c779e] {
+.upload-panel[data-v-3c6568f7] {
   width: var(--kds-dimension-component-width-20x);
   max-width: var(--kds-dimension-component-width-20x);
 }
-.upload-panel-content[data-v-7d4c779e] {
+.upload-panel-content[data-v-3c6568f7] {
   max-height: var(--kds-dimension-component-height-20x);
 }
 .skeleton-container {
-&[data-v-7d4c779e] {
+&[data-v-3c6568f7] {
   display: grid;
   grid-template-columns: min-content 1fr;
   gap: var(--kds-spacing-container-0-25x);
@@ -145,17 +145,17 @@
   padding: 0 var(--kds-spacing-container-1x);
   border-top: var(--kds-border-base-subtle);
   }
-& .skeleton-text-content[data-v-7d4c779e] {
+& .skeleton-text-content[data-v-3c6568f7] {
     display: flex;
     flex-direction: column;
     gap: var(--kds-spacing-container-0-25x);
 }
 }
 
-.download-panel[data-v-f5f774fd] {
+.download-panel[data-v-cbf5d3c0] {
   width: var(--kds-dimension-component-width-20x);
   max-width: var(--kds-dimension-component-width-20x);
 }
-.download-panel-content[data-v-f5f774fd] {
+.download-panel-content[data-v-cbf5d3c0] {
   max-height: var(--kds-dimension-component-height-20x);
 }

package/dist/dataTransfer.js

@@ -162,14 +162,17 @@ const ProgressItem = /* @__PURE__ */ _export_sfc(_sfc_main$5, [["__scopeId", "da
 
 const _sfc_main$4 = {  };
 
-const _hoisted_1$2 = { class: "progress-list" };
+const _hoisted_1$2 = {
+  class: "progress-list",
+  tabindex: "0"
+};
 
 function _sfc_render(_ctx, _cache) {
   return (openBlock(), createElementBlock("div", _hoisted_1$2, [
     renderSlot(_ctx.$slots, "default", {}, undefined, true)
   ]))
 }
-const ProgressList = /*#__PURE__*/_export_sfc(_sfc_main$4, [['render',_sfc_render],['__scopeId',"data-v-66f55049"]]);
+const ProgressList = /*#__PURE__*/_export_sfc(_sfc_main$4, [['render',_sfc_render],['__scopeId',"data-v-8f14a3cb"]]);
 
 class AbortError extends Error {
 }
@@ -283,10 +286,9 @@ const _sfc_main$3 = /* @__PURE__ */ defineComponent({
   __name: "UploadProgressPanelItem",
   props: {
     item: {},
-    allowCancel: { type: Boolean },
-    allowRemove: { type: Boolean }
+    allowCancel: { type: Boolean }
   },
-  emits: ["remove", "cancel"],
+  emits: ["cancel"],
   setup(__props, { emit: __emit }) {
     const props = __props;
     const emit = __emit;
@@ -327,34 +329,15 @@ const _sfc_main$3 = /* @__PURE__ */ defineComponent({
     const shouldShowCancelAction = computed(
       () => props.allowCancel && props.item.status === "inprogress"
     );
-    const shouldShowRemoveAction = computed(() => {
-      const allowedRemoveStatuses = [
-        "complete",
-        "failed",
-        "cancelled"
-      ];
-      const showRemove = props.allowRemove && props.item.status && allowedRemoveStatuses.includes(props.item.status);
-      return showRemove;
-    });
     const actions = computed(() => {
-      const actions2 = [];
-      if (shouldShowCancelAction.value) {
-        actions2.push({
+      return shouldShowCancelAction.value ? [
+        {
           id: "cancel",
           icon: "x-close",
           label: "Cancel",
           handler: () => emit("cancel")
-        });
-      }
-      if (shouldShowRemoveAction.value) {
-        actions2.push({
-          id: "remove",
-          icon: "trash",
-          label: "Remove",
-          handler: () => emit("remove")
-        });
-      }
-      return actions2;
+        }
+      ] : [];
     });
     return (_ctx, _cache) => {
       return openBlock(), createBlock(unref(ProgressItem), {
@@ -377,13 +360,13 @@ const _sfc_main$2 = /* @__PURE__ */ defineComponent({
   props: /* @__PURE__ */ mergeModels({
     items: {},
     allowCancel: { type: Boolean, default: true },
-    allowRemove: { type: Boolean, default: true },
+    allowRemove: { type: Boolean },
     placeholderItems: { default: 0 }
   }, {
     "expanded": { type: Boolean, ...{ default: true } },
     "expandedModifiers": {}
   }),
-  emits: /* @__PURE__ */ mergeModels(["remove", "cancel", "close"], ["update:expanded"]),
+  emits: /* @__PURE__ */ mergeModels(["cancel", "close"], ["update:expanded"]),
   setup(__props, { emit: __emit }) {
     const emit = __emit;
     const expanded = useModel(__props, "expanded");
@@ -400,14 +383,6 @@ const _sfc_main$2 = /* @__PURE__ */ defineComponent({
         ({ status }) => status === "inprogress" || status === "processing"
       ) || __props.placeholderItems > 0
     );
-    watch(
-      () => __props.items,
-      (newItems, prevItems) => {
-        if (prevItems?.length > 0 && newItems.length === 0) {
-          emit("close");
-        }
-      }
-    );
     return (_ctx, _cache) => {
       return openBlock(), createBlock(unref(CollapsiblePanel), {
         modelValue: expanded.value,
@@ -426,10 +401,8 @@ const _sfc_main$2 = /* @__PURE__ */ defineComponent({
                     key: item.id,
                     item,
                     "allow-cancel": __props.allowCancel,
-                    "allow-remove": __props.allowRemove,
-                    onRemove: ($event) => emit("remove", item),
                     onCancel: ($event) => emit("cancel", item)
-                  }, null, 8, ["item", "allow-cancel", "allow-remove", "onRemove", "onCancel"]);
+                  }, null, 8, ["item", "allow-cancel", "onCancel"]);
                 }), 128))
               ]),
               _: 1
@@ -454,14 +427,14 @@ const _sfc_main$2 = /* @__PURE__ */ defineComponent({
   }
 });
 
-const UploadProgressPanel = /* @__PURE__ */ _export_sfc(_sfc_main$2, [["__scopeId", "data-v-7d4c779e"]]);
+const UploadProgressPanel = /* @__PURE__ */ _export_sfc(_sfc_main$2, [["__scopeId", "data-v-3c6568f7"]]);
 
 const _sfc_main$1 = /* @__PURE__ */ defineComponent({
   __name: "DownloadProgressPanelItem",
   props: {
     item: {}
   },
-  emits: ["remove", "cancel", "download"],
+  emits: ["cancel", "download"],
   setup(__props, { emit: __emit }) {
     const props = __props;
     const emit = __emit;
@@ -481,9 +454,6 @@ const _sfc_main$1 = /* @__PURE__ */ defineComponent({
     });
     const isCancellable = computed(() => props.item.status === "IN_PROGRESS");
     const isReady = computed(() => props.item.status === "READY");
-    const isRemovable = computed(
-      () => props.item.status === "CANCELLED" || props.item.status === "FAILED" || props.item.status === "READY"
-    );
     const actions = computed(() => {
       const actions2 = [];
       if (isCancellable.value) {
@@ -502,14 +472,6 @@ const _sfc_main$1 = /* @__PURE__ */ defineComponent({
           handler: () => emit("download")
         });
       }
-      if (isRemovable.value) {
-        actions2.push({
-          id: "remove",
-          icon: "trash",
-          label: "Remove",
-          handler: () => emit("remove")
-        });
-      }
       return actions2;
     });
     return (_ctx, _cache) => {
@@ -533,7 +495,7 @@ const _sfc_main = /* @__PURE__ */ defineComponent({
     "expanded": { type: Boolean, ...{ default: true } },
     "expandedModifiers": {}
   }),
-  emits: /* @__PURE__ */ mergeModels(["remove", "cancel", "download", "close"], ["update:expanded"]),
+  emits: /* @__PURE__ */ mergeModels(["cancel", "download", "close"], ["update:expanded"]),
   setup(__props, { emit: __emit }) {
     const props = __props;
     const emit = __emit;
@@ -574,10 +536,9 @@ const _sfc_main = /* @__PURE__ */ defineComponent({
                   return openBlock(), createBlock(_sfc_main$1, {
                     key: item.downloadId,
                     item,
-                    onRemove: ($event) => emit("remove", item),
                     onCancel: ($event) => emit("cancel", item),
                     onDownload: ($event) => emit("download", item)
-                  }, null, 8, ["item", "onRemove", "onCancel", "onDownload"]);
+                  }, null, 8, ["item", "onCancel", "onDownload"]);
                 }), 128))
               ]),
               _: 1
@@ -590,14 +551,14 @@ const _sfc_main = /* @__PURE__ */ defineComponent({
   }
 });
 
-const DownloadProgressPanel = /* @__PURE__ */ _export_sfc(_sfc_main, [["__scopeId", "data-v-f5f774fd"]]);
+const DownloadProgressPanel = /* @__PURE__ */ _export_sfc(_sfc_main, [["__scopeId", "data-v-cbf5d3c0"]]);
 
 //#region src/predicate/isPlainObject.ts
 /**
 * Checks if a given value is a plain object.
 *
-* @param {object} value - The value to check.
-* @returns {value is Record<PropertyKey, any>} - True if the value is a plain object, otherwise false.
+* @param value - The value to check.
+* @returns True if the value is a plain object, otherwise false.
 *
 * @example
 * ```typescript
@@ -667,9 +628,9 @@ function isUnsafeProperty(key) {
 *
 * Note that this function mutates the target object.
 *
-* @param {T} target - The target object into which the source object properties will be merged. This object is modified in place.
-* @param {S} source - The source object whose properties will be merged into the target object.
-* @returns {T & S} The updated target object with properties from the source object merged in.
+* @param target - The target object into which the source object properties will be merged. This object is modified in place.
+* @param source - The source object whose properties will be merged into the target object.
+* @returns The updated target object with properties from the source object merged in.
 *
 * @template T - Type of the target object.
 * @template S - Type of the source object.