Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
  • Loading branch information
aduh95 committed Jan 8, 2024
2 parents f62ae48 + 984bdc1 commit 7681b7a
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 7 deletions.
2 changes: 1 addition & 1 deletion .eslintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -471,7 +471,7 @@ module.exports = {
files: ['packages/@uppy/*/src/**/*.ts', 'packages/@uppy/*/src/**/*.tsx'],
excludedFiles: ['packages/@uppy/**/*.test.ts', 'packages/@uppy/core/src/mocks/*.ts'],
rules: {
'@typescript-eslint/explicit-function-return-type': 'error',
'@typescript-eslint/explicit-module-boundary-types': 'error',
},
},
{
Expand Down
4 changes: 2 additions & 2 deletions packages/@uppy/core/src/BasePlugin.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
import Translator from '@uppy/utils/lib/Translator'
import type { I18n, Locale } from '@uppy/utils/lib/Translator'
import type { Body, Meta } from '@uppy/utils/lib/UppyFile'
import type { Uppy } from '.'
import type { State, Uppy } from './Uppy'

export type PluginOpts = {
locale?: Locale
Expand Down Expand Up @@ -103,7 +103,7 @@ export default class BasePlugin<
uninstall(): void {}

// eslint-disable-next-line @typescript-eslint/no-unused-vars
update(state: any): void {}
update(state: Partial<State<M, B>>): void {}

// Called after every state update, after everything's mounted. Debounced.
afterUpdate(): void {}
Expand Down
5 changes: 3 additions & 2 deletions packages/@uppy/core/src/UIPlugin.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ import getTextDirection from '@uppy/utils/lib/getTextDirection'
import type { Body, Meta } from '@uppy/utils/lib/UppyFile'
import BasePlugin from './BasePlugin.ts'
import type { PluginOpts } from './BasePlugin.ts'
import type { State } from './Uppy.ts'

/**
* Defer a frequent call to the microtask queue.
Expand Down Expand Up @@ -48,7 +49,7 @@ class UIPlugin<
M extends Meta,
B extends Body,
> extends BasePlugin<Opts, M, B> {
#updateUI: (state: any) => void
#updateUI: (state: Partial<State<M, B>>) => void

isTargetDOMEl: boolean

Expand Down Expand Up @@ -175,7 +176,7 @@ class UIPlugin<
)
}

update(state: any): void {
update(state: Partial<State<M, B>>): void {
if (this.el != null) {
this.#updateUI?.(state)
}
Expand Down
5 changes: 4 additions & 1 deletion packages/@uppy/form/src/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@ import packageJson from '../package.json'
export default class Form extends BasePlugin {
static VERSION = packageJson.version

/** @type {HTMLFormElement} */
form // TODO: make this private (or at least, mark it as readonly)

constructor (uppy, opts) {
super(uppy, opts)
this.type = 'acquirer'
Expand Down Expand Up @@ -50,7 +53,7 @@ export default class Form extends BasePlugin {
}

if (this.opts.submitOnSuccess) {
this.form.submit()
this.form.requestSubmit()
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ export default (props) => {
return <VideoIcon />
default: {
const { alt } = props
return <img src={itemIconString} alt={alt} loading="lazy" width={16} height={16} />
return <img src={itemIconString} alt={alt} referrerPolicy="no-referrer" loading="lazy" width={16} height={16} />
}
}
}

0 comments on commit 7681b7a

Please sign in to comment.