Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Akram Znini, [email protected] #67

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

akramznini
Copy link

Pull request for the React Native Technical Challenge.

- Created a reusable button component for start, stop, reset, and lap actions.
- Defined explicit prop types to address TypeScript warnings.
- Added useEffect to update elapsed time every 100 milliseconds when running.
- Exposed start, stop, reset, and getFormattedTime functions using useImperativeHandle.
- Formatted time to display in "hh:mm:ss" format.
- Added functionality to start, stop, reset, and record laps in the stopwatch.
- Corrected a bug in the reset function within useImperativeHandle.
- Previously, the 'isRunning' state was not set to false, causing incorrect behavior on reset.
- Now, the reset function properly sets 'isRunning' to false, resolving the issue.
- Adjusted lap display to be scrollable
- Fine-tuned styling for a visually appealing user interface.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant