You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Aug 2, 2025. It is now read-only.
The current implementation of the ThemeLoadingOverlay component uses extra state variables (shouldShow, isAnimatingOut) and a setTimeout to manage the fade-out animation and visibility of the overlay. This adds unnecessary complexity and can be simplified.
Suggested Improvement:
Remove the additional state and timer logic.
Use the isThemeLoaded prop directly to control the overlay's visibility and transitions.
Leverage Tailwind/CSS transitions (e.g., transition-opacity duration-300 ease-in-out) to handle the fade-out effect.
Conditionally render the overlay only when !isThemeLoaded.
Optionally, add pointer-events-none when the overlay is hidden to prevent interaction.
The current implementation of the
ThemeLoadingOverlaycomponent uses extra state variables (shouldShow,isAnimatingOut) and asetTimeoutto manage the fade-out animation and visibility of the overlay. This adds unnecessary complexity and can be simplified.Suggested Improvement:
isThemeLoadedprop directly to control the overlay's visibility and transitions.transition-opacity duration-300 ease-in-out) to handle the fade-out effect.!isThemeLoaded.pointer-events-nonewhen the overlay is hidden to prevent interaction.Example Implementation:
Action Items:
ThemeLoadingOverlaycomponent to remove unnecessary state and timer logic.This change will simplify the component and make the transition logic more maintainable.
I created this issue for @Its4Nik from Its4Nik/DockStat#104 (comment).
Tips and commands
Getting Help