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

[5.x]: Assets Field Type re-arranging asset lagging 5 to 10 seconds in Safari #15728

Closed
zawilliams opened this issue Sep 12, 2024 · 3 comments
Closed
Assignees
Labels

Comments

@zawilliams
Copy link

What happened?

Description

I'm seeing an issue with a client where when having a large amount of assets in an Asset field type (50 items definitely is laggy), the moving around of assets when trying to arrange order completely locks up the page - scrolling seems to work still, but interaction on the whole page freezes. Chrome seems to lag a tiny bit, but Safari is almost unusable.

Anyone encountering this?

Just upgraded to Craft 5 recently and it seems to be since then that this is happening. Went to upgrade the last few minor updates today to see if it fixed things, but no dice.

Included a video to show:

Screen.Recording.2024-09-12.at.5.40.24.PM.mov

Steps to reproduce

  1. Load up 50 assets in an asset field. Drag and drop using the "Reorder" 4 dot diamond.

Expected behavior

Smooth dragging.

Actual behavior

Browser locks up for 5 to 10 seconds.

Craft CMS version

5.4.3

PHP version

8.3

Operating system and version

No response

Database type and version

No response

Image driver and version

No response

Installed plugins and versions

No response

@zawilliams
Copy link
Author

Thanks for looking into this @brianjhanson! Following along.

@brandonkelly
Copy link
Member

brandonkelly commented Sep 27, 2024

We just released Craft 5.4.6 with some improvements to drag-n-drop responsiveness for Safari (#15788). There’s still some work to do, but it’s much more usable now.

@zawilliams
Copy link
Author

Thanks for the fix @brandonkelly!

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

Successfully merging a pull request may close this issue.

3 participants