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

Strip whitespace for link input (New Instance > Import) #2146

Closed
1 task done
NovaSagittarii opened this issue Feb 19, 2024 · 0 comments · Fixed by #2592
Closed
1 task done

Strip whitespace for link input (New Instance > Import) #2146

NovaSagittarii opened this issue Feb 19, 2024 · 0 comments · Fixed by #2592
Labels
bug Something isn't working

Comments

@NovaSagittarii
Copy link

NovaSagittarii commented Feb 19, 2024

Operating System

Windows

Version of Prism Launcher

Prism Launcher 8.0

Version of Qt

Qt 6.6.0

Description of bug

I expected the link to be accepted even with leading whitespace.

Link will not be accepted if there is leading whitespace. It works with trailing whitespace though.

Steps to reproduce

  • Add instance
  • Import
  • Put a space
  • Paste a valid link curseforge://install?addonId=715572&fileId=5113947&source=cf_website
    • It does not work.
  • Remove the leading space... it does work.
with extra leading whitespace no extra whitespace
image image

Suspected cause

Leading whitespace is not stripped.

This issue is unique

  • I have searched the issue tracker and did not find an issue describing my bug.
@NovaSagittarii NovaSagittarii added the bug Something isn't working label Feb 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant