Skip to content

Conversation

@michelroegl-brunner
Copy link
Member

Summary

Add a loading modal with spinning circle that displays during start/stop/destroy container operations to provide clear visual feedback to users.

Changes

  • New LoadingModal component with full-screen overlay and spinning animation
  • Enhanced user experience during container operations with action-specific messages
  • Consistent modal styling matching existing ConfirmationModal and ErrorModal patterns

User Flow

  1. User clicks start/stop/destroy → confirmation modal shows
  2. User confirms → loading modal shows with spinning circle + action description
  3. Operation completes → loading modal closes, success/error modal shows

Technical Details

  • Uses icon from lucide-react for spinning animation
  • Integrates with existing mutation handlers in InstalledScriptsTab
  • Shows descriptive messages like 'Starting container 101...', 'Destroying container 102...'
  • Automatically closes on operation completion (success or error)

Fixes the issue where users had no visual feedback during background operations.

- Add LoadingModal component with spinning circle animation
- Show loading modal during start/stop/destroy container operations
- Display current action being performed (e.g., 'Starting container 101...')
- Close loading modal when operation completes (success or error)
- Maintains consistent modal styling with existing components

Fixes user experience by providing clear visual feedback during
background operations instead of silent processing.
@michelroegl-brunner michelroegl-brunner requested a review from a team as a code owner October 16, 2025 13:50
@michelroegl-brunner michelroegl-brunner changed the title feat: add loading modal for container operations fix: add loading modal for container operations Oct 16, 2025
@michelroegl-brunner michelroegl-brunner merged commit d95a854 into main Oct 16, 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.

2 participants