Skip to content

fix: constrain dashboard column height - #13433

Merged
ssongliu merged 1 commit into
dev-v2from
fix/dashboard-column-height
Jul 31, 2026
Merged

fix: constrain dashboard column height#13433
ssongliu merged 1 commit into
dev-v2from
fix/dashboard-column-height

Conversation

@ssongliu

@ssongliu ssongliu commented Jul 31, 2026

Copy link
Copy Markdown
Member

Refs #13421

Copilot AI review requested due to automatic review settings July 31, 2026 10:17

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adjusts the home dashboard layout/monitor chart configuration to better constrain the right-side dashboard column height, and includes a small cleanup in a license restore dialog stylesheet.

Changes:

  • Add an explicit ECharts legend position for the home monitor line charts (IO/Network) to stabilize chart vertical layout.
  • Add contain: size; to the desktop .dashboard-right column to help constrain/decouple sizing behavior.
  • Remove an unused .restore-mode-group style block from the community restore dialog view.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
frontend/src/views/setting/license-required/community-restore/index.vue Removes unused scoped CSS (.restore-mode-group) that is not referenced in the template.
frontend/src/views/home/index.vue Sets a shared legend position for monitor charts and applies contain: size to the right dashboard column in desktop layout.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ssongliu
ssongliu merged commit 0d8835d into dev-v2 Jul 31, 2026
4 checks passed
@ssongliu
ssongliu deleted the fix/dashboard-column-height branch July 31, 2026 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants