Seasonal: Autumn redesign (#237), needs human approval #241
No Reviewers
Labels
Clear labels
a11y
area: appearance
area: auth-session
area: build-ci
area: calls
area: desktop
area: media
area: messaging
area: mobile
area: moderation
area: navigation
area: notifications
area: settings
area: threads
bug
dependencies
docs
duplicate
enhancement
help wanted
invalid
needs-human-review
performance
planning
priority: critical
priority: high
priority: low
priority: medium
qa
question
research
security
tech-debt
ux
wontfix
Accessibility: keyboard, screen reader, contrast, motion
Client area: appearance
Client area: auth-session
Client area: build-ci
Client area: calls
Client area: desktop
Client area: media
Client area: messaging
Client area: mobile
Client area: moderation
Client area: navigation
Client area: notifications
Client area: settings
Client area: threads
Something is not working
Third-party package versions and advisories
README / LOTUS_* docs wrong or missing
This issue or pull request already exists
New feature
Need some help
Something is wrong
Re-render storms, leaks, heavy work on hot paths
Data loss, security hole, or crash on a main path
Broken feature or serious usability problem
Minor issue or polish
Wrong behaviour in an edge case or notable degradation
Manual QA: shipped, needs a human in a real environment
More information is needed
XSS, unsafe URLs, data leaks, auth/session
Code health, dead code, fragile patterns
Usability or visual inconsistency
This won't be fixed
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: LotusGuild/cinny#241
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
For #237. Do not merge without human review. Per the issue, only a human approves this and closes #237.
What changed
contain: layout paint style. Paint containment clips children to the box, so each leaf stopped being drawn as soon as it fell out of its first ~40 px. That's why Autumn looked empty. The page still reported 10–16 leaves on screen, because a position check doesn't see the clipping; only the rendered image showed it. It's nowcontain: layout style.Rendered and looked at
I captured frames at 10, 15 and 20 s after load, in light, dark, phone (Pixel 7) and reduced motion, in a room with real messages. I went through three design passes, each judged on the renders, not the code.
Light, 15 s in (left: before, right: after)

Dark, 15 s in

Reduced motion and phone at 20 s (before / after / before / after)

Close-up of the new leaves (light theme, actual size)

Open questions for the reviewer
Unit tests pass (1,245). Merging this PR is the approval step.
🤖 Generated with Claude Code
https://claude.ai/code/session_01PPmy3tPq869XDW4njjVaKA
Pass 2 is ready for human review (PR #241, now at
3e132069)Jared said the orange tint felt off and asked me to keep redesigning until I was satisfied. These are the changes since pass 1, each judged on renders:
previewprop on the seasonal overlays lets Autumn draw a composition made for the tile.Light: original vs now

Dark: original vs now

Reduced motion (desktop), phone, and phone with reduced motion

A gust passing, full size (t ≈ 10 s)

Settings thumbnail

Still open for the reviewer
Only a human approves this, merges PR #241 and closes #237.