Skip to content

Comments

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

Open
kvvarun-mecha wants to merge 9 commits intomecha-org:devfrom
kvvarun-mecha:dev
Open

bug-fix (files-app) [wip]: Copying, Deleting, Renaming Files #173
kvvarun-mecha wants to merge 9 commits intomecha-org:devfrom
kvvarun-mecha:dev

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