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

Item thumbnails exist but do not load on initial route #1046

Closed
AndrewZachWood opened this issue Mar 9, 2021 · 4 comments · Fixed by #1219
Closed

Item thumbnails exist but do not load on initial route #1046

AndrewZachWood opened this issue Mar 9, 2021 · 4 comments · Fixed by #1219
Assignees
Labels
Milestone

Comments

@AndrewZachWood
Copy link
Contributor

When I generate thumbnails for items they do not load initially. I can refresh the page but even after I've cleared all related cache I have to re-route the the item for the thumbnail to display.

To reproduce:
Submit a new item.
Run the filter media process.
Route to your new item.
Verify the "No Thumbnail" default display.
Press the back button.
Re-route to the your new item.
Notice the thumbnail now displays.

It's unclear to me why re-routing to an item after initially routing to the item would display the thumbnail.

@AndrewZachWood AndrewZachWood added bug needs triage New issue needs triage and/or scheduling labels Mar 9, 2021
@cwilper
Copy link
Contributor

cwilper commented Mar 10, 2021

Note: At first glance this might look similar to #797 but it is different: This is a case where thumbnails that were generated long ago (many restarts and browser cache clears) will not display on initial page display, and only after you browse through the interface are they finally displayed for the user.

@tdonohue tdonohue added this to the 7.0 milestone Apr 16, 2021
@tdonohue tdonohue added component: Item (Archived) Item display or editing low priority and removed needs triage New issue needs triage and/or scheduling labels Apr 16, 2021
@tdonohue
Copy link
Member

@cwilper or @AndrewZWood : Have you done any further analysis of the issue here? This seems like something we may want to solve in 7.0, but also seems low priority. If you don't feel experienced enough to figure out the issue, we could also pull in @artlowel or someone on his team.

For now, moving this to 7.0 board, flagging as low priority & assigning back to @cwilper and @AndrewZWood (please give feedback on whether I should reassign it though)

@tdonohue
Copy link
Member

Per today's meeting, I'm reassigning this to @artlowel so that he can delegate to appropriate team members to analyze the issue.

@tdonohue tdonohue assigned artlowel and unassigned cwilper and AndrewZachWood Apr 22, 2021
@MarisArins
Copy link

Hi, I would like to add an extra scenario that I see and it could be very much related to this bug.

  1. I navigate to an item with thumbnail via links
  2. Thumbnail is displayed in place as supposed.
  3. I hit page refresh (regular and full refresh without cache usage all the same)
  4. Thumbnail does not get loaded (No thumbnail svg in place)

This also has same effect if I pass the item URL to someone else - every attempt to load item fully fails.

P.S.
did not found any thumbnails on demo site, so this is only observed on local installation

@artlowel artlowel mentioned this issue Jun 7, 2021
6 tasks
4science-it pushed a commit to 4Science/dspace-angular that referenced this issue Dec 5, 2023
[DSC-1352] Move breadcrumb char limit property inside layout configuration properties

Approved-by: Andrea Barbasso
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.

5 participants