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

PR: Fix "Show in external file explorer" for non-existing file (Windows) #17064

Merged
merged 1 commit into from
Dec 23, 2021

Conversation

rear1019
Copy link
Contributor

Description of Changes

See commit message for details.

PS: The same action in right-click menu of editor’s tabs is possibly affected as well.

Affirmation

By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct:

rear1019

The action can be found in the menu of the editor’s tabbar. It did not
work under Windows for newly created files which haven’t beened saved
yet.
@dalthviz dalthviz changed the title Fix “Show in external file explorer” for non-existing file (Windows) PR: Fix “Show in external file explorer” for non-existing file (Windows) Dec 22, 2021
@dalthviz dalthviz added this to the v5.2.2 milestone Dec 22, 2021
@ccordoba12
Copy link
Member

Hey @rear1019, thanks for your help with this! You said:

PS: The same action in right-click menu of editor’s tabs is possibly affected as well.

Do you plan to check, and if failing fix, that as part of this pull request?

@rear1019
Copy link
Contributor Author

Hey @rear1019, thanks for your help with this! You said:

PS: The same action in right-click menu of editor’s tabs is possibly affected as well.

Do you plan to check, and if failing fix, that as part of this pull request?

Just confirmed that the patch also fixes the action in the right-click menu of editor’s tabs (the button in the right corner of the tabbar displays the same menu object).

@ccordoba12
Copy link
Member

Great! Thanks for confirming that @rear1019!

Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

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

Thanks for your help with this @rear1019!

@ccordoba12 ccordoba12 changed the title PR: Fix “Show in external file explorer” for non-existing file (Windows) PR: Fix "Show in external file explorer" for non-existing file (Windows) Dec 23, 2021
@ccordoba12 ccordoba12 merged commit 8292f46 into spyder-ide:5.x Dec 23, 2021
ccordoba12 added a commit that referenced this pull request Dec 23, 2021
@rear1019 rear1019 deleted the fix_external_explorer_windows branch April 27, 2022 06:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants