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

Overall code quality improved, renamings applied and text animation a… #66

Merged
merged 1 commit into from
Feb 21, 2023

Conversation

hasanozen
Copy link

…dded for built-in text

@hasanozen
Copy link
Author

hasanozen commented Feb 4, 2023

  • Class names changed
  • Folder structures refactored
  • Duplicate codes removed
  • Text animation support added for built-in text

PS. These changes made for Runtime

@hasanozen hasanozen closed this Feb 4, 2023
@hasanozen hasanozen reopened this Feb 4, 2023
@brunomikoski brunomikoski merged commit 5cec29d into brunomikoski:develop Feb 21, 2023
@irakli
Copy link
Contributor

irakli commented Feb 25, 2023

I know this was already merged, but isn't Sequencer way too generic of a name? Maybe something like AnimationSequencer would be better.

@brunomikoski
Copy link
Owner

I know this was already merged, but isn't Sequencer way too generic of a name? Maybe something like AnimationSequencer would be better.

It's a good point @irakli , I'm still doing some more tweaks and changes to the new release. I'm trying to remake it using UIElements so the performance is better.
I will rename it for AnimationSequencer
But this gonna be a breaking change, so I will approach with care

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.

4 participants