Skip to content

Conversation

@microbit-grace
Copy link
Contributor

@microbit-grace microbit-grace commented Aug 1, 2025

Recover missing image is a screenshot of the Web tool page

Fixes #45

@microbit-carlos
Copy link
Collaborator

microbit-carlos commented Aug 1, 2025

It makes the README easier to read and it clearly shows it's not just a library and that it also includes the online tools.
Is there a reason not to include the screenshot? Would it get deleted automatically every time the docs are re-generated? If that's the case, do we need to do something to make sure the minified js file doesn't get deleted again?

@microbit-grace
Copy link
Contributor Author

It makes the README easier to read and it clearly shows it's not just a library and that it also includes the online tools. Is there a reason not to include the screenshot? Would it get deleted automatically every time the docs are re-generated? If that's the case, do we need to do something to make sure the minified js file doesn't get deleted again?

If you think strongly about including the image, I can add it back. I just thought that the text seemed sufficient in saying what the image says

@microbit-carlos
Copy link
Collaborator

If there isn't a reason not to include (and there could be plenty!) I would prefer to keep it, yes.

@microbit-grace microbit-grace changed the title Remove missing image in README.md Recover missing web tool screenshot in README.md Aug 1, 2025
@microbit-grace
Copy link
Contributor Author

@microbit-carlos Have recovered the missing image in the latest two commits

Copy link
Collaborator

@microbit-carlos microbit-carlos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks Grace!

@microbit-carlos microbit-carlos merged commit be72854 into main Aug 1, 2025
3 checks passed
@microbit-carlos microbit-carlos deleted the fix-missing-img branch August 1, 2025 10:24
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.

README image missing in "Web tool" section

3 participants