Skip to content

Conversation

@mxmou
Copy link
Member

@mxmou mxmou commented Jul 15, 2025

Changes

image

image

image

Reason for changes

Scratch will add a "Set Thumbnail" button on the project page.

Tests

Tested locally on Edge.

The modal normally only opens when the button is clicked for the first time. To open it again, delete the isFirstManualThumbnailUpdate key in localStorage and reload the page.

@mxmou mxmou self-assigned this Jul 15, 2025
@mxmou mxmou added scope: addon Related to one or multiple addons scope: upstream Related to something we depend on (like Scratch, a library, or the browser) labels Jul 15, 2025
@CST1229
Copy link
Member

CST1229 commented Jul 15, 2025

?!!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!??!?!!?!?!?!?!??!?!?!?!?!?!?

@medlans
Copy link

medlans commented Jul 17, 2025

?!!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!?!??!?!!?!?!?!?!??!?!?!?!?!?!?

agreed

@mxmou mxmou marked this pull request as ready for review July 17, 2025 16:20
@mxmou
Copy link
Member Author

mxmou commented Jul 17, 2025

@WorldLanguages I think we should try at least to release this before the Scratch update - otherwise users with dark mode won't be able to read the popup explaining the new feature. The other two PRs aren't as important.

@DNin01
Copy link
Member

DNin01 commented Jul 17, 2025

I think we should try at least to release this before the Scratch update

And remember, we can always release a minor update if v1.43.0 isn't quite ready yet.

Copy link
Member

@DNin01 DNin01 left a comment

Choose a reason for hiding this comment

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

Seems OK at first glance

@mxmou mxmou merged commit ad45c6b into ScratchAddons:master Jul 17, 2025
1 check passed
@mxmou mxmou deleted the wdm-set-thumbnail branch July 17, 2025 18:45
@WorldLanguages
Copy link
Member

@WorldLanguages I think we should try at least to release this before the Scratch update

Sure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: addon Related to one or multiple addons scope: upstream Related to something we depend on (like Scratch, a library, or the browser)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants