-
Notifications
You must be signed in to change notification settings - Fork 1.1k
fix: improve AI Bridge request logs UI/UX #21252
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.
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
base: main
Are you sure you want to change the base?
Conversation
site/src/pages/AIBridgePage/RequestLogsPage/RequestLogsRow/RequestLogsRow.tsx
Show resolved
Hide resolved
site/src/pages/AIBridgePage/RequestLogsPage/RequestLogsRow/RequestLogsRow.tsx
Outdated
Show resolved
Hide resolved
site/src/pages/AIBridgePage/RequestLogsPage/RequestLogsRow/RequestLogsRow.tsx
Outdated
Show resolved
Hide resolved
site/src/pages/AIBridgePage/RequestLogsPage/RequestLogsRow/RequestLogsRow.tsx
Outdated
Show resolved
Hide resolved
site/src/pages/AIBridgePage/RequestLogsPage/RequestLogsRow/RequestLogsRow.tsx
Outdated
Show resolved
Hide resolved
…uestLogsRow.tsx Co-authored-by: ケイラ <mckayla@hey.com>
📚 Documentation Check✨ Suggested Enhancement (Optional)The UI improvements in this PR (provider icons, token badges, improved layout) could benefit the user-facing documentation:
This would help users understand they can monitor AI Bridge usage directly in Coder's UI, not just through Grafana. Note: This is purely a visual enhancement to existing functionality - no new features or behavior changes, so documentation updates are optional. This comment was generated by an AI Agent through Coder Tasks |
|
All contributors have signed the CLA ✍️ ✅ |
0a75b0c to
4941f85
Compare
Closes #21052 and is related to #21218
This pull request implements a bunch of changes the team requested to improve the overall user experience when navigating to the Request Logs page. Since these are mostly UI changes, screenshots do a much better job of showing what's different than me trying to explain everything in words, so I've included them below.
Warning
More globally we've changed how we're rendering
<PaginationHeader />, therefore we've renamed it to<PaginationAmount />. This will effect multiple layouts however its a global improvement we're intentionally looking to make.Preview
Changes
MMM(i.eDec).namefield falling back on.username.text-xstotext-smto increase overall legibility globally.<Badge />and gives longer names a chance to render.<Badge />alongside their icons.<Badge />-group alongside truncation (when necessary).5is rendered with the proper name being visible on hover (for use if truncated).Showing X of Xcounts to the bottom of the tables (effects other components outside of AI Bridge).Providerfilter.<dt>element with the badgeExperimentalHandler