Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

@uppy/dashboard: add new autoOpen option #5001

Merged
merged 12 commits into from
Mar 25, 2024
Merged

@uppy/dashboard: add new autoOpen option #5001

Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
89b98f4
Add autoOpenView option that defaults to meta
subvertallchris Mar 15, 2024
b474867
<Dashboard/> - deprecate `autoOpenFileEditor`, use `autoOpen` instead
lakesare Mar 18, 2024
7a4cf47
tests - account for the case where `opts` are undefined
lakesare Mar 18, 2024
2a28782
Update packages/@uppy/dashboard/src/Dashboard.jsx
lakesare Mar 19, 2024
900e018
types - properly deprecate option `autoOpenFileEditor`
lakesare Mar 19, 2024
b29f8e7
Merge branch 'auto-open-controls-and-event' of https://github.com/sub…
lakesare Mar 19, 2024
dfe388b
everywhere - rename "fileEditor" => "imageEditor"
lakesare Mar 20, 2024
c0bc1db
merge #2
lakesare Mar 20, 2024
a60ba7d
<DashboardModal/> - refactor `props` passing
lakesare Mar 20, 2024
5b3b19a
types - copypaste types
lakesare Mar 20, 2024
3c1c9b5
Dashboard.tsx - autoOpen: `false` => `null`
lakesare Mar 25, 2024
cffdf39
change the default value too
lakesare Mar 25, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
24 changes: 17 additions & 7 deletions packages/@uppy/dashboard/src/Dashboard.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -88,13 +88,23 @@ export default class Dashboard extends UIPlugin {
showNativePhotoCameraButton: false,
showNativeVideoCameraButton: false,
theme: 'light',
autoOpenFileEditor: false,
autoOpen: false,
disabled: false,
disableLocalFiles: false,
}

// support for the legacy `autoOpenFileEditor` option,
// TODO: we can remove this code when we update the Uppy major version
let autoOpen
if (!opts) {
autoOpen = false
} else if (opts.autoOpen === undefined) {
autoOpen = opts.autoOpenFileEditor ? "imageEditor" : false
} else {
autoOpen = opts.autoOpen
}
// merge default options with the ones set by user
this.opts = { ...defaultOptions, ...opts }
this.opts = { ...defaultOptions, ...opts, autoOpen }

this.i18nInit()

Expand Down Expand Up @@ -751,11 +761,11 @@ export default class Dashboard extends UIPlugin {

const {metaFields} = this.getPluginState()
const isMetaEditorEnabled = metaFields && metaFields.length > 0
const isFileEditorEnabled = this.canEditFile(firstFile)
const isImageEditorEnabled = this.canEditFile(firstFile)

if (isMetaEditorEnabled) {
if (isMetaEditorEnabled && this.opts.autoOpen === 'metaEditor') {
this.toggleFileCard(true, firstFile.id)
} else if (isFileEditorEnabled) {
} else if (isImageEditorEnabled && this.opts.autoOpen === 'imageEditor') {
this.openFileEditor(firstFile)
}
}
Expand Down Expand Up @@ -792,7 +802,7 @@ export default class Dashboard extends UIPlugin {
this.el.addEventListener('keydown', this.handleKeyDownInInline)
}

if (this.opts.autoOpenFileEditor) {
if (this.opts.autoOpen) {
this.uppy.on('files-added', this.#openFileEditorWhenFilesAdded)
}
}
Expand Down Expand Up @@ -823,7 +833,7 @@ export default class Dashboard extends UIPlugin {
this.el.removeEventListener('keydown', this.handleKeyDownInInline)
}

if (this.opts.autoOpenFileEditor) {
if (this.opts.autoOpen) {
this.uppy.off('files-added', this.#openFileEditorWhenFilesAdded)
}
}
Expand Down
2 changes: 2 additions & 0 deletions packages/@uppy/dashboard/types/index.d.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,9 @@ export interface DashboardOptions extends Options {
theme?: 'auto' | 'dark' | 'light'
trigger?: string
width?: string | number
/** @deprecated use option autoOpen instead */
autoOpenFileEditor?: boolean
lakesare marked this conversation as resolved.
Show resolved Hide resolved
autoOpen?: 'metaEditor' | 'imageEditor' | false
disabled?: boolean
disableLocalFiles?: boolean
onRequestCloseModal?: () => void
Expand Down
8 changes: 4 additions & 4 deletions packages/@uppy/react/src/DashboardModal.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ class DashboardModal extends Component {
note,
metaFields, // eslint-disable-line no-shadow
proudlyDisplayPoweredByUppy,
autoOpenFileEditor,
autoOpen,
animateOpenClose,
browserBackButtonClose,
closeAfterFinish,
Expand Down Expand Up @@ -90,7 +90,7 @@ class DashboardModal extends Component {
note,
metaFields,
proudlyDisplayPoweredByUppy,
autoOpenFileEditor,
autoOpen,
animateOpenClose,
browserBackButtonClose,
closeAfterFinish,
Expand Down Expand Up @@ -161,7 +161,7 @@ DashboardModal.propTypes = {
note: PropTypes.string,
metaFields,
proudlyDisplayPoweredByUppy: PropTypes.bool,
autoOpenFileEditor: PropTypes.bool,
autoOpen: PropTypes.oneOf(['imageEditor', 'metaEditor', false]),
animateOpenClose: PropTypes.bool,
browserBackButtonClose: PropTypes.bool,
closeAfterFinish: PropTypes.bool,
Expand Down Expand Up @@ -217,7 +217,7 @@ DashboardModal.defaultProps = {
showRemoveButtonAfterComplete: false,
browserBackButtonClose: false,
theme: 'light',
autoOpenFileEditor: false,
autoOpen: false,
disabled: false,
disableLocalFiles: false,

Expand Down