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

copy and paste omits file extension if (possibly) unknown #2071

Closed
cmburg opened this issue Aug 18, 2022 · 1 comment · Fixed by #2131
Closed

copy and paste omits file extension if (possibly) unknown #2071

cmburg opened this issue Aug 18, 2022 · 1 comment · Fixed by #2131
Labels
Status: Confirmed The occurrence and circumstances of the issue have been verified

Comments

@cmburg
Copy link

cmburg commented Aug 18, 2022

What Happened?

I have a jupyter notebook which I copy-pasted via ctrt+c and ctrl+v in pantheon files. Although the text (copy) has been added, the extension .ipynb was not. This issue does not appear on more common extensions as .txt or .py

Steps to Reproduce

  1. Create a file, e.g, foo.ipynb via the right mouse-button menu.
  2. Select the file and press ctrl+c, ctrl+v

Expected Behavior

The extension is kept

OS Version

6.x (Odin)

Software Version

Latest release (I have run all updates)

Log Output

No response

Hardware Info

No response

@jeremypw jeremypw added the Status: Confirmed The occurrence and circumstances of the issue have been verified label Feb 8, 2023
@jeremypw
Copy link
Collaborator

jeremypw commented Feb 8, 2023

There is a note in the code: /* Strings longer than 4 or shorter than 1 are not regarded as extensions */
I guess this limit could be increased to 5, however there may be some filenames containing . where the last 5 chars are not in fact an extension. I do not think it is practical to try to maintain a list of "recognized extensions".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Confirmed The occurrence and circumstances of the issue have been verified
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants