Skip to content

Create and rename files and dirs in dirpreview#1156

Merged
esimkowitz merged 18 commits into
mainfrom
evan/0.9-new-file-dir
Dec 3, 2024
Merged

Create and rename files and dirs in dirpreview#1156
esimkowitz merged 18 commits into
mainfrom
evan/0.9-new-file-dir

Conversation

@esimkowitz
Copy link
Copy Markdown
Contributor

@esimkowitz esimkowitz commented Oct 28, 2024

New context menu options are available in the directory preview to create and rename files and directories

It's missing three pieces of functionality, none of which are a regression:

  • Editing or creating an entry does not update the focused index. Focus index right now is pretty dumb, it doesn't factor in the column sorting so if you change that, the selected item will change to whatever is now at that index. We should update this so we use the actual file name to determine which element to focus and let the table determine which index to then highlight given the current sorting algo
  • Open in native preview should not be an option on remote connections with the exception of WSL, where it should resolve the file in the Windows filesystem, rather than the WSL one
  • We should catch CRUD errors in the dir preview and display a popup

@esimkowitz esimkowitz changed the title Evan/0.9 new file dir Create and rename files and dirs in dirpreview Oct 28, 2024
@esimkowitz esimkowitz marked this pull request as ready for review October 28, 2024 04:19
@esimkowitz esimkowitz merged commit 04c4f0a into main Dec 3, 2024
@esimkowitz esimkowitz deleted the evan/0.9-new-file-dir branch December 3, 2024 06:23
xxyy2024 pushed a commit to xxyy2024/waveterm_aipy that referenced this pull request Jun 24, 2025
New context menu options are available in the directory preview to
create and rename files and directories

It's missing three pieces of functionality, none of which are a
regression:
- Editing or creating an entry does not update the focused index. Focus
index right now is pretty dumb, it doesn't factor in the column sorting
so if you change that, the selected item will change to whatever is now
at that index. We should update this so we use the actual file name to
determine which element to focus and let the table determine which index
to then highlight given the current sorting algo
- Open in native preview should not be an option on remote connections
with the exception of WSL, where it should resolve the file in the
Windows filesystem, rather than the WSL one
- We should catch CRUD errors in the dir preview and display a popup
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.

1 participant