Skip to content

[UI]: Optimize table ui and deduplicate 403 login error#4093

Open
majin1102 wants to merge 1 commit intoapache:masterfrom
majin1102:fix-table-explorer
Open

[UI]: Optimize table ui and deduplicate 403 login error#4093
majin1102 wants to merge 1 commit intoapache:masterfrom
majin1102:fix-table-explorer

Conversation

@majin1102
Copy link
Contributor

@majin1102 majin1102 commented Feb 15, 2026

Optimize Table UI and Deduplicate 403 Login Error

Summary

This PR addresses two main issues: deduplicating repetitive login error messages and optimizing the table explorer UI for better user experience.

Changes

Bug Fix

  • Deduplicate 403 Login Error: When multiple API requests return 403 simultaneously, the application was displaying multiple "need login" error messages. Added a loginTipShown flag to ensure the error message is displayed only once per login session. The flag is reset via resetLoginTip() when navigating to the login page:
image

UI Optimization

  • Default Sidebar Width: Changed the default sidebar width from 512px to 320px for a more compact initial view.
  • Divider Style: Reduced divider width from 8px to 5px, with the dividing line moved to the right side for better visual alignment.
  • Router Guard: Improved the authentication flow by adding clear route guards that redirect unauthenticated users to the login page.

Test Plan

  • Verify login error message appears only once when multiple 403 errors occur
  • Verify sidebar width can be resized and persists after page refresh
  • Verify the divider can be dragged to resize sidebar
  • Verify padding is symmetrical on both sides of sidebar content
  • Verify unauthenticated users are redirected to login page

@github-actions github-actions bot added the module:ams-dashboard Ams dashboard module label Feb 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:ams-dashboard Ams dashboard module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant