processing / processing/p5.js-web-editor
Being unable to delete the last file and attempting to delete the last folder causes the app to crash.
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 1.7k
- Forks
- 1.7k
- Avg merge
- 3d 4h
- Merged PRs (30d)
- 8
Description
What is your operating system?
Windows
Actual Behavior
The issue can be divided into two parts :-
- Not being able to delete the last file.
https://github.com/processing/p5.js-web-editor/assets/80570105/f38c0437-b8c3-47be-918b-90ba00aa6fa6
- Deleting the last folder causes the app to crash.
https://github.com/processing/p5.js-web-editor/assets/80570105/7a6f8f15-d7dc-402c-8f14-6330234321e0
Hey @lindapaiste , @raclim can you assign this issue to me as I am already working on this.
Expected Behavior
- We should be able to delete the last file just like other files.
- Also we should be able to delete the last folder without any problems.
Steps to reproduce
Steps:
For the first one try to delete all the files that are available.
For the second one create an folder and just some files into it, then delete all root folder files. Now if you delete the created folder the web app would crash.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reproducing both deletion cases on Windows using the steps in the issue: delete all files, then delete the last folder after emptying its contents. No files or tests are named; done means the last file can be deleted and deleting the last folder does not crash the web app.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend, web-dev
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100