Skip to content

Added a new clear grid button functionality to enhance user experience when … #569

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Mereck600
Copy link

…drawing new bitmaps. When trying to use the bitmap for one of the projects, I found an issue where the bitmap was not displayed on the screen upon first opening the bitmap editor. This is likely due to something in the initialization, however, adding a console log (initially used for debugging) in the Init function solved this issue. I think it just bought enough time. That is not an optimal solution however, it works for now.

…drawing new bitmaps, found an issue where the bitmap was not displayed on the screen. This is likely due to something in the initialization, however, adding a console log which was originally just used for debugging solved this issue. That is not the optimal solution however, it works for now.
@ErikUmble
Copy link

I also noticed this problem and will look into fixing it with perhaps a more robust approach.

@ErikUmble
Copy link

Yes, you were right about the issue being in the initialization. I think the problem was that the startup code assumes grid exists in localStorage while it might not actually. I decided to submit a different pull request (#579) because there were other features (such as importing png and bmp files) that should really be brought over to this platform. That pull request should resolve the loading issue.

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