Aias00 opened a new issue, #1970:
URL: https://github.com/apache/rocketmq-dashboard/issues/1970

   ### Description
   
   The General Settings page exposes Theme mode, Compact mode, Desktop 
notifications, and Notification sound controls. These values are persisted by 
the settings API, but no runtime component consumes them:
   
   - active theme is controlled independently by ThemeProvider and browser 
local storage
   - compact mode is not applied to the Ant Design configuration
   - desktop notifications do not use the browser Notification API
   - notification sound has no playback path
   
   Saving these controls reports success even though the running UI and alert 
behavior do not change.
   
   ### Expected behavior
   
   The settings page should only expose controls that have a working runtime 
effect. Existing persisted fields should remain compatible with the backend 
until their contract is redesigned.
   
   ### Actual behavior
   
   Users can change four controls and receive a success message, but none of 
the selected behavior is applied.
   
   ### Proposed fix
   
   Remove the unsupported visible controls from General Settings while 
retaining their loaded values as hidden compatibility fields in the save 
payload. Keep the existing working header theme toggle unchanged. Add a 
regression test that verifies the controls are not presented and existing 
values are preserved when saving.


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