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 animations #49

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Add animations #49

wants to merge 9 commits into from

Conversation

Lorenzo0111
Copy link

@Lorenzo0111 Lorenzo0111 commented Dec 26, 2021

Closes #48

Example usage:

        Gui gui = new Gui(3,"Hello", EnumSet.noneOf(InteractionModifier.class));
        gui.setItem(0, ItemBuilder.from(Material.DIAMOND).asGuiItem());
        
        Animation animation = Animation.infinite(5,
                Frame.builder()
                        .addItem(0, ItemBuilder.from(Material.DIAMOND)
                                .name(Component.text(ChatColor.translateAlternateColorCodes('&', "&6&lHello World")))
                                .asGuiItem())
                        .build(),
                Frame.builder()
                        .addItem(0, ItemBuilder.from(Material.DIAMOND)
                                .name(Component.text(ChatColor.translateAlternateColorCodes('&', "&f&lHello World")))
                                .asGuiItem())
                        .build());
        
        gui.addAnimation(animation);
        gui.open(player);

@LichtHund
Copy link
Member

Sorry for late reply.
I like the idea, I'll keep this open for now though, I want to rework the updating system a bit more and this would be perfect for when that is ready.

@Lorenzo0111
Copy link
Author

Yeah, it is a bit raw, can't wait to see this awesome feature in the library.

@Lorenzo0111
Copy link
Author

any news?

@xMrAfonso
Copy link

xMrAfonso commented Oct 10, 2022

any news?

The update system seems to have not been updated yet, gotta wait sadly. But love the work you did here. I love the way you did it.

@harvanchik
Copy link

This is pretty awesome. Sad to see it still hasn't been merged.

@CantBeKarma
Copy link

@LichtHund Any updates on this?

@5RoD
Copy link

5RoD commented Dec 6, 2024

is this going to happen ?

@LichtHund
Copy link
Member

LichtHund commented Dec 8, 2024

I'll finally give an update on this. As of the current version v3 this will not be merged, I am happy if people use @Lorenzo0111's fork for this instead. v3 will still be around and I'll maintain it only for new releases and bugs, no more new features.

I am pretty close to finishing v4 (which you can read a bit more about in #113) and this would be a lot easier to implement, but even then I am not sure about it being part of the main library.
As a separate module definitely, which I am okay with either having it in the main repo with @Lorenzo0111 as the maintainer of it or if it was published as an add-on to it.
I can give a quick guide on how to implement this on v4 if needed.

@Lorenzo0111
Copy link
Author

Yeah sure, if you can guide me to do that I'll be happy to maintain it here

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.

Add animations
6 participants