Skip to content

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

Closed
wants to merge 4 commits into from

Conversation

gave92
Copy link
Member

@gave92 gave92 commented Jul 22, 2023

Resolved / Related Issues

  • Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
    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?

  • Did you build the app and test your changes?

@hishitetsu
Copy link
Member

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.

@gave92
Copy link
Member Author

gave92 commented Jul 23, 2023

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).

@hishitetsu
Copy link
Member

I think a separate repository would solve the licensing issue.
@0x5bfa what do you think?

@0x5bfa
Copy link
Member

0x5bfa commented Jul 23, 2023

Definitely should, like already seven zip do.

@gave92 gave92 changed the title Code Quality: Remove file selection code from Launcher Fix: Fixed an issue preventing Home to open in Files when clicking on This PC Jul 23, 2023
@gave92
Copy link
Member Author

gave92 commented Jul 23, 2023

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

@hishitetsu
Copy link
Member

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?

@hishitetsu
Copy link
Member

@ferrariofilippo fyi

@gave92
Copy link
Member Author

gave92 commented Jul 23, 2023

NB: This PR restores opening Files Home when user clicks ThisPC on the desktop (Files set as default).

Open "real" This PC shell folder (current main):
image

Or open Files Home (this PR and before #11890):
image

Which is the desired behaviour @yaira2?

@yaira2
Copy link
Member

yaira2 commented Jul 23, 2023

Open "real" This PC shell folder (current main):

This makes more sense to me but either way is fine.

@gave92 gave92 closed this Jul 25, 2023
@gave92 gave92 deleted the no_selection branch July 25, 2023 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants