Skip to content

fix possible desync in quote carousel#152

Merged
adibarra merged 2 commits intomasterfrom
fix/quote-carousel-garbled-text
Apr 3, 2026
Merged

fix possible desync in quote carousel#152
adibarra merged 2 commits intomasterfrom
fix/quote-carousel-garbled-text

Conversation

@adibarra
Copy link
Copy Markdown
Contributor

@adibarra adibarra commented Apr 3, 2026

No description provided.

adibarra added 2 commits April 3, 2026 14:13
Race condition: advance() and goTo() scheduled independent timeouts,
so concurrent fades could leave multiple quotes at intermediate opacity
in the same grid cell. Fix by tracking the fade timeout (clearing before
new fades) and rendering a single quote element instead of stacking all
quotes with CSS opacity.
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
inferencemax-app Ready Ready Preview, Comment Apr 3, 2026 7:18pm

Request Review

@adibarra adibarra marked this pull request as ready for review April 3, 2026 19:19
@adibarra adibarra merged commit 8221636 into master Apr 3, 2026
16 checks passed
@adibarra adibarra deleted the fix/quote-carousel-garbled-text branch April 3, 2026 19:24
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.

1 participant