Skip to content

Add menu item to load a file as your image / drawing#2

Open
robertecurtin wants to merge 4 commits intoMarioManTAW:masterfrom
robertecurtin:master
Open

Add menu item to load a file as your image / drawing#2
robertecurtin wants to merge 4 commits intoMarioManTAW:masterfrom
robertecurtin:master

Conversation

@robertecurtin
Copy link

@robertecurtin robertecurtin commented Mar 2, 2026

Overview

My friend had an issue with their browser and lost progress, so I put together a build that allowed them to load an image. I'd like to propose this as a full feature, I'm willing to workshop it as needed.

This adds a menu option File->Load Saved Drawing menu item, which does what File->Open Goal Image does but for the user's image canvas. That way someone who loses progress or changes to another computer / browser can carry their image progress.

You can try out the changes here: https://robertecurtin.github.io/jspaint/

Key changes

  • Updated in-app documentation
  • Added File->Load Saved Drawing menu item, which does what File->Open Goal Image does but for the user's image canvas
    • Added pop-up confirming that you want to do this, in case people confuse this with File->Open Goal Image

Testing

  • Opened a file larger than the canvas space, observed that the image was cut off and additional checks were not awarded
  • Opened a file smaller than the canvas space, observed that the image filled in as much as it could and left the rest of the canvas untouched
    • This is a little strange, but probably desirable
  • Confirmed that cancelling the dialog results in no change to the canvas and returns to the app

Questions for Mario

  • I went with the word "drawing" because that felt more clear than "your image", does that name make sense?
  • Should this be tied to an archipelago setting? You can use it to cheat, so some people might not want to play with it available

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