Scroll top #46
Scroll top #46
Conversation
✅ Deploy Preview for color-conjure ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
src/pages/Home.jsx
Outdated
| @@ -167,6 +196,15 @@ const Home = () => { | |||
| } // Callback when colors are generated | |||
| /> | |||
| ))} | |||
| {isVisible && ( | |||
| <button | |||
There was a problem hiding this comment.
We have a component in Shared, you can use that
| @@ -75,7 +76,7 @@ const Home = () => { | |||
| [id]: colorData, | |||
| })); | |||
| }; | |||
|
|
|||
| const baseColor = Object.values(generatedColors)[0]?.baseColor; | |||
There was a problem hiding this comment.
Please add a code comment explaining what is this for
src/pages/Home.jsx
Outdated
| @@ -120,6 +121,34 @@ const Home = () => { | |||
| () => toast.error("Failed to copy colors to clipboard."), | |||
| ); | |||
| }; | |||
|
|
|||
| const [isVisible, setIsVisible] = useState(false); | |||
There was a problem hiding this comment.
You can move this useState below line 22
Bro in India it's 1:20am i have to sleep now... I have to wake up early tomorrow... I'll fix these and submit the PR first thing in the morning. |
|
Yeah sure, whenever it suits 😅 I leave comments so that I can / you can look at them later |
Cool 😅 |
|
Any updates on this @mohtasheem135 ? |
|
Sorry bro actually I was not well... That's why I was unable to share the PR. I will give the the PR by today. Again sorry for the delay. |
|
check now bro... Again sorry for being so late |
|
The project has been updated to use Typescript, please pull the latest |
Description
Added Scroll-to-top button
Fixes #32
Type of Change
Checklist