-
Notifications
You must be signed in to change notification settings - Fork 23
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
Create email invite confirmation screen #215
Changes from 2 commits
e516b4b
c2f2316
01a5fc2
0aa1443
b24fde7
b6a7937
6ba63b6
c91597d
9cfac85
fbe0717
6369285
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,32 @@ | ||
import React, { useEffect } from 'react'; | ||
import { useNavigate } from 'react-router-dom'; | ||
|
||
import './stylesheet.scss'; | ||
|
||
export default function EmailInviteConfirmation(): React.ReactElement { | ||
const navigate = useNavigate(); | ||
|
||
useEffect(() => { | ||
setTimeout(() => { | ||
navigate('/'); | ||
}, 30000); | ||
}); | ||
|
||
return ( | ||
<div className="EmailInviteConfirmation"> | ||
<h1>Congratulations on Adding a New Schedule to your View!</h1> | ||
<p>You are being redirected to our main site, please wait...</p> | ||
<p> | ||
If you have not been redirected in 30 seconds, please click the button | ||
below | ||
</p> | ||
<button type="button" className="continue-button"> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The button is missing the on-click redirect functionality. |
||
Continue | ||
</button> | ||
|
||
<a className="footer" href="https://bitsofgood.org/"> | ||
<img alt="Bits of Good Logo" src="bitsOfGood.png" /> | ||
</a> | ||
</div> | ||
); | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,53 @@ | ||
@import '../../variables'; | ||
|
||
body { | ||
background-color: $theme-dark-background; | ||
color: $theme-dark-foreground; | ||
} | ||
|
||
.EmailInviteConfirmation { | ||
display: flex; | ||
flex-direction: column; | ||
align-items: center; | ||
padding-top: 160px; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
padding-bottom: 38px; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This can be removed since the footer is already absolutely positioned. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This applies to line 13 |
||
padding-left: 32px; | ||
padding-right: 32px; | ||
font-size: 24px; | ||
height: 100%; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Let's add a |
||
|
||
@media (max-width: 1000px) { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
padding-top: 80px; | ||
|
||
h1 { | ||
font-size: 24px; | ||
align-self: flex-start; | ||
} | ||
|
||
p { | ||
align-self: flex-start; | ||
} | ||
} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The text should be center-aligned even in mobile view. So, this can be removed. |
||
|
||
h1 { | ||
font-size: 36px; | ||
font-weight: 600; | ||
} | ||
|
||
.footer { | ||
position: absolute; | ||
bottom: 38px; | ||
} | ||
|
||
.continue-button { | ||
background-color: #FE7C53; | ||
color: white; | ||
margin-top: 96px; | ||
padding: 8px 24px; | ||
border: none; | ||
border-radius: 16px; | ||
cursor: pointer; | ||
font-size: 32px; | ||
font-weight: 600; | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's change the timeout to 5s. Websites usually mention 30s in case the redirect fails or takes too much time, but they trigger the redirect much earlier on.