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

   ### Which Issue(s) This PR Fixes
   
   Fixes #3282
   
   ### Brief Description
   
   Message queries already return bounded `properties` and a 
`propertiesTruncated` flag, but the Message Explorer hid them. This adds one 
shared properties section to the message detail modal and queue-browser 
results. The section stays compact for ordinary messages, scrolls for long 
property lists, wraps long keys and values, and explains when the server 
shortened the data. The notice uses the existing Chinese/English translation 
system.
   
   This revisits the still-open issue after #3290 closed. The PR targets 
`rocketmq-studio` and reuses one renderer across both views.
   
   ### How Did You Test This Change?
   
   - `npm test -- src/pages/instance/__tests__/MessagePage.test.tsx 
src/components/__tests__/QueueBrowser.test.tsx`: 26 tests passed, including 
both property display paths and the truncation notice.
   - `npx tsc -b` and `npx vite build`: passed.
   - ESLint on the changed files: 0 errors; 2 existing 
`react-refresh/only-export-components` warnings in `QueueBrowser.tsx`.
   - Opened the message detail modal in Chrome at 375, 768, 1280 and 1440 px. 
The properties section had no horizontal overflow at any width.
   - `git diff --check`: clean.
   
   The base branch's full `npm run build` license gate is tracked separately in 
#5018 / #5019; this change builds with Vite and TypeScript.
   


-- 
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