-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Fix: Fixed an issue preventing Home to open in Files when clicking on This PC #13039
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
Conversation
Instead of removing file selection code, I think we should keep the file selection functionality by moving Launcher to a separate repository and reinstating the GPL code. |
IIRC someone was not happy with that solution on the basis that since Launcher executable is bundled with Files all Files code woud have to be published with GPL (file selection was not working well regardless). |
I think a separate repository would solve the licensing issue. |
Definitely should, like already seven zip do. |
… opening Home" This reverts commit 1e7f7b8.
I concur. In the meantime this PR just fixes an issue for which opening ThisPC from desktop did not open a new Home tab in Files |
This reverts changes made in #11890. Which is better? |
@ferrariofilippo fyi |
This makes more sense to me but either way is fine. |
Resolved / Related Issues
Closes #issue...
This PR removes leftover code from FilesLaucher related to selected item highlight (removed in #11509). Allows to remove a 500ms delay before opening folders.Also fixes an issue for which opening MyComputer from desktop did not open Files Home.
Validation
How did you test these changes?