Skip to content

feat: Add logo_animation toggle#23086

Open
pjotrvdh wants to merge 3 commits intoanomalyco:devfrom
pjotrvdh:dev
Open

feat: Add logo_animation toggle#23086
pjotrvdh wants to merge 3 commits intoanomalyco:devfrom
pjotrvdh:dev

Conversation

@pjotrvdh
Copy link
Copy Markdown

@pjotrvdh pjotrvdh commented Apr 17, 2026

Issue for this PR

Closes #22528

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

This PR adds a new TUI config option: logo_animation in tui.json.

  • When logo_animation: false, logo animations are disabled globally (home logo + Go upsell logo), including interactive burst and sound effects.
  • When omitted or set to true, current behavior stays the same.
  • The setting is wired in the same style as other TUI toggles (like mouse): config is resolved at usage sites, not inside the logo component.
  • Legacy migration from opencode.json now also supports tui.logo_animation.
  • Docs are updated with an example and option description in the TUI config section.

Why this works:

  • The toggle is part of the validated TUI schema, so config parsing is consistent with existing options.
  • Animation control is passed explicitly to logo components, which keeps behavior predictable and avoids hidden config coupling.
  • Migration support ensures existing users can carry over the setting from legacy config.
  • Config tests cover loading from tui.json and legacy migration behavior.

If you paste a large clearly AI generated description here your PR may be IGNORED or CLOSED!

How did you verify your code works?

  • Added config tests for:
    • loading logo_animation from tui.json
    • migrating tui.logo_animation from legacy opencode.json
  • Manually verified behavior:
    • logo_animation: false -> static logos, no burst/sound
    • logo_animation: true (or unset) -> existing animation behavior

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@github-actions github-actions bot added needs:compliance This means the issue will auto-close after 2 hours. needs:title labels Apr 17, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Hey! Your PR title Add logo_animation toggle doesn't follow conventional commit format.

Please update it to start with one of:

  • feat: or feat(scope): new feature
  • fix: or fix(scope): bug fix
  • docs: or docs(scope): documentation changes
  • chore: or chore(scope): maintenance tasks
  • refactor: or refactor(scope): code refactoring
  • test: or test(scope): adding or updating tests

Where scope is the package name (e.g., app, desktop, opencode).

See CONTRIBUTING.md for details.

@pjotrvdh pjotrvdh changed the title Add logo_animation toggle feat: Add logo_animation toggle Apr 17, 2026
@github-actions github-actions bot removed needs:title needs:compliance This means the issue will auto-close after 2 hours. labels Apr 17, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Thanks for updating your PR! It now meets our contributing guidelines. 👍

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.

How to turn off sound effects and animations in 1.4.4

1 participant