Skip to content

Conversation

d2dyno1
Copy link
Member

@d2dyno1 d2dyno1 commented Sep 27, 2021

Resolved / Related Issues

Details of Changes

  • Ctrl+Shift+C will now copy the path at directory

Validation
How did you test these changes?

  • Built the app

Screenshots (optional)
Add screenshots here.

@d2dyno1 d2dyno1 added the ready for review Pull requests that are ready for review label Sep 27, 2021
@gave92
Copy link
Member

gave92 commented Sep 27, 2021

Is this also supposed to allow copying the path of the selected item?

@yaira2 yaira2 requested a review from gave92 September 30, 2021 00:59
yaira2
yaira2 previously approved these changes Sep 30, 2021
@d2dyno1
Copy link
Member Author

d2dyno1 commented Sep 30, 2021

👀 Thank you @gave92! I really need to get my eyes checked...

@gave92
Copy link
Member

gave92 commented Sep 30, 2021

You know? Perhaps you can replace everything with:

case (true, true, false, true, VirtualKey.C):
    SlimContentPage?.CommandsViewModel.CopyPathOfSelectedItemCommand.Execute(null);

@gave92 gave92 added ready to merge Pull requests that are approved and ready to merge and removed ready for review Pull requests that are ready for review labels Sep 30, 2021
@yaira2 yaira2 merged commit 470b128 into files-community:main Sep 30, 2021
@yaira2
Copy link
Member

yaira2 commented Sep 30, 2021

@d2dyno1 thank you!

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.

Add ctrl + shift + c to copy item path

3 participants