majin1102 opened a new pull request, #4093: URL: https://github.com/apache/amoro/pull/4093
# 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. ### 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. - **Padding Symmetry**: Added consistent padding-top (12px) to both sidebar and main content areas. Adjusted search box and tree navigation padding (8px left, 7px right) to achieve symmetrical white space with the divider. - **Resizable TablesMenu**: Added drag-to-resize functionality to the TablesMenu component with a minimum width of 400px, allowing users to customize the panel size. - **Router Guard**: Improved the authentication flow by adding clear route guards that redirect unauthenticated users to the login page. ## Test Plan - [x] Verify login error message appears only once when multiple 403 errors occur - [x] Verify sidebar width can be resized and persists after page refresh - [x] Verify the divider can be dragged to resize sidebar - [x] Verify padding is symmetrical on both sides of sidebar content - [x] Verify unauthenticated users are redirected to login page -- 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]
