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

Matrix scrolling text animation #283

Merged
merged 6 commits into from
Apr 24, 2022
Merged

Conversation

StaleHyena
Copy link
Contributor

Thought it would look cool for a rice :P
Mods the already existing fire animation to allocate a state object,
as to keep the animation and term_buf data separate.

Doesn't leak memory with any combination of config values i
tested. I tried to follow the existing allocation error handling style,
not sure if it is sound!

Would appreciate feedback if any.

@mevangelisti
Copy link

This is cool as hell, nice work.

yobleck added a commit to yobleck/ly that referenced this pull request May 11, 2021
scrolls vertical bars of the 8 ANSI colors from left to right
TODO: config vars for: horizontal bars, reverse scrolling, speed
Note: currently incompatible with StaleHyena's PR: fairyglade#283. will fix mine to match theirs
@mjevange
Copy link

mjevange commented Oct 7, 2021

any change we can pull these? o:)

idlecore pushed a commit to idlecore/ly that referenced this pull request Oct 30, 2021
scrolls vertical bars of the 8 ANSI colors from left to right
TODO: config vars for: horizontal bars, reverse scrolling, speed
Note: currently incompatible with StaleHyena's PR: fairyglade#283. will fix mine to match theirs
@nullgemm nullgemm merged commit 6245639 into fairyglade:master Apr 24, 2022
@nullgemm nullgemm mentioned this pull request Apr 24, 2022
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