msyavuz opened a new pull request, #35321: URL: https://github.com/apache/superset/pull/35321
## SUMMARY Fixed dual background issue in CRUD views by updating the ListView component theming. The component was using `theme.colorBgLayout` which created an unwanted layered background effect under tables. Changed to `theme.colorBgContainer` and added consistent padding for a cleaner appearance. ## BEFORE/AFTER **Before:** - CRUD views had two visible background layers creating visual clutter - Inconsistent spacing around table content **After:** - Single, clean background using the appropriate container theme token - Consistent padding using theme spacing tokens - Cleaner visual hierarchy in CRUD interfaces ## TESTING INSTRUCTIONS 1. Navigate to any CRUD view (Charts, Dashboards, Datasets, etc.) 2. Verify there's only one background color visible 3. Check that spacing around the table content is consistent 4. Test with different themes to ensure the fix works across all theme variants ## ADDITIONAL INFORMATION This change aligns with the ongoing Ant Design theming migration by: - Using proper Ant Design theme tokens (`colorBgContainer` vs `colorBgLayout`) - Applying consistent spacing using theme padding tokens - Following Ant Design best practices for container backgrounds No breaking changes - purely visual improvement. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
