fix metric table width for bigger screen - #31653
Conversation
Code Review ✅ ApprovedUpdates the metric table width to prevent empty gaps on wide screens. No issues found. OptionsDisplay: compact → Showing less information. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Powered by Gitar — free for open source |
|
✅ Playwright Results — workflow succeededValidated commit ✅ 551 passed · ❌ 0 failed · 🟡 0 flaky · ⏭️ 0 skipped · 🧰 0 lifecycle flaky PerformanceBlocking targets: ✅ met · Optimization targets: 🟡 in progress Shard-job maxima below are not the full workflow wall time; the linked run includes build, fixture, planning, and reporting. 🕒 Full workflow signal wall (to summary) 1h 25m 51s ⏱️ Max setup 3m 13s · max shard execution 16m 33s · max shard-job elapsed before upload 22m 22s · reporting 3s 🌐 211.58 requests/attempt · 2.83 app boots/UI scenario · 13.80% common-shard skew Optimization targets still in progress:
How to debug locally# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip # view trace |



Describe your changes:
The metrics list table rendered at the summed width of its fixed-width columns, leaving an empty gap on the right side of the card on wide screens.
Type of change:
High-level design:
N/A — small change.
Tests:
Use cases covered
Unit tests
Backend integration tests
Ingestion integration tests
Playwright (UI) tests
Manual testing performed
UI screen recording / screenshots:
Not applicable.
Checklist:
Fixes <issue-number>: <short explanation>Fixes #<issue-number>above.Greptile Summary
The PR adds an explicit full-width declaration to the metrics-list table.
Confidence Score: 4/5
The PR does not appear safe to merge because the previously reported ineffective width fix remains outstanding.
The metrics table already receives
tw:w-full, so the added declaration supplies the same computed width and leaves the reported wide-screen layout defect unresolved.Files Needing Attention: openmetadata-ui/src/main/resources/ui/src/pages/MetricsPage/MetricListPage/metric-list-page.less
Important Files Changed
width: 100%declaration to the metrics-list table rule.Reviews (2): Last reviewed commit: "Merge branch 'main' into metric-table" | Re-trigger Greptile