Skip to content

Fix: notif animation#2827

Merged
ItzNotABug merged 1 commit intomainfrom
fix-notifications
Feb 4, 2026
Merged

Fix: notif animation#2827
ItzNotABug merged 1 commit intomainfrom
fix-notifications

Conversation

@ItzNotABug
Copy link
Member

@ItzNotABug ItzNotABug commented Feb 4, 2026

What does this PR do?

Fix notification animation.

Test Plan

Manual.

Related PRs and Issues

N/A/

Have you read the Contributing Guidelines on issues?

Yes.

Summary by CodeRabbit

  • Style
    • Simplified notification animations by removing flip transitions, streamlining the animation behavior for a cleaner user experience.

@ItzNotABug ItzNotABug self-assigned this Feb 4, 2026
@appwrite
Copy link

appwrite bot commented Feb 4, 2026

Console (appwrite/console)

Project ID: 688b7bf400350cbd60e9

Sites (1)
Site Status Logs Preview QR
 console-stage
688b7cf6003b1842c9dc
Ready Ready View Logs Preview URL QR Code

Tip

Environment variable changes require redeployment to take effect

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 4, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

The pull request removes the flip animation functionality from the notifications component. Specifically, the flip animation import from Svelte is removed, and the corresponding animate:flip attribute is deleted from the notification element. The component retains its fly transition for notifications and maintains all existing rendering and notification management logic.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-notifications

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Reviews will stop working after February 8, 2026 if the new IP is not added to your allowlist.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ItzNotABug ItzNotABug merged commit 9b0ea7f into main Feb 4, 2026
2 of 4 checks passed
@ItzNotABug ItzNotABug deleted the fix-notifications branch February 4, 2026 06:18
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.

2 participants