Skip to content

Feature/more visibility close icon in multipart form data on collumn value#7341

Open
Bobonimo111 wants to merge 2 commits intousebruno:mainfrom
Bobonimo111:feature/more-visibility-icons-and-texts-MultipartFormData
Open

Feature/more visibility close icon in multipart form data on collumn value#7341
Bobonimo111 wants to merge 2 commits intousebruno:mainfrom
Bobonimo111:feature/more-visibility-icons-and-texts-MultipartFormData

Conversation

@Bobonimo111
Copy link

@Bobonimo111 Bobonimo111 commented Mar 2, 2026

This is a old position :
Alteração na posição de um botão antes

Problem :
Alteração na posição de um botão problema

I am changed to a new position, the button will never disappear :
Alteração na posição de um botão

issues : #7340

Summary by CodeRabbit

  • Style
    • Reordered the Remove button position in file parameter controls within multipart forms for improved accessibility and visibility.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 2, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 910581a and 5b9bb19.

📒 Files selected for processing (1)
  • packages/bruno-app/src/components/RequestPane/MultipartFormParams/index.js

Walkthrough

The Remove file button's position was reordered in the multipart form file parameters component, moving from after the file icon and name to before them. No functional changes or styling modifications were applied.

Changes

Cohort / File(s) Summary
UI Element Reordering
packages/bruno-app/src/components/RequestPane/MultipartFormParams/index.js
Reordered the Remove button to appear before the file icon and name in the file-type parameter render output.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

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

Poem

🔘 A button took a little stroll,
From back to front, it plays its role,
No logic changed, just moved around,
UI polish, clean and sound. ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title mentions repositioning a close icon in multipart form data, which aligns with the PR's objective of improving button visibility and accessibility.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant