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

Scroll button functionality #127

Merged
merged 6 commits into from
Feb 6, 2023
Merged

Conversation

jenniferchen11
Copy link
Contributor

@jenniferchen11 jenniferchen11 commented Jan 20, 2023

Screen Shot 2023-01-25 at 1 24 27 PM

Screen Shot 2023-01-25 at 1 25 17 PM

@netlify
Copy link

netlify bot commented Jan 20, 2023

Deploy Preview for gifted-fermat-a1fc62 ready!

Name Link
🔨 Latest commit 12be401
🔍 Latest deploy log https://app.netlify.com/sites/gifted-fermat-a1fc62/deploys/63da07c8561e110009cc0a38
😎 Deploy Preview https://deploy-preview-127--gifted-fermat-a1fc62.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 site settings.

@jenniferchen11 jenniferchen11 marked this pull request as ready for review January 25, 2023 21:33
Copy link
Member

@justinyaodu justinyaodu left a comment

Choose a reason for hiding this comment

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

Tiny nitpick, otherwise lgtm 😍


import CustomImage from "./CustomImage";

export default function Navbar() {
Copy link
Member

Choose a reason for hiding this comment

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

Looks like this should be named ScrollButton? Although tbh I make this mistake everywhere and the name doesn't actually matter 😅

Copy link
Contributor Author

Choose a reason for hiding this comment

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

LMAO i copied and pasted from another file and forgot to change...

Copy link
Contributor

@navidboloorian navidboloorian left a comment

Choose a reason for hiding this comment

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

The image seems a little low res and the top is cut flat, can you guys make it an svg? (Can download the svg from the figma itself by right clicking on the left-hand side and saving as svg)

@jenniferchen11
Copy link
Contributor Author

@navidboloorian For some reason the icons have to be PNGs (I tried it with the SVG and some of the properties cannot be applied to it). Could you take another look at it and let me know if the image quality is noticeably low?

@navidboloorian navidboloorian self-requested a review February 1, 2023 06:34
Copy link
Contributor

@navidboloorian navidboloorian left a comment

Choose a reason for hiding this comment

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

Yeah it's actually not noticeable, I fixed the image cutoff issue where the top of the circle was cut off. LGTM!

@jenniferchen11 jenniferchen11 merged commit 73b3737 into main Feb 6, 2023
@jenniferchen11 jenniferchen11 deleted the feature/jenniferchen/scroll-button branch February 6, 2023 21:57
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.

3 participants