RockteMQ-AI commented on issue #1970:
URL: 
https://github.com/apache/rocketmq-dashboard/issues/1970#issuecomment-5262507923

   **Issue Evaluation**
   
   Category: `bug` | Status: **Confirmed**
   
   The reported issue has been verified against the settings implementation.
   
   **Root Cause:** The General Settings page exposes UI controls (Theme mode, 
Compact mode, Desktop notifications, Notification sound) that are persisted by 
the backend API but not consumed by any runtime component. The active theme is 
controlled by ThemeProvider and browser storage, compact mode is not applied to 
Ant Design config, and notification features have no implementation.
   
   **Impact:** Users can change settings and receive success feedback, but none 
of the selected behaviors are applied. This creates a misleading user 
experience and suggests broken functionality.
   
   **Severity:** Medium — UX issue with no data loss, but erodes user trust
   
   **Proposed Fix Assessment:** The proposed fix is appropriate:
   - Remove non-functional controls from the UI while preserving backend 
compatibility
   - Keep existing values as hidden fields in save payloads
   - Maintain the working header theme toggle
   - Add regression tests
   
   An automated fix proposal can be generated. Reply `/approve` to proceed with 
PR generation.
   
   ---
   *Automated evaluation by github-manager-bot*
   


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