8000 Tools redesign by ManishMadan2882 · Pull Request #1812 · arc53/DocsGPT · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Tools redesign #1812

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 15 commits into from
May 23, 2025
Merged

Tools redesign #1812

merged 15 commits into from
May 23, 2025

Conversation

ManishMadan2882
Copy link
Collaborator
@ManishMadan2882 ManishMadan2882 commented May 22, 2025 •
edited
Loading

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

    • closes 🚀 Feature: Tools menu design adjustments, also name setting #1780
    • UI correction
      • add actions button is visible for the api tool type only
      • remove duplicate placeholder from the add action modal
      • allow deleting any user added action
      • place the add action at the bottom of the tools table
      • responsiveness changes for the Input component in the ToolConfg
      • fixes the issue in Input.tsx - floating placeholder grows on loosing focus even when the value is present

  • Why was this change needed? (You can also link to an open issue here)
    UX

  • Other information:
    customName field is stored in the db, the UI should reflect the custom name if present otherwise the displayName

@vercel Vercel
Copy link
vercel bot commented May 22, 2025

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

A member of the Team first needs to authorize it.

@ManishMadan2882 ManishMadan2882 requested a review from dartpain May 22, 2025 22:17
@vercel Vercel
Copy link
vercel bot commented May 23, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
oss-docsgpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 23, 2025 10:49pm

@dartpain dartpain merged commit c662725 into arc53:main May 23, 2025
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dartpain dartpain Awaiting requested review from dartpain

Assignees
No one assigned
Labels
application Application frontend
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

🚀 Feature: Tools menu design adjustments, also name setting
2 participants
@ManishMadan2882 @dartpain
0