Skip to content

fix(banner): half padding on single-banner re-stream, none on multi#47

Merged
Velascat merged 1 commit intomainfrom
fix/banner-single-padding
May 8, 2026
Merged

fix(banner): half padding on single-banner re-stream, none on multi#47
Velascat merged 1 commit intomainfrom
fix/banner-single-padding

Conversation

@Velascat
Copy link
Copy Markdown
Owner

@Velascat Velascat commented May 8, 2026

Summary

  • Single-banner (healthy, loops itself): lp = (w-1)//4, g = max(6, (w-1)//6) — half the original values, gives a visible breath between repeats
  • Multi-banner: lp = "", g = " " — crossfade coloring is the boundary signal between conditions

Test plan

  • With one condition active (healthy): verify message scrolls with a comfortable gap before looping
  • With multiple conditions active: verify messages stream continuously with minimal gap, crossfade colors the boundary

Single-banner loops itself on the tape so a breath between repeats
reads better: lp=(w-1)//4, g=max(6,(w-1)//6) — half the originals.
Multi-banner keeps lp="" and g="    "; crossfade coloring is the
boundary signal between distinct conditions.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@Velascat Velascat merged commit 1ec6b14 into main May 8, 2026
2 checks passed
@Velascat Velascat deleted the fix/banner-single-padding branch May 8, 2026 20:09
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