Skip to content

refactor: reduce format bar animation duration#84

Open
n00ki wants to merge 1 commit intoerictli:mainfrom
n00ki:refactor/faster-toolbar-animation
Open

refactor: reduce format bar animation duration#84
n00ki wants to merge 1 commit intoerictli:mainfrom
n00ki:refactor/faster-toolbar-animation

Conversation

@n00ki
Copy link

@n00ki n00ki commented Mar 5, 2026

  • Speed up the format bar (editing toolbar) show/hide animation when toggling markdown source mode (Cmd+Shift+M) or focus mode (Cmd+Shift+Enter)
  • Reduced from 1000ms transition + 500ms delay (1.5s total) to 150ms with no delay, so the toolbar feels snappy rather than sluggish
  • The slow cinematic fade on the header toolbar elements (sidebar toggle, date,
    save status) in focus mode is intentionally preserved

Summary by CodeRabbit

  • Style
    • Enhanced Format Bar animations: Reduced animation duration and eliminated initial delay for faster, more responsive interactions.

@coderabbitai
Copy link

coderabbitai bot commented Mar 5, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: fe0ef5ad-7fd7-4742-80b1-5d398564643e

📥 Commits

Reviewing files that changed from the base of the PR and between 3331c1e and 56f5e9c.

📒 Files selected for processing (1)
  • src/components/editor/Editor.tsx

📝 Walkthrough

Walkthrough

CSS transition timing adjustment in the Editor component's Format Bar. The animation transition class is updated to reduce duration from 1000ms to 150ms and remove the 500ms delay when the hasTransitioned state is active.

Changes

Cohort / File(s) Summary
Format Bar Animation Timing
src/components/editor/Editor.tsx
Updated transition class from "transition-all duration-1000 delay-500" to "transition-all duration-150", reducing animation duration and eliminating initial delay when hasTransitioned is truthy.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 Whoosh! goes the Format Bar,
No more delays, no more waiting,
A hundred fifty milliseconds—swift and far,
Animation springs to life, elating!
This rabbit's code now hops with speed,
Transitions quick to meet your need. ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarizes the main change: reducing the format bar animation duration from 1.5 seconds to 150ms.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

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