Skip to content

Conversation

winston-de
Copy link
Contributor

@winston-de winston-de commented Mar 5, 2021

Closes #3852.

  • When Files is launched from the command line, the folder will open in a new tab if Files is already open, instead of a new window

  • Changed "when opening from jumplist" text to "When opening folder in Files"

  • This pr also fixes a bug I noticed where "open in new instance" setting was ignored because it was being read from the roaming settings in program.cs, while the setting was saved to local settings.

  • When Files is launched from the start menu with no event args and "open in new instance" is disabled, it will open the current instance (like Edge does)
    NOTE: this will prevent shift+click from opening a new window in the taskbar, so if thats an issue I can undo those changes.

- Removed dependency on args to open new tab instead of instance when launching from start
- Fixed bug where instancing setting was ignored
@winston-de winston-de marked this pull request as ready for review March 5, 2021 23:05
@yaira2 yaira2 requested a review from tsvietOK March 7, 2021 00:48
@tsvietOK
Copy link
Contributor

tsvietOK commented Mar 15, 2021

Hmmm, i guess last change (in description) should be reverted, because i pretty often use middle mouse button click to open new instance of any program

@winston-de
Copy link
Contributor Author

@tsvietOK Ok, changes made.

@tsvietOK tsvietOK added ready to merge Pull requests that are approved and ready to merge and removed needs - code review labels Mar 17, 2021
@yaira2 yaira2 merged commit 2186349 into files-community:main Mar 17, 2021
@winston-de winston-de deleted the InstanceFix branch March 18, 2021 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Pull requests that are approved and ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature request] Change launch handling to allow opening in new tabs instead of new instances for different launch types
3 participants