Skip to content
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

Add some images #334

Merged
merged 2 commits into from
May 20, 2023
Merged

Add some images #334

merged 2 commits into from
May 20, 2023

Conversation

mybearworld
Copy link
Contributor

Add images for:

  • expanded-backpack
  • middle-click-popup
  • opacity-slider
  • paint-snap
  • pick-colors-from-stage
  • share-through-mystuff
  • sprite-properties

Update images for:

  • hide-project-stats (addon updated)
  • message-filters (full coverage of the addon)

@Wjplatformer
Copy link
Contributor

seems ok, but to really show the essence of each addon, the "extended backpack" image should have like a ton of stuff in it, to show that it IS extended. Just a tip, there are other addons like the share from the mystuff, I think it would be more suitable for it to have the title mystuff and a single project shown instead of just 2 words "delete" and "share". Makes it more clearer.
image

Hope this helps! I can also contribute more images if you ask for them :)

@Wjplatformer
Copy link
Contributor

Mostly should be ok if accepted by SA devs.

@Hans5958 Hans5958 added type: enhancement New feature or request scope: docs Related to the documentation (Scratch Addons Docs) labels May 20, 2023
@Hans5958
Copy link
Member

Related: #4

@Hans5958 Hans5958 added scope: content Related to the contents of the website, excluding blog and docs and removed scope: docs Related to the documentation (Scratch Addons Docs) labels May 20, 2023
@Hans5958
Copy link
Member

Since I've merged #245 (it came earlier), would you please resolve these conflicts (aka replacing the old with the new or reconsider if needed)? Apologizes.

Copy link
Member

@Hans5958 Hans5958 left a comment

Choose a reason for hiding this comment

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

Forgot to do this, please read post before

@mybearworld
Copy link
Contributor Author

As far as I can tell, sprite-properties was the only conflict - did I miss some?

Copy link
Member

@Hans5958 Hans5958 left a comment

Choose a reason for hiding this comment

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

Yeah. Looks good to me. Thanks!

@Hans5958 Hans5958 merged commit 7c02a86 into ScratchAddons:master May 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: content Related to the contents of the website, excluding blog and docs type: enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants