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

Implement looking for manually installed idea on windows #193

Merged
merged 2 commits into from
Aug 15, 2024

Conversation

lyze237
Copy link
Contributor

@lyze237 lyze237 commented Aug 14, 2024

Hey!

I’ve created a quick PR that adds functionality for the “Open in IntelliJ” button to detect manually installed IntelliJ versions.

The sorting is based on the “last modified” date, so it should ideally open the most recently installed version.

Since Liftoff doesn’t run in a Windows sandbox, I couldn’t fully test it myself. However, I did test it using jshell by copying and pasting the code line by line. I prefer not to install IntelliJ manually on my PC.

It might be beneficial to identify the default install locations for Mac and Linux as well, but I think Linux are good enough to figure out a disabled button might have a tooltip.

@lyze237
Copy link
Contributor Author

lyze237 commented Aug 14, 2024

I just noticed the flatpak check from #159 isn't included, i'll re-add that one too.

@lyze237
Copy link
Contributor Author

lyze237 commented Aug 14, 2024

This needs some testing but should be the same as from the previous PR

Copy link
Member

@tommyettinger tommyettinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me; I guess we don't have any reasonable way to check this on Macs, but working on more Windows and Linux setups is good.

@tommyettinger tommyettinger merged commit 2f34479 into libgdx:master Aug 15, 2024
2 checks passed
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

Successfully merging this pull request may close these issues.

2 participants