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

RichCommands: Open & Close Pane #12002

Merged
merged 4 commits into from
Apr 14, 2023

Conversation

ferrariofilippo
Copy link
Contributor

@ferrariofilippo ferrariofilippo commented Apr 6, 2023

Resolved / Related Issues

Notes
I had to change an hotkey of OpenNewPane (Alt + Shift + Add -> Ctrl + Shift + Add) since there's a conflict with an external shortcut (if you hit Alt + Add SearchBox will be focused).
I searched through all the source code but there's no conflict in it (SearchAction is not executed). The stacktrace shows that the event is raised by ExternalCode.
THIS ISSUE IS NOT RELATED WITH #11837 (Existed even before that commit)

Validation
How did you test these changes?

  • Did you build the app and test your changes?
  • Did you check for accessibility? You can use Accessibility Insights for this.
  • Did you remove any strings from the en-us resource file?
    • Did you search the solution to see if the string is still being used?
  • Did you implement any design changes to an existing feature?
    • Was this change approved?
  • Are there any other steps that were used to validate these changes?
    1. Open app
    2. Use toolbar commands or shortcuts to open and close panes

QuaintMako
QuaintMako previously approved these changes Apr 7, 2023
Copy link
Contributor

@QuaintMako QuaintMako left a comment

Choose a reason for hiding this comment

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

LGTM.

Regarding the change of the shortcut, when the PR is merged, the shortcut on the website documentation will need to be refreshed.

@yaira2
Copy link
Member

yaira2 commented Apr 11, 2023

I had to change an hotkey of OpenNewPane (Alt + Shift + Add -> Ctrl + Shift + Add) since there's a conflict with an external shortcut (if you hit Alt + Add SearchBox will be focused).

I can't reproduce this issue

@ferrariofilippo
Copy link
Contributor Author

Maybe it depends on windows language or some other windows setting 🤔
If I'm the only one having this issue I'll change back the shortcut

@yaira2 yaira2 merged commit 82aec13 into files-community:main Apr 14, 2023
@yaira2 yaira2 added ready to merge Pull requests that are approved and ready to merge and removed needs - code review labels Apr 14, 2023
@ferrariofilippo ferrariofilippo deleted the New_Pane_Commands branch April 14, 2023 13:39
@yaira2 yaira2 mentioned this pull request Apr 14, 2023
78 tasks
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.

Bug: Starting dual-pane mode raises a NotAFolder error in other than English
3 participants