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

Insane long page due to folder listing #203

Closed
arifje opened this issue May 17, 2020 · 7 comments
Closed

Insane long page due to folder listing #203

arifje opened this issue May 17, 2020 · 7 comments

Comments

@arifje
Copy link

arifje commented May 17, 2020

Screenshot 2020-05-17 at 10 17 41

Also takes some serious time to load in our case...

@khalwat
Copy link
Contributor

khalwat commented May 26, 2020

How/why are there so many folders in this asset volume?

I guess I could have an option to disable this feature if you like?

@saltymouse
Copy link

This came up for me as well for optimized images in blog posts. The image asset field's "Default Asset Location" is set to <VolumeName>: {owner.slug} for organizational purposes (knowing which image belongs to which blog post).

A "Show more/less" toggle after 10 or so items might work.

@khalwat
Copy link
Contributor

khalwat commented Apr 14, 2021

Maybe I can just not display them if there are, say, > 20 of them (because it's really unlikely you're going to be doing anything per-subfolder in that case)?

@saltymouse
Copy link

That would work for me. Indeed, I think it's safe to assume sub-folders would just inherit optimizations from the parent folder if it gets that long.

@khalwat
Copy link
Contributor

khalwat commented Apr 14, 2021

You can completely disable this via the Plugin Settings too.

@saltymouse
Copy link

I still have a few sub-folder optimizations I'm using on other volumes, so I still need that setting for now. I'll look into that setting ("Limit by Sub-Folder") if the sub-folders get too unwieldy, in which case I'll reorganize those remaining sub-folders into their own asset volumes, which will allow me to turn off that setting.

khalwat pushed a commit that referenced this issue Apr 15, 2021
@khalwat
Copy link
Contributor

khalwat commented Apr 15, 2021

Fixed in 400b083

@khalwat khalwat closed this as completed Apr 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants