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

Missing Win32 Icons #19138

Closed
1 task
davidegiacometti opened this issue Jul 2, 2022 · 9 comments
Closed
1 task

Missing Win32 Icons #19138

davidegiacometti opened this issue Jul 2, 2022 · 9 comments
Labels
Issue-Bug Something isn't working Product-PowerToys Run Improved app launch PT Run (Win+R) Window Resolution-Fix Committed Fix is checked in, but it might be 3-4 weeks until a release. Run-Results (Programs) Issue with program search in PowerToys Run

Comments

@davidegiacometti
Copy link
Collaborator

Microsoft PowerToys version

0.60 dev

Running as admin

  • Yes

Area(s) with issue?

PowerToys Run

Steps to reproduce

Originally posted: #19036 (comment)
Xref #7456

Sometimes icons for Win32 programs failed to load.

image

-------------------------- Begin exception --------------------------
Message: Failed to get thumbnail for C:\ProgramData\Microsoft\Windows\Start Menu\Programs\NAPS2\NAPS2.lnk

Exception full name  : System.Runtime.InteropServices.InvalidComObjectException
Exception message    : Error while extracting thumbnail for C:\ProgramData\Microsoft\Windows\Start Menu\Programs\NAPS2\NAPS2.lnk
Exception stack trace:
   at Wox.Infrastructure.Image.WindowsThumbnailProvider.GetHBitmap(String fileName, Int32 width, Int32 height, ThumbnailOptions options) in C:\Users\david\source\repos\PowerToys\src\modules\launcher\Wox.Infrastructure\Image\WindowsThumbnailProvider.cs:line 126
   at Wox.Infrastructure.Image.WindowsThumbnailProvider.GetThumbnail(String fileName, Int32 width, Int32 height, ThumbnailOptions options) in C:\Users\david\source\repos\PowerToys\src\modules\launcher\Wox.Infrastructure\Image\WindowsThumbnailProvider.cs:line 82
   at Wox.Infrastructure.Image.ImageLoader.LoadInternal(String path, Boolean loadFullImage) in C:\Users\david\source\repos\PowerToys\src\modules\launcher\Wox.Infrastructure\Image\ImageLoader.cs:line 183
Exception source     : Wox.Infrastructure
Exception target site: IntPtr GetHBitmap(System.String, Int32, Int32, Wox.Infrastructure.Image.ThumbnailOptions)
Exception HResult    : -2146233049

Exception full name  : System.Runtime.InteropServices.COMException
Exception message    : The data necessary to complete this operation is not yet available. (0x8000000A)
Exception stack trace:

Exception source     : 
Exception target site: 
Exception HResult    : -2147483638
-------------------------- End exception --------------------------

✔️ Expected Behavior

Icons should be displayed

❌ Actual Behavior

Icons are missing

Other Software

No response

@davidegiacometti davidegiacometti added Issue-Bug Something isn't working Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams labels Jul 2, 2022
@davidegiacometti davidegiacometti added Product-PowerToys Run Improved app launch PT Run (Win+R) Window Run-Results (Programs) Issue with program search in PowerToys Run labels Jul 2, 2022
@htcfreek
Copy link
Collaborator

htcfreek commented Jul 2, 2022

@davidegiacometti , @jaimecbernardo
Related to #16625?

@davidegiacometti
Copy link
Collaborator Author

From my investigation the issue isn't really related to the cache, but to the code used to retrieve thumbnails, mostly when executed in parallel (cache).

@htcfreek
Copy link
Collaborator

htcfreek commented Jul 2, 2022

@davidegiacometti
In which situation does the crash happens? Randomly or at the first execution?

@davidegiacometti
Copy link
Collaborator Author

From my test it happens during startup, when images are preloaded from cache in parallel, but not at every run.

@jaimecbernardo
Copy link
Collaborator

I think this is related to having the items in cache, since it seems never replicate at first time after deleting %localappdata%\Microsoft\PowerToys.

@jaimecbernardo jaimecbernardo removed the Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams label Jul 6, 2022
@davidegiacometti
Copy link
Collaborator Author

davidegiacometti commented Jul 6, 2022

@jaimecbernardo it looks related to fetching thumbnails in parallel that of course happens when preloading images from the cache.
Not sure if during a search thumbnails are retrieved in parallel.

@stefansjfw
Copy link
Collaborator

@davidegiacometti @htcfreek I can't repro this? Can you ? We did some changes in the meantime related to thumbnails logic, so it might affected this one too. /needinfo

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Jan 29, 2024
@stefansjfw
Copy link
Collaborator

I coudn't repro what you were seeing , but I found the cause of not getting notepad.exe icon properly : #31170

@jaimecbernardo jaimecbernardo added Resolution-Fix Committed Fix is checked in, but it might be 3-4 weeks until a release. and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Feb 2, 2024
@jaimecbernardo
Copy link
Collaborator

This has been worked on during the 0.79.0 release cycle.
Please update to the latest: https://github.com/microsoft/PowerToys/releases

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue-Bug Something isn't working Product-PowerToys Run Improved app launch PT Run (Win+R) Window Resolution-Fix Committed Fix is checked in, but it might be 3-4 weeks until a release. Run-Results (Programs) Issue with program search in PowerToys Run
Projects
None yet
Development

No branches or pull requests

4 participants