Skip to content

Conversation

michelroegl-brunner
Copy link
Member

Summary

This PR optimizes the UI button layout in the installed scripts table and fixes a dependency array loop issue.

Changes Made

UI Improvements

  • Added Open UI button next to IP:Port in the installed scripts table
  • Moved Re-detect button to Actions dropdown for better space usage and cleaner layout
  • Hide buttons for stopped containers to prevent invalid actions
  • Improved font consistency by removing monospace from IP:Port text
  • Enhanced auto-detect success message with LXC ID and hostname for better context

Bug Fixes

  • Fixed dependency array loop in fetchContainerStatuses useCallback that was causing infinite re-renders
  • Optimized button rendering to only show when container status is confirmed as running

UX Improvements

  • Better screen real estate usage with cleaner, more scannable layout
  • Logical button grouping with primary actions visible and utility functions in dropdown
  • Consistent styling across all action buttons

Testing

  • Buttons only appear for running containers
  • Open UI button works correctly
  • Re-detect button accessible via Actions dropdown
  • No more dependency loop causing performance issues
  • Enhanced success messages display properly

Screenshots

The interface now has a much cleaner layout with better space utilization and improved user experience.

- Add Open UI button next to IP:Port in installed scripts table
- Move Re-detect button to Actions dropdown for better space usage
- Fix dependency array loop in fetchContainerStatuses useCallback
- Hide buttons for stopped containers to prevent invalid actions
- Enhance auto-detect success message with LXC ID and hostname
- Improve font consistency by removing monospace from IP:Port text
- Optimize screen real estate with cleaner, more scannable layout
@michelroegl-brunner michelroegl-brunner requested a review from a team as a code owner October 14, 2025 14:21
@michelroegl-brunner michelroegl-brunner merged commit 5b11a6b into main Oct 14, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant