mi6 / mi6/ic-ui-kit

Investigate replacing setTimeout for rendering/displaying components with animation

Open
#476 1 comment 0 reactions 0 assignees View on GitHub
spike
Dominant language
TypeScript
Stars
53
Forks
63
Avg merge
2d 4h
Merged PRs (30d)
15

Description

## Summary
Investigate other methods to conditionally render/display components which are initially hidden and which require animation when they are rendered/displayed.

## 💬 Description
We currently add animation to components which are initially hidden but which require animation when they are rendered/displayed using the setTimeout method. While this does seem to be a common method in the wider industry it is worth investigating whether there are other, potentially cleaner, methods of achieving this same functionality, e.g. event listeners that trigger when the component has been set to display and add the animation class.

## 💰 Use value
Potentially improve the overall quality of our code, otherwise validate our current solution.

## Additional info

Contributor guide

Open the contributing guide

Research direction

Start by locating the components that use setTimeout for initially hidden content and inspect how their rendering, display state, and animation classes interact. Compare the current approach with event-listener or other alternatives, then document whether a cleaner replacement is justified or validate the existing solution.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
frontend
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.