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

bug-fix (files-app) [wip]: Copying, Deleting, Renaming Files #173

Open
wants to merge 8 commits into
base: dev
Choose a base branch
from

Conversation

kvvarun-mecha
Copy link

@kvvarun-mecha kvvarun-mecha commented Jan 23, 2025

Earlier Copying and pasting/Deleting a file would end up copying and pasting or deleting the parent folder rather than the selected file. This was due to not changing the selected_file state variable when the file_options modal is open,(the selected_file state variable was only changed when we either open a file or when we open a folder only earlier) the solution that i have proposed, changes the value of selected_file whenever the file_options modal is opened, when the modal is closed, it goes back to None, rather than persisting the value (earlier code).

The Rename option had a small bug too, after renaming a folder it would go back to the parent directory, display entries, this is not how usual process, also it had a bug that, when we rename a file, it would go into that file, misinterrupting it as a folder.

Sorry, I forgot to change git config in my local machine, the user who committed is my account only. Most of the additions and deletions are from code formatter, sorry for that

@kvvarun-mecha kvvarun-mecha changed the title bug-fix (files-app): Copying and Deleting Files bug-fix (files-app): Copying, Deleting, Renaming Files Jan 23, 2025
@kvvarun-mecha kvvarun-mecha changed the title bug-fix (files-app): Copying, Deleting, Renaming Files bug-fix (files-app) [wip]: Copying, Deleting, Renaming Files Feb 6, 2025
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.

2 participants