@michelleracho
Latest comments
- @kvncnls
Looks great Michelle!
- It looks like there's a small HTML issue with the label. It shouldn't have a div inside of it.
- If I had to be nitpicky, I'd just say that the default state should be monthly just like the design comparison. :P
- P@hhamza1@kvncnls
Hi Hamza, nice work!
- There's a small bug around the 769px to 1023px media query where the SVG covers the "FAQ" text.
- Also, the text should change colors on hover.
Other than that, looks great!
- @ApplePieGiraffe@kvncnls
Hey APG! Your site is awesome! Can I ask how you made the navbar close when you have it open on mobile/tablet size then expand to desktop size?
So let's say your window was 500px width, you open the navbar menu, then you expand the window passed the navbar menu limit (where the hamburger menu should disappear). How did you get the navbar to close automatically? I don't think I saw an event listener on window resize.
- @jerline01@kvncnls
Use the Dev Tools and check your layout at 375px width (iPhone X size). The purple background quote sticks out of the card. I used 375px because that's what the style guide said (375px and 1440px). The text also gets too narrow, so it's probably a padding issue.
