Skip to content

Code Quality: Fixed an issue where the "This app can't run on your PC" dialogue displayed inappropriately#18739

Merged
yair100 merged 2 commits into
files-community:mainfrom
Lamparter:193
Jul 19, 2026
Merged

Code Quality: Fixed an issue where the "This app can't run on your PC" dialogue displayed inappropriately#18739
yair100 merged 2 commits into
files-community:mainfrom
Lamparter:193

Conversation

@Lamparter

@Lamparter Lamparter commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Resolved / Related Issues

Steps used to test these changes

  1. Created a file "dnskjdnjkf.exe" with bad binary contents
  2. Opened Files
  3. Opened the executable via Files and observed that the "This app can't run on your PC" dialogue still displays
  4. Opened one of the erroneous file types as listed in Bug: "This app can't run on your PC" dialogue sometimes inappropriately displays #18737 (e.g. an HTML file)
  5. Observed that the file opens correctly and the "This app can't run on your PC" did not appear

Note that creating an executable file with no contents will not result in the dialogue showing anymore. However, this is justified by the fact that in practice, a user will never be opening an executable file with no binary contents unless they are just playing around with their PC.

@Lamparter
Lamparter marked this pull request as ready for review July 19, 2026 06:39
Comment thread src/Files.App/Utils/Shell/LaunchHelper.cs Outdated
@yair100 yair100 added the changes requested Changes are needed for this pull request label Jul 19, 2026
Co-authored-by: yair100 <39923744+yair100@users.noreply.github.com>
Signed-off-by: Jupiter <71598437+Lamparter@users.noreply.github.com>
@Lamparter
Lamparter requested a review from yair100 July 19, 2026 20:18
@yair100 yair100 added ready for review Pull requests that are ready for review and removed changes requested Changes are needed for this pull request labels Jul 19, 2026

@yair100 yair100 left a comment

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.

LGTM

@yair100
yair100 merged commit e3e466b into files-community:main Jul 19, 2026
7 checks passed
@yair100 yair100 added ready to merge Pull requests that are approved and ready to merge and removed ready for review Pull requests that are ready for review labels Jul 19, 2026
@yair100 yair100 changed the title Fix: Fixed an issue where the "This app can't run on your PC" dialogue displayed inappropriately Code Quality: Fixed an issue where the "This app can't run on your PC" dialogue displayed inappropriately Jul 19, 2026
@Lamparter
Lamparter deleted the 193 branch July 19, 2026 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Pull requests that are approved and ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: "This app can't run on your PC" dialogue sometimes inappropriately displays

2 participants