Skip to content

adjusted edit button padding and response box padding & roundness#2021

Merged
ManishMadan2882 merged 1 commit intoarc53:mainfrom
M4cr0Chen:enhance-edit-button-and-response-ui
Oct 6, 2025
Merged

adjusted edit button padding and response box padding & roundness#2021
ManishMadan2882 merged 1 commit intoarc53:mainfrom
M4cr0Chen:enhance-edit-button-and-response-ui

Conversation

@M4cr0Chen
Copy link
Contributor

@M4cr0Chen M4cr0Chen commented Oct 5, 2025

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    This change introduces some slight UI enhancements on Edit Button and Response box.
  1. I- added a pt-1.5 and pl-1.5 to the Edit button to better center-align the svg.
  2. I also modified the conversation box's roundness from 28px to 18px; modified horizontal padding from px-7 to px-6; removed a hard-coded vertical padding and changed it from 18px to py-4.5.
  • Why was this change needed? (You can also link to an open issue here)
    Opened Issue: Enhancement on Edit Button UI #2010
    This change is supposed to make the Response box appears more elegantly, and make Edit button aligns better.
    This changes effectively ensures the consistency in the DocsGPT's UI/UX design and thus providing a better end-user experience.

@vercel
Copy link

vercel bot commented Oct 5, 2025

@M4cr0Chen is attempting to deploy a commit to the Arc53 Team on Vercel.

A member of the Team first needs to authorize it.

@ManishMadan2882 ManishMadan2882 self-requested a review October 6, 2025 07:26
Copy link
Collaborator

@ManishMadan2882 ManishMadan2882 left a comment

Choose a reason for hiding this comment

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

Perfect, Thanks!

@codecov
Copy link

codecov bot commented Oct 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 34.67%. Comparing base (c00b645) to head (b505d20).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2021   +/-   ##
=======================================
  Coverage   34.67%   34.67%           
=======================================
  Files         131      131           
  Lines        8703     8703           
=======================================
  Hits         3018     3018           
  Misses       5685     5685           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ManishMadan2882 ManishMadan2882 merged commit 4eb9471 into arc53:main Oct 6, 2025
7 of 9 checks passed
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.

2 participants