silverstripe / silverstripe-asset-admin

Silverstripe assets gallery for asset management
BSD 3-Clause "New" or "Revised" License
19 stars 78 forks source link

CMS remembers tab selection, file "used on" doesn't fire when loaded as 'active' tab #1423

Open jules0x opened 7 months ago

jules0x commented 7 months ago

A client reported an issue seeing the "used on" information of a file, and indicated they had to click away and back again to see it.

Files are not immediately showing where they are in use. They will show as not in use anywhere even if they are. It takes selecting them multiple times for them to show where they are in use.

I tested this on their project, then confirmed there is an issue on a fresh project using composer create-project silverstripe/installer test

It appears the “usage” request is triggered by the click on the “used on” tab, or when returning to the file. The effect of this is that the usage will be retrieve when you’ve arrived onto a file and “details” is the active tab, but if “used on” is the active tab, then you’ll need to switch tabs and back, or switch files and back.

"name": "silverstripe/asset-admin", "version": "2.1.2",

GuySartorelli commented 7 months ago

Hi,

Thank you for reporting this. Do you know if this affects version 1.x of this module as well? Or only 2.x?

We have a large backlog at the moment, so this bug likely won't be prioritised by the CMS Squad for a long time. If you'd like to give a go at fixing it we'd love to review a pull request.

jules0x commented 7 months ago

Unsure if affects 1.x