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.

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

Dashboard: "showProgressDetails: true" not working #2481

Closed
Grovespaz opened this issue Aug 26, 2020 · 3 comments
Closed

Dashboard: "showProgressDetails: true" not working #2481

Grovespaz opened this issue Aug 26, 2020 · 3 comments
Assignees
Labels
UI User Interface issues

Comments

@Grovespaz
Copy link

I've set the Dashboard up to show progress details, but it does not show:
afbeelding
aining upload size and time, set this to true.

I would expect it to say "Uploading: 45%・43 MB of 101 MB・8s left" instead of "Uploading: 45%", as per the documentation

I've included the full code I use to setup Uppy just to be complete:

const uppy = Uppy({
	restrictions: {
		allowedFileTypes: ['video/*'],
	},
	onBeforeUpload: (files) => {
		const updatedFiles = Object.assign({}, files)
		Object.keys(updatedFiles).forEach(fileId => {
			(updatedFiles[fileId].meta as any).categoryId = this.$props.category;
		})
		return updatedFiles
	} 
}).use(Dashboard, {
	target: this.$refs.uploadThing,
	inline: true,
	showLinkToFileUploadResult: false,
	showProgressDetails: true,
	proudlyDisplayPoweredByUppy: false,
	disableThumbnailGenerator: true,
}).use(Tus, {
	endpoint: process.env.backendUrl + '/files/',
	resume: true,
	retryDelays: [0, 1000, 3000, 5000],
	limit: 5,
	metadata: {
		categoryId: '123123',
	}
})
@paulbellamy
Copy link

I just got bit by the same thing when using the React StatusBar component. After some digging, I think it is because I had it mounted inside a container of the wrong size, but I can't quite track down how the containerWidth prop gets initialized. There's a check if it doesn't have the .uppy-size--md html class, the additional info line gets display: none. I worked around it for now by adding the CSS:

.uppy-Root:not(.uppy-size--md) .uppy-StatusBar-additionalInfo {
  display: block;
}

/* Stolen from the styles.css to make sure the statusBar is tall enough to hold both lines. */
.uppy-StatusBar {
  height: 46px;
}

@goto-bus-stop
Copy link
Contributor

Right, it looks like this is intended behaviour to prevent cluttering the mobile UI, but maybe we can be a bit more granular about the sizing constraints…

@goto-bus-stop goto-bus-stop added UI User Interface issues and removed Bug Triage labels Dec 7, 2020
@arturi arturi self-assigned this Dec 22, 2020
@Murderlon
Copy link
Member

Closed in #3174

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI User Interface issues
Projects
None yet
Development

No branches or pull requests

5 participants