feat(web): Enhance dashboard visualization and table-chart formatting#2731
Merged
chenliang15405 merged 3 commits intoMay 28, 2025
Merged
Conversation
… scenario\n\n- Merge type-value structure into single row\n- Unify number formatting (keep two decimals for scores/percentages)\n- Remove hardcoded mappings, keep code clean and robust\n- Improve cell and column formatting for better UX
… Remove axis titles for bar chart for better generalization\n- Unify number formatting (keep two decimals for scores/percentages)\n- Merge type-value structure into single row in table-chart\n- Remove hardcoded mappings, keep code clean and robust\n- Improve cell and column formatting for better UX\n- Update and clean up static web files
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR replaces the built frontend assets under static/web—including index.html and numerous hashed webpack chunk files—likely reflecting a new production build. It does not touch any source files or actual table-chart/bar-chart components.
Reviewed Changes
Copilot reviewed 101 out of 101 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| packages/dbgpt-app/src/dbgpt_app/static/web/chat/index.html | Updated script and build-manifest hashes |
| packages/dbgpt-app/src/dbgpt_app/static/web/_next/static/chunks/**/*.js (many files) | Replaced autogenerated webpack chunk outputs |
Comments suppressed due to low confidence (1)
packages/dbgpt-app/src/dbgpt_app/static/web/chat/index.html:1
- This PR includes a large number of autogenerated build artifacts. Consider removing these from version control (e.g., via .gitignore) and committing only source files to keep the repo clean and maintainable.
<!DOCTYPE html><html lang="en"><head>...
…Chart components - In the BarChart component, ensure numeric values for proper y-axis ordering and sort values in descending order for better visualization. - In the LineChart component, ensure numeric values for proper y-axis ordering. - Updated chart styles and theme settings to improve user experience.
Contributor
Author
|
Ding @fangyinc |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR enhances the
table-chart&bar-chartcomponent for dashboard scenarios:Resolves #2730
How Has This Been Tested?
Snapshots:
Checklist: