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.

Open
#2,816 10 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Bug
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 :-

  1. Not being able to delete the last file.

https://github.com/processing/p5.js-web-editor/assets/80570105/f38c0437-b8c3-47be-918b-90ba00aa6fa6

  1. 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
  1. We should be able to delete the last file just like other files.
  2. 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

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.