@hyunnies95
Nick
@Nick331102All comments
- @Nick331102
Great job, looks very nice. Best one I have seen
- @Leonard-O@Nick331102
nice work. kevin powell did a recent segment about grid using this project as an example if you are looking for more clarity.
https://www.youtube.com/watch?v=rg7Fvvl3taU&t=1438s
I would play with your media query a bit so it breaks earlier, maybe around 800-850px.
Marked as helpful - @danM51@Nick331102
nice work. kevin powell did a recent segment about grid using this project as an example if you are looking for more clarity.
https://www.youtube.com/watch?v=rg7Fvvl3taU&t=1438s
setting a max-width om your main container would make it responsive, and then you can utilize media queries and change your grid template areas as needed.
Marked as helpful - @DavidMaillard@Nick331102
If you have a free moment would you be able to look at my solution for this challenge? I haven't been able to get the error messages to show using JS for the form validation.
Thank You
- @madhusudanbabar@Nick331102
This is the easiest way I have found for the overlay, but there are other ways try using an inset box shadow box-shadow: inset 0 0 0 1000px rgba(109, 54, 199, 0.596); you can adjust the color.
Marked as helpful - @Shyam101@Nick331102
This is the easiest way I have found for the overlay, but there are other ways try using an inset box shadow box-shadow: inset 0 0 0 1000px rgba(109, 54, 199, 0.596); you can adjust the color.
- P@somecallmejosh@Nick331102
nice job. Looks great
- @smhmurat@Nick331102
This is the easiest way I have found for the overlay, but there are other ways try using an inset box shadow box-shadow: inset 0 0 0 1000px rgba(109, 54, 199, 0.596); you can adjust the color.
- P@outerpreneur@Nick331102
This is the easiest way I have found for the overlay, but there are other ways try using an inset box shadow box-shadow: inset 0 0 0 1000px rgba(109, 54, 199, 0.596); you can adjust the color.
you can utilize media queries to make your card more responsive, so I would research them.
if you want the columns to be the same width you can display: grid on the container and set grid-template-columns: repeat(2, 1fr)
Marked as helpful - @jchapar@Nick331102
Nice work overall. You may want to wrap everything in a main and set a max-width, as I viewed your project in a widescreen monitor and would look better with a max width. You could also add a hover state to the request access button. As you mentioned, the margin and padding need to be fixed for the text. The project does look pretty good on mobile though. The success message for the email validation doesn't seem to work. You could reference my project if you would like.
well done
Marked as helpful - @xvferdy@Nick331102
nice work. kevin powell did a recent segment about grid using this project as an example if you are looking for more clarity.
https://www.youtube.com/watch?v=rg7Fvvl3taU&t=1438s
- @tomsky90@Nick331102
Very nice. This is quite a large project, looks great
- @effycoco@Nick331102
Nice work.
You might try making your main container more responsive by using max-width: 918px instead of using a fixed width. Also, you can add a hover state to or cursor:pointer to the learn more buttons.
- @wcgordon1
- @Kaji1127@Nick331102
Nicely done. The only thing I would say is to maybe have your hamburger menu lines rotate on click and add some hover states for the nav links in the mobile menu. Other than that it looks great.
Marked as helpful - @hamidousow@Nick331102
If the you look at the JS in my pricing.js file for the photosnap project it would be helpful to you and give an idea about how to toggle the value.
https://github.com/Nick331102/Photosnap
Marked as helpful - @nashee@Nick331102
thank you
- @ArturdaSilvaRezende@Nick331102
thank you
