lizhimins opened a new pull request, #4212:
URL: https://github.com/apache/rocketmq-dashboard/pull/4212

   Closes #4209
   
   ## Changes
   
   **Metric panels.** The chart now comes first and the query metadata is one 
secondary line
   (`2/2 series ยท 242 samples`) with the full breakdown behind an info popover. 
Previously five
   `Statistic` blocks and a six-item `Descriptions` sat above every chart. Page 
total drops from
   **53** `Statistic` blocks to **8**; the nine charts are unchanged.
   
   **Cluster health table.** Eight identity and topology columns instead of 
fifteen, with the seven
   traffic columns moved into the expandable row โ€” they duplicated the traffic 
insight card directly
   above. Table width 1620px โ†’ fits the container, no horizontal scrollbar.
   
   **Traffic insight card.** All four summary cards render the same structure 
including a caption
   slot, so they are the same height whether or not they wrap onto two flex 
lines (was 112/112/90/90,
   now uniform). The findings alert is a single `message` line instead of a 
`description` block with
   one tag per finding (62px tall).
   
   **Other pages.**
   
   - System alert filter row gets `wrap` and `align="center"`, and the `Input`s 
get `flex: none` so
     they stop being the elements that absorb the shrink. Measured overflow 
188px โ†’ 0; the four
     inputs render at their declared 150/190/190/190px instead of 16px.
   - Notification settings test buttons and submit button share one row instead 
of two flush
     `Form.Item`s.
   - Dashboard refresh button and instance-filter placeholder go through i18n.
   
   ## Verification
   
   - Deployed and measured: 8 statistic blocks, 21 chart polylines still 
rendering, 0 error alerts,
     no hard-coded English buttons, cluster table 9 columns with no horizontal 
scrollbar.
   - `tsc --noEmit` clean, `eslint` no errors, `vitest run` 980 passing.
   - `DashboardPage.test.tsx` traffic assertions now expand the row first; the 
aria-label lookups
     follow the i18n change. `DashboardTrafficInsights.test.tsx` matches the 
findings text as a
     substring since it is now joined into one line.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to