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

[Bug] Allow files with non latin paths to be opened from preview #667

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SkeleyM
Copy link
Contributor

@SkeleyM SkeleyM commented Dec 24, 2024

this change fixes #666 and #350.

entering files full path into cmd opens it in its associated program, and using that instead of "start" fixes the issues.

im assuming it has something to do with "start" being almost 30 years old at this point but this seems to fix it for me

@CyanVoxel CyanVoxel added Type: Bug Something isn't working as intended Priority: High An important issue requiring attention Status: Review Needed A review of this is needed Type: File System File system interactions labels Dec 24, 2024
@CyanVoxel CyanVoxel added this to the Alpha v9.5 (Post-SQL) milestone Dec 24, 2024
@CyanVoxel
Copy link
Member

This works for me, however I was never able to reproduce any of the original issues on my system (Windows 10) so I'd appreciate it if someone else experiencing these issues could help verify this fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: High An important issue requiring attention Status: Review Needed A review of this is needed Type: Bug Something isn't working as intended Type: File System File system interactions
Projects
Status: 🏓 Ready for Review
Development

Successfully merging this pull request may close these issues.

[Bug]: Opening file with an '&' fails
2 participants