Skip to content

Added cooldown to sign in button#120

Open
abrar-amin wants to merge 1 commit intomainfrom
Abrar-Add-Cooldown
Open

Added cooldown to sign in button#120
abrar-amin wants to merge 1 commit intomainfrom
Abrar-Add-Cooldown

Conversation

@abrar-amin
Copy link
Copy Markdown
Collaborator

Summary

Added a 15 second cooldown to the send sign-in email link button.

Test Plan

Tested in developer sandbox

Notes

This is purely a frontend change.

Simulator.Screen.Recording.-.iPhone.17.-.2026-04-06.at.14.55.05.mov

@abrar-amin abrar-amin requested review from clementroze and laylaliuuu and removed request for laylaliuuu April 6, 2026 18:58
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 6, 2026

Deploy Preview for redi-love ready!

Name Link
🔨 Latest commit bfb943b
🔍 Latest deploy log https://app.netlify.com/projects/redi-love/deploys/69d402668be05c0008dda1a1
😎 Deploy Preview https://deploy-preview-120--redi-love.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 6, 2026

Copy link
Copy Markdown
Collaborator

@clementroze clementroze left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Abrar! ⏲️

<View style={styles.buttonContainer}>
<Button
title="Send Sign-In Link"
title={cooldownSeconds > 0 ? `Resend in ${cooldownSeconds}s` : 'Send Sign-In Link'}
Copy link
Copy Markdown
Collaborator

@clementroze clementroze Apr 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is really minor, but to prevent flickering of the numbers + the label changing width, do u wanna try adding the font-variant-numeric: tabular-nums; CSS property?

See this SO post for more details

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.

2 participants