Skip to content

Fixed an issue where file thumbnails failed to appear after saving a …#18466

Open
workbysaran wants to merge 1 commit into
files-community:mainfrom
workbysaran:sg/fix-thumbnail-retry-timer-fallback
Open

Fixed an issue where file thumbnails failed to appear after saving a …#18466
workbysaran wants to merge 1 commit into
files-community:mainfrom
workbysaran:sg/fix-thumbnail-retry-timer-fallback

Conversation

@workbysaran
Copy link
Copy Markdown
Contributor

Fixed an issue where file thumbnails failed to appear after saving a file using Chrome "Save as PDF", Adobe PDF printer, or Microsoft PDF printer.

This fix adds a 2-second timer-based self-retry directly in LoadThumbnailAsync as a fallback for when no FILE_ACTION_MODIFIED event follows a thumbnail failure. Diagnostic logging also added to investigate the issue.

Closes #18352

Steps used to test these changes

  1. Navigate to a folder in Files
  2. Print to PDF from Chrome using "Save as PDF" into that folder and verify thumbnail appears within ~2 seconds
  3. Repeat using Adobe PDF printer or Microsoft PDF printer and verify whether thumbnail appears

…file such as using Chrome "Save as PDF", Adobe PDF printer, or Microsoft PDF printer.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks good, can you please add some code comments?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The situation is not improving. I have absolutely no idea why.
Files 4.1.0.0
2026-05-11_09h56_54

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yamachari2025-ops if you're using v4.1, this hasn't been released yet. Did you build the source code for this PR?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm sorry. I'm not a programmer, I'm just a user.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No problem, hopefully this PR fixes the issue for the next release. If not, make sure to let us know.

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

Successfully merging this pull request may close these issues.

Bug: PDF thumbnail not displaying until refresh

3 participants