@uppy/provider-views - npm Package Compare versions

Comparing version 4.0.0-beta.8 to 4.0.0-beta.9

CHANGELOG.md

 # @uppy/provider-views
 
-<<<<<<< HEAD
+## 4.0.0-beta.9
+
+Released: 2024-06-18
+Included in: Uppy v4.0.0-beta.12
+
+- @uppy/provider-views: Renames & `eslint-disable react/require-default-props` removal (Evgenia Karunus / #5251)
+- examples,@uppy/locales,@uppy/provider-views,@uppy/transloadit: Release: uppy@3.26.1 (github-actions[bot] / #5242)
+- @uppy/provider-views: fix wrong font for files (Merlijn Vos / #5234)
+
 ## 4.0.0-beta.7
@@ -28,3 +36,2 @@
 
-=======
 ## 3.12.1
@@ -36,3 +43,2 @@
 - @uppy/provider-views: fix wrong font for files (Merlijn Vos / #5234)
->>>>>>> a89009dbf007b04caedfb84c7ef3fcc7e8bb1698
 
@@ -39,0 +45,0 @@ ## 3.12.0

lib/Browser.d.ts

@@ -33,3 +33,3 @@ import { h } from 'preact';
     noResultsLabel: string;
-    loadAllFiles?: boolean;
+    virtualList?: boolean;
 };
@@ -36,0 +36,0 @@ declare function Browser<M extends Meta, B extends Body>(props: BrowserProps<M, B>): h.JSX.Element;

lib/Browser.js

@@ -51,3 +51,3 @@ /* eslint-disable react/require-default-props */
     author: f.author,
-    getItemIcon: () => f.icon,
+    getItemIcon: () => viewType === 'grid' && f.thumbnail ? f.thumbnail : f.icon,
     isChecked: isChecked(f),
@@ -93,3 +93,3 @@ toggleCheckbox: event => toggleCheckbox(event, f),
     noResultsLabel,
-    loadAllFiles
+    virtualList
   } = props;
@@ -125,3 +125,3 @@ const selected = currentSelection.length;
     }
-    if (loadAllFiles) {
+    if (virtualList) {
       return h("div", {
@@ -128,0 +128,0 @@ className: "uppy-ProviderBrowser-body"

lib/Item/components/ItemIcon.d.ts

 import { h } from 'preact';
-export default function ItemIcon(props: {
+type ItemIconProps = {
     itemIconString: string;
     alt?: string;
-}): h.JSX.Element | null;
+};
+export default function ItemIcon({ itemIconString, alt, }: ItemIconProps): h.JSX.Element | null;
+export {};
 //# sourceMappingURL=ItemIcon.d.ts.map

lib/Item/components/ItemIcon.js

@@ -1,2 +0,1 @@
-/* eslint-disable react/require-default-props */
 import { h } from 'preact';
@@ -44,6 +43,7 @@ function FileIcon() {
 }
-export default function ItemIcon(props) {
-  const {
-    itemIconString
-  } = props;
+export default function ItemIcon(_ref) {
+  let {
+    itemIconString,
+    alt = undefined
+  } = _ref;
   if (itemIconString === null) return null;
@@ -59,5 +59,2 @@ switch (itemIconString) {
       {
-        const {
-          alt
-        } = props;
         return h("img", {
@@ -64,0 +61,0 @@ src: itemIconString,

lib/Item/index.js

@@ -6,4 +6,4 @@ function _extends() { return _extends = Object.assign ? Object.assign.bind() : function (n) { for (var e = 1; e < arguments.length; e++) { var t = arguments[e]; for (var r in t) ({}).hasOwnProperty.call(t, r) && (n[r] = t[r]); } return n; }, _extends.apply(null, arguments); }
 import ItemIcon from "./components/ItemIcon.js";
-import GridListItem from "./components/GridLi.js";
-import ListItem from "./components/ListLi.js";
+import GridItem from "./components/GridItem.js";
+import ListItem from "./components/ListItem.js";
 export default function Item(props) {
@@ -30,3 +30,3 @@ const {
     case 'grid':
-      return h(GridListItem, _extends({}, props, {
+      return h(GridItem, _extends({}, props, {
         className: className,
@@ -41,3 +41,3 @@ itemIconEl: itemIconEl
     case 'unsplash':
-      return h(GridListItem, _extends({}, props, {
+      return h(GridItem, _extends({}, props, {
         className: className,
@@ -44,0 +44,0 @@ itemIconEl: itemIconEl

lib/ProviderView/AuthView.d.ts

@@ -14,4 +14,4 @@ import { h } from 'preact';
 };
-export default function AuthView<M extends Meta, B extends Body>(props: AuthViewProps<M, B>): h.JSX.Element;
+export default function AuthView<M extends Meta, B extends Body>({ loading, pluginName, pluginIcon, i18n, handleAuth, renderForm, }: AuthViewProps<M, B>): h.JSX.Element;
 export {};
 //# sourceMappingURL=AuthView.d.ts.map

lib/ProviderView/AuthView.js

@@ -1,2 +0,1 @@
-/* eslint-disable react/require-default-props */
 import { h } from 'preact';
@@ -77,4 +76,4 @@ import { useCallback } from 'preact/hooks';
 };
-export default function AuthView(props) {
-  const {
+export default function AuthView(_ref3) {
+  let {
     loading,
@@ -86,3 +85,3 @@ pluginName,
     renderForm = defaultRenderForm
-  } = props;
+  } = _ref3;
   return h("div", {
@@ -89,0 +88,0 @@ className: "uppy-Provider-auth"

lib/ProviderView/ProviderView.d.ts

@@ -16,2 +16,3 @@ import { h } from 'preact';
     loadAllFiles: boolean;
+    virtualList: boolean;
 };
@@ -25,2 +26,3 @@ export interface ProviderViewOptions<M extends Meta, B extends Body> extends ViewOptions<M, B, PluginType> {
     }) => h.JSX.Element;
+    virtualList?: boolean;
 }
@@ -27,0 +29,0 @@ type Opts<M extends Meta, B extends Body> = DefinePluginOpts<ProviderViewOptions<M, B>, keyof typeof defaultOptions>;

lib/ProviderView/ProviderView.js

@@ -16,3 +16,3 @@ function _classPrivateFieldLooseBase(e, t) { if (!{}.hasOwnProperty.call(e, t)) throw new TypeError("attempted to use private field on non-instance"); return e; }
 const packageJson = {
-  "version": "4.0.0-beta.8"
+  "version": "4.0.0-beta.9"
 };
@@ -42,3 +42,4 @@ function formatBreadcrumbs(breadcrumbs) {
   showBreadcrumbs: true,
-  loadAllFiles: false
+  loadAllFiles: false,
+  virtualList: false
 };
@@ -430,2 +431,3 @@ var _abortController = /*#__PURE__*/_classPrivateFieldLooseKey("abortController");
       loadAllFiles: this.opts.loadAllFiles,
+      virtualList: this.opts.virtualList,
       // For SearchFilterInput component
@@ -432,0 +434,0 @@ showSearchFilter: targetViewOptions.showFilter,

lib/SearchProviderView/SearchProviderView.js

@@ -13,3 +13,3 @@ function _classPrivateFieldLooseBase(e, t) { if (!{}.hasOwnProperty.call(e, t)) throw new TypeError("attempted to use private field on non-instance"); return e; }
 const packageJson = {
-  "version": "4.0.0-beta.8"
+  "version": "4.0.0-beta.9"
 };
@@ -16,0 +16,0 @@ const defaultState = {

lib/View.js

@@ -1,3 +0,1 @@
-import getFileType from '@uppy/utils/lib/getFileType';
-import isPreviewSupported from '@uppy/utils/lib/isPreviewSupported';
 import remoteFileObjToLocal from '@uppy/utils/lib/remoteFileObjToLocal';
@@ -111,6 +109,3 @@
     };
-    const fileType = getFileType(tagFile);
-
-    // TODO Should we just always use the thumbnail URL if it exists?
-    if (fileType && isPreviewSupported(fileType)) {
+    if (file.thumbnail) {
       tagFile.preview = file.thumbnail;
@@ -117,0 +112,0 @@ }

package.json

 {
   "name": "@uppy/provider-views",
   "description": "View library for Uppy remote provider plugins.",
-  "version": "4.0.0-beta.8",
+  "version": "4.0.0-beta.9",
   "license": "MIT",
@@ -29,5 +29,5 @@ "main": "lib/index.js",
   "peerDependencies": {
-    "@uppy/core": "^4.0.0-beta.9"
+    "@uppy/core": "^4.0.0-beta.10"
   },
   "stableVersion": "3.11.0"
 }

src/View.ts

@@ -7,4 +7,2 @@ import type {
 import type { CompanionFile } from '@uppy/utils/lib/CompanionFile'
-import getFileType from '@uppy/utils/lib/getFileType'
-import isPreviewSupported from '@uppy/utils/lib/isPreviewSupported'
 import remoteFileObjToLocal from '@uppy/utils/lib/remoteFileObjToLocal'
@@ -146,6 +144,3 @@
 
-    const fileType = getFileType(tagFile)
-
-    // TODO Should we just always use the thumbnail URL if it exists?
-    if (fileType && isPreviewSupported(fileType)) {
+    if (file.thumbnail) {
       tagFile.preview = file.thumbnail
@@ -152,0 +147,0 @@ }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Worsened metrics

Total package byte prevSize

330948

decreased by-0.04%

Lines of code

4658

decreased by-0.06%

No dependency changes