PR: Open file pasted into working directory toolbar

[impact27]

Description of Changes

When pasting a file location in the working directory bar, it will go to this file's directory instead of failing, and open the file in the editor. so:
/path/to/my/file.py transforms to /path/to/my/

• Wrote at least one-line docstrings (for any new functions)
• Added unit test(s) covering the changes (if testable)

• Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #15254

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:

[pep8speaks]

Hello @impact27! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

• In the file external-deps/spyder-kernels/spyder_kernels/console/shell.py:

Line 51:80: E501 line too long (80 > 79 characters)

• In the file external-deps/spyder-kernels/spyder_kernels/console/tests/test_console_kernel.py:

Line 420:39: E231 missing whitespace after ':'
Line 585:35: E128 continuation line under-indented for visual indent
Line 597:35: E128 continuation line under-indented for visual indent
Line 608:80: E501 line too long (81 > 79 characters)
Line 609:41: E128 continuation line under-indented for visual indent
Line 1133:69: E272 multiple spaces before keyword
Line 1139:80: E501 line too long (83 > 79 characters)

• In the file spyder/plugins/framesexplorer/widgets/main_widget.py:

Line 216:5: E303 too many blank lines (2)
Line 218:5: E301 expected 1 blank line, found 0

Comment last updated at 2022-07-28 21:55:50 UTC

[ccordoba12]

Hey @impact27, sorry for missing this one and thanks for updating it. Why do you think this is a good idea?

[impact27]

The usecase is: I have a python file path in my clipboard (from a failed test or something else). I want to open this file. There is a path dialog in spyder so pasting it there would make sense. It results in the cwd to be set to the folder containing the file, and the file to open.
It also solves the problem of: I want to go to the folder containing this file.

[ccordoba12]

The usecase is: I have a python file path in my clipboard (from a failed test or something else). I want to open this file. There is a path dialog in spyder so pasting it there would make sense

Hey @impact27, I don't know if you're aware of the %edit magic. That's what I use when I want open a file I have in my clipboard in Spyder.

It results in the cwd to be set to the folder containing the file, and the file to open.
It also solves the problem of: I want to go to the folder containing this file.

Ok, that makes sense but I don't know how discoverable this new UX could be. I understand this would be quite useful to you, but the idea is that others can benefit from it as well (of course, using the %edit magic is not discoverable either).

[impact27]

I guess the argument would be: what should we do if the user pastes a file in the wd box? I think that in that case, the most useful result would be to set the wd to the file dir and open the file.

[ccordoba12]

I guess the argument would be: what should we do if the user pastes a file in the wd box? I think that in that case, the most useful result would be to set the wd to the file dir and open the file.

Very good point. I'm convinced that this is a nice addition, so I'll proceed to review the code.

[ccordoba12]

Last small suggestions for you @impact27 then this should be ready code-wise.

One thing that's missing though is a test for this. Could you also add a simple one? That way we'll be sure this functionality doesn't break in the future.

[ccordoba12]

I'll fix the problem with the Spyder 5/6 changelog files later today. So, don't worry about it.

[ccordoba12]

Looks good to me now, thanks @impact27!