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

Not working #44

Closed
github-account1111 opened this issue Mar 15, 2022 · 4 comments
Closed

Not working #44

github-account1111 opened this issue Mar 15, 2022 · 4 comments

Comments

@github-account1111
Copy link
Contributor

I've checked every checkbox in the manager but still don't see any thumbnails:
image

@fire-eggs
Copy link
Owner

I'm sorry to hear you're having difficulties. As a quick check, have you made certain to disable the "Always show icons, never thumbnails" option in Explorer?

DarkThumbs does not show thumbnails "for every image in an archive". It shows a thumbnail for the archive itself.

Based on what you've shown here, a thumbnail might be visible for Temp1_Microsoft_Wallpapers.zip when viewing the Temp folder in Explorer. If you were to extract Microsoft_Office_Icons.zip into Temp, then view Temp in thumbnail view, you'll hopefully see office_icons_01.png as the thumbnail for that archive.

I think you've found a couple of limitations of DarkThumbs:

  1. A thumbnail is not shown for an archive containing archives. I.e. a ZIP file containing an EPUB won't show a thumbnail.
  2. Explorer's "Extract" feature doesn't seem to talk to DarkThumbs.

I'll have to dig into those.

@github-account1111
Copy link
Contributor Author

github-account1111 commented Mar 16, 2022

DarkThumbs does not show thumbnails "for every image in an archive". It shows a thumbnail for the archive itself.

Oh haha my bad.
I didn't understand the purpose of the program then.

I've just checked a cbz and rar file (both are books), and they do have cover thumbnails, whereas I don't think they did before.
So I think it is working then.

Feel free to rename the issue to something more descriptive of the limitations you listed.

@fire-eggs
Copy link
Owner

Oh haha my bad.
I didn't understand the purpose of the program then.

No problem! Is there some way I can clarify the readme to prevent possible confusion in the future?

@fire-eggs
Copy link
Owner

Thank you for the PR!

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

2 participants