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

[Bug]: Delayed visual feedback when uploading a large folder via drag and drop #46641

Closed
4 of 8 tasks
kesselb opened this issue Jul 19, 2024 · 8 comments
Closed
4 of 8 tasks

Comments

@kesselb
Copy link
Contributor

kesselb commented Jul 19, 2024

⚠️ This issue respects the following points: ⚠️

Bug description

There's a delay between dragging the folder and a visual feedback.

Steps to reproduce

  1. Have a folder with ~4000 images
  2. Drag the folder to Nextcloud
  3. It takes about 15 seconds until there's some visual feedback

Nextcloud 28:

9664472.28.webm

Nextcloud 27:

9664472.27.webm

Expected behavior

Feedback that the data is processed.

Installation method

None

Nextcloud Server version

master

Operating system

None

PHP engine version

None

Web server

None

Database engine version

None

Is this bug present after an update or on a fresh install?

None

Are you using the Nextcloud Server Encryption module?

None

What user-backends are you using?

  • Default user-backend (database)
  • LDAP/ Active Directory
  • SSO - SAML
  • Other

Configuration report

No response

List of activated Apps

No response

Nextcloud Signing status

No response

Nextcloud Logs

No response

Additional info

logger.debug('Handling recursive file tree', { entry: entry.name })

The above log message appears when you drop the folder and therefore we should be able to provide some visual feedback already (e.g. we processing the folder, stay tuned ....)

@kesselb kesselb added bug 0. Needs triage Pending check for reproducibility or if it fits our roadmap 1. to develop Accepted and waiting to be taken care of regression 28-feedback 29-feedback 30-feedback and removed 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Jul 19, 2024
@susnux
Copy link
Contributor

susnux commented Jul 19, 2024

This is only drag and drop, not on folder upload using the menu?

@kesselb
Copy link
Contributor Author

kesselb commented Jul 22, 2024

This is only drag and drop, not on folder upload using the menu?

Good point.

I don't think it's faster when using the upload files menu.

Screencast.from.2024-07-22.17-07-22.webm

@sorbaugh
Copy link
Contributor

cc @skjnldsv maybe a regression?

@susnux
Copy link
Contributor

susnux commented Aug 16, 2024

This is solved with nextcloud-upload v1.5.0 (once it is released). Tested it locally already and there is no delay anymore.

@susnux
Copy link
Contributor

susnux commented Aug 16, 2024

@kesselb
Copy link
Contributor Author

kesselb commented Aug 16, 2024

Is the fix backportable to 28?

@susnux
Copy link
Contributor

susnux commented Aug 16, 2024

yes

@susnux
Copy link
Contributor

susnux commented Oct 28, 2024

31, 30, 29 fixed. Backport for 28 in #48946

@susnux susnux closed this as completed Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants