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

[Update] Flash and transition painter #1243

Open
3 tasks done
VegeTato opened this issue Apr 3, 2024 · 8 comments
Open
3 tasks done

[Update] Flash and transition painter #1243

VegeTato opened this issue Apr 3, 2024 · 8 comments
Assignees
Labels
🔄 Extension update An update for an existing extension ✨ New extension A new extension

Comments

@VegeTato
Copy link
Contributor

VegeTato commented Apr 3, 2024

Description

  • Added the extension author name.
  • Added an option for the extension action (Yes/No) to remove the paint after the transition has finished.
    (by default is set to Yes)
    d2

How to use the extension

No response

Checklist

  • I've followed all of the best practices.
  • I confirm that this extension can be integrated to this GitHub repository, distributed and MIT licensed.
  • I am aware that the extension may be updated by anyone, and do not need my explicit consent to do so.

What tier of review do you aim for your extension?

Reviewed

Example file

Example.zip

Extension file

FlashTransitionPainter.zip

@VegeTato VegeTato added the ✨ New extension A new extension label Apr 3, 2024
Copy link
Contributor

github-actions bot commented Apr 3, 2024

👋 Thanks for your submission! We are sorry, but it seems we already have a reviewed extension with that name. Extensions cannot be in both the community and reviewed extensions list at once. If you are trying to update that extension, please ask a member of the extension team for help. If the name clash is a coincidence, please update your original submission post with a new zip file containing your extension with a file name that is not already taken 🙏

@VegeTato VegeTato added the 🔄 Extension update An update for an existing extension label Apr 3, 2024
Copy link
Contributor

github-actions bot commented Apr 3, 2024

👋 Thanks for your submission! We are sorry, but it seems we already have a reviewed extension with that name. Extensions cannot be in both the community and reviewed extensions list at once. If you are trying to update that extension, please ask a member of the extension team for help. If the name clash is a coincidence, please update your original submission post with a new zip file containing your extension with a file name that is not already taken 🙏

2 similar comments
Copy link
Contributor

github-actions bot commented Apr 4, 2024

👋 Thanks for your submission! We are sorry, but it seems we already have a reviewed extension with that name. Extensions cannot be in both the community and reviewed extensions list at once. If you are trying to update that extension, please ask a member of the extension team for help. If the name clash is a coincidence, please update your original submission post with a new zip file containing your extension with a file name that is not already taken 🙏

Copy link
Contributor

github-actions bot commented Apr 4, 2024

👋 Thanks for your submission! We are sorry, but it seems we already have a reviewed extension with that name. Extensions cannot be in both the community and reviewed extensions list at once. If you are trying to update that extension, please ask a member of the extension team for help. If the name clash is a coincidence, please update your original submission post with a new zip file containing your extension with a file name that is not already taken 🙏

@tristanbob tristanbob self-assigned this Apr 6, 2024
@tristanbob
Copy link
Contributor

I looked into this update. I wasted some time thinking this was like tweens and deleting the object, but I think you want the paint to stay in the final position after it is completed. We need to think of the best wording for this option.

@VegeTato
Copy link
Contributor Author

VegeTato commented Apr 9, 2024

I looked into this update. I wasted some time thinking this was like tweens and deleting the object, but I think you want the paint to stay in the final position after it is completed. We need to think of the best wording for this option.

Not sure what you mean at the tweens part, yea i just want the paint to stay in the final position when its completed 👍

The current one is not good ?
(Remove the paint when the transition has finished)
I mean it says what it supposes to do

@D8H
Copy link
Contributor

D8H commented Apr 21, 2024

This extension is overly complicated. I guess the 3 of us have wasted too much time on it so I rewrote it with half the events and fixed a few things:

I think that the black screen should always be kept so I didn't added a parameter like VegeTato suggested but forced it. Users can always clear the shape painter if they need to.

Please take a look to the PR and tell me what you think about the changes and the todo list (for a future PR that I probably won't do).

@VegeTato
Copy link
Contributor Author

This extension is overly complicated. I guess the 3 of us have wasted too much time on it so I rewrote it with half the events and fixed a few things:

* [[Transition] Fix the animation for zoomed layers #1258](https://github.com/GDevelopApp/GDevelop-extensions/pull/1258)

I think that the black screen should always be kept so I didn't added a parameter like VegeTato suggested but forced it. Users can always clear the shape painter if they need to.

Please take a look to the PR and tell me what you think about the changes and the todo list (for a future PR that I probably won't do).

Perfect !
i responded in the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔄 Extension update An update for an existing extension ✨ New extension A new extension
Projects
None yet
Development

No branches or pull requests

3 participants