Skip to content

fix: make the folder runner spinner consistent with the single-request spinner#7496

Open
fantpmas wants to merge 1 commit intousebruno:mainfrom
fantpmas:fix/make-folder-runner-spinner-consistent
Open

fix: make the folder runner spinner consistent with the single-request spinner#7496
fantpmas wants to merge 1 commit intousebruno:mainfrom
fantpmas:fix/make-folder-runner-spinner-consistent

Conversation

@fantpmas
Copy link
Copy Markdown
Contributor

@fantpmas fantpmas commented Mar 16, 2026

Description

This makes the folder runner spinner consistent with the single-request spinner.
The arrows were pointing the wrong way.

Before

wrongway.mov

After

rightway.mov

Contribution Checklist:

  • I've used AI significantly to create this pull request
  • The pull request only addresses one issue or adds one feature.
  • The pull request does not introduce any breaking changes
  • I have added screenshots or gifs to help explain the change if applicable.
  • I have read the contribution guidelines.
  • Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.

Publishing to New Package Managers

Please see here for more information.

Summary by CodeRabbit

  • Style
    • Updated the loading indicator animation in the runner results component with a new visual effect.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 16, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: e5da05e6-6ada-49b8-86fb-7a2873898f9d

📥 Commits

Reviewing files that changed from the base of the PR and between 7e0b8d9 and 9594555.

📒 Files selected for processing (2)
  • packages/bruno-app/src/components/RunnerResults/StyledWrapper.js
  • packages/bruno-app/src/components/RunnerResults/index.jsx

Walkthrough

The RunnerResults component's loading indicator receives a custom animation style. A new .loading-icon CSS class implements vertical flip and counter-clockwise rotation, replacing the previous utility class assignment.

Changes

Cohort / File(s) Summary
Loading Icon Animation
packages/bruno-app/src/components/RunnerResults/StyledWrapper.js, packages/bruno-app/src/components/RunnerResults/index.jsx
Added .loading-icon CSS class with scaleY(-1) flip and rotateCounterClockwise infinite rotation animation. Updated IconRefresh component to use new class instead of animate-spin utility.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

  • helloanoop
  • lohit-bruno
  • naman-bruno
  • bijin-bruno

Poem

⚙️ Spinning upside-down with flair,
Counter-clockwise through the air,
A loading dance, so smooth and fine,
The icon twirls along the line! ✨

🚥 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 accurately describes the main change: making the folder runner spinner visually consistent with the single-request spinner by fixing the arrow direction.

✏️ 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
📝 Coding Plan
  • Generate coding plan for human review comments

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.

Tip

CodeRabbit can generate a title for your PR based on the changes with custom instructions.

Set the reviews.auto_title_instructions setting to generate a title for your PR based on the changes in the PR with custom instructions.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant