Skip to content

fix: remove mount animation from projects announcement banner - #517

Closed
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1784742848-remove-banner-animation
Closed

fix: remove mount animation from projects announcement banner#517
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/1784742848-remove-banner-animation

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

The "Teams are being renamed to projects" banner animated in on mount. This removes that animation so the banner appears instantly.

  • Dropped animate-announcement-in motion-reduce:animate-none from the banner container in banner.tsx.
  • Removed the now-unused @keyframes announcement-in and --animate-announcement-in from theme.css.

Link to Devin session: https://app.devin.ai/sessions/9848acc809e247a5babe9185626b98ee


View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

Co-Authored-By: ben@e2b.dev <ben@e2b.dev>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@vercel

vercel Bot commented Jul 22, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
web-foxtrot Ready Ready Preview, Comment Jul 22, 2026 5:56pm

Request Review

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM — trivial removal of a mount animation and its unused keyframe/variable definitions.

Extended reasoning...

Overview

This PR removes the animate-announcement-in motion-reduce:animate-none class from the projects announcement banner container (banner.tsx) and deletes the now-unused @keyframes announcement-in and --animate-announcement-in theme variable from theme.css. No logic, state, or props are touched — the component's mount/dismiss/localStorage behavior is unchanged, only a purely presentational CSS animation is removed.

Security risks

None. This touches only static CSS class names and a Tailwind theme keyframe definition. No user input, auth, or data handling is involved.

Level of scrutiny

Low. This is a cosmetic, single-purpose UI tweak confined to one non-critical banner component and a shared theme file. I confirmed via grep that announcement-in/--animate-announcement-in are not referenced anywhere else in the codebase, so removing them does not orphan any other usage.

Other factors

The diff is minimal and mechanical (class removal + dead CSS deletion), matches the PR description exactly, and there are no outstanding review comments to address.

@vercel
vercel Bot temporarily deployed to Preview – web-foxtrot July 22, 2026 17:56 Inactive
@ben-fornefeld

Copy link
Copy Markdown
Member

closed because it's better to have the animation in the end.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant