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

Collision Tags #889

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

LilyMakesThings
Copy link
Contributor

Collision Tags

This is intended to serve as an accompaniment (or perhaps a replacement) for Clones Plus' lackluster "touching" blocks.

image

You can apply multiple Collision Tags to a sprite, which allows users to index their clones far easier. And of course, tags are not limited to a single sprite. You can touch other sprites with a specified collision tag.
image

Most of the issues of Clones Plus could be solved by using this system. In fact, nothing is stopping this from being Clones Plus 2. Feedback is appreciated on this draft.

@NexusKitten

This comment was marked as spam.

@LilyMakesThings
Copy link
Contributor Author

Congrats on passing eslint first try

image

@LilyMakesThings
Copy link
Contributor Author

Changes:

  • Clones now copy tags from parent
  • Added more options for specifying targets

To do:

  • What else do people want out of this

@softedco
Copy link
Contributor

Might I suggest adding a <tag () touches tag ()?> block?

@LilyMakesThings
Copy link
Contributor Author

Might I suggest adding a <tag () touches tag ()?> block?

That's a good idea!

@LilyMakesThings
Copy link
Contributor Author

In the latest commit I removed the "onTargetCreated" message that would duplicate tags between a parent and a clone, as it appeared to copy that data across all targets for some reason. I will add it back eventually, but for now it's a temporary fix.

@UndermanReal
Copy link

Is this still being worked on?

@LilyMakesThings
Copy link
Contributor Author

Yes

@LilyMakesThings LilyMakesThings marked this pull request as ready for review October 10, 2023 20:10
@LilyMakesThings LilyMakesThings changed the title Collision Tags [work in progress] Collision Tags May 10, 2024
@LilyMakesThings LilyMakesThings marked this pull request as draft July 1, 2024 17:23
@LilyMakesThings
Copy link
Contributor Author

starting to realise the potential of this after using this in a project, and i want to work more on it.

@GarboMuffin GarboMuffin added the pr: new extension Pull requests that add a new extension label Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: new extension Pull requests that add a new extension
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants