First time using gatsby, still alot to learn specially on animation and transitions. All feedbacks are greatly appreciated.
-edit Code refactored, SEO and Manifest added. fixed some alignment issues at pricing page.
Looking to hire developers?
@justAdevTV
First time using gatsby, still alot to learn specially on animation and transitions. All feedbacks are greatly appreciated.
-edit Code refactored, SEO and Manifest added. fixed some alignment issues at pricing page.
@justAdevTV
Posted
@flquenano this is AMAZING! Super clean and I love the animations you added in.
Suggestions for improvement:
alt
tags for images (it's good for accessibility)./pricing
page has a horizontal scroll starting at screen width ~540 and lower. Removing the horizontal scroll will make it a lot better.Again, really nice work. It's hard to find points for improvement at this stage.
Keep up the great work 🙏
The rating stars look a little off. In addition, I cound not figure out why the page has a vertical scroll. I am just starting with web development, so I am sure my code has a lot of issues. This is my first sito from scratch. Thank you.
@justAdevTV
Posted
@jdvillarroel, this looks GREAT! I wouldn't have even known that it's your first site from scratch. Love that it's responsive too.
Here are some suggestions for improvement:
.grid-container
, if you add padding-bottom: 0;
it should remove the scrolling issue you're experiencing.<li>
tags in .rates-list
display:flex;
. Then make .stars-container
have a top and bottom margin of auto
. From there it should work!Again, amazing work! Keep it up 🙏🏼 -Justin P
@tafnesj
Submitted
I'd like to receive feedbacks from my code.
@justAdevTV
Posted
Looks pretty good on my desktop and phone!
Suggestions for improvement:
<a>
tag for accessibility too.Overall, nice work!
Keep it up 🙌
@chri55
Submitted
Please let me know what you think. Thanks!
@justAdevTV
Posted
Nice work 😃
Everything works really well and it's hard to find a point to improve on haha. I'm really impressed with the keyframes!
My one suggestion would be to add an effect when the buttons are clicked on.
Keep up the great work 🙌
Any feedback is welcome. c:
@justAdevTV
Posted
It looks great! I agree with what @ApplePieGiraffe has to say though.
Other than that my only suggestion would be to add some sort of hover effect to the button you have there. Stuff like changing the mouse to a pointer. Check these out if you want references:
Hope this helps!
@josno
Submitted
Any feedback would be great! Especially with CSS.
@justAdevTV
Posted
@joanne it looks good on my phone and desktop. Nice work :D
Suggestions for improvement:
Again, great work overall 😄
-Justin P
@Mostafa-T
Submitted
Hey,
Kindly if you have time review my code and tell me if there any comments to improve the CSS or how could i make it more effective.
All comments are welcome and thank you in advance
@justAdevTV
Posted
@Mustafa, it looks pretty good on my desktop. Nice touch with the share button animations!
Suggestions for improvement:
Hope this helps, Justin P.
@raslab
Submitted
@justAdevTV
Posted
I like the footer and hover effect that you added in. It's a nice touch!
Suggestion/Extra-Challenge:
Best, Justin P
@foolmonkey
Submitted
Feel free to provide any feedback or critiques with my code/solution!
@justAdevTV
Posted
Andy, this is really cool! The loading mouse effect is a nice touch 😃
My revision suggestions:
Great work overall, Justin P
@DevcodeCJ
Submitted
Comments are always welcome. Thanks!
@justAdevTV
Posted
Great work overall 👏
It's responsive and pretty!
The only thing that sticks out to me is that you have more than one h1 tag on your page. Your main header, "Our Pricing", should be in an h1 tag and the card titles like "Basic" should be in h2 tags.
I hope you find this useful, Justin P
@ClemHlrdt
Submitted
@justAdevTV
Posted
Nice work 😃
I really like the shadows! My only tip would be that if you really want to get pixel-perfect, check this plugin out: https://chrome.google.com/webstore/detail/perfectpixel-by-welldonec/dkaagdgjmgdmbnecmcefdhjekcoceebi
Best, Justin P
@MilitusInnocent
Submitted
Please, could you please go through my solution and suggest ways I can improve? I would really appreciate!
@justAdevTV
Posted
@Militus! I'm happy to see you pushing out another GREAT project 😃
Please message me on slack, I'd love to get connected with you there!
Because you asked, some suggestions for improvement:
Overall amazing work. I can't wait to see what else you push out.
Hoping to hear back from you 😃
@joshysmart
Submitted
@justAdevTV
Posted
Hey @joshysmart,
Great solution overall! The filters work on my browser btw.
Also, I agree with @davidomarf's comments regarding widths 780-1080 and the clear button.
I also did this challenge recently! It'd be cool to get your thoughts on my solution if you have time: https://www.frontendmentor.io/solutions/near-perfect-pixels-11-w-nextjs-and-styledcomponents-FYfg1bDZY
P.S. I didn't bother fixing up the JSON so that the tags aligned with the mockups lol. Do you think one of us should fix it and share it with each other?
@MilitusInnocent
Submitted
Hey guys! Please can you go through my latest project and drop some feedback? That'd be greatly appreciated!
Thank you.
@justAdevTV
Posted
Hey Militus,
Great work overall! Your live example works well on my desktop and my phone :)
I see that you used React to make your project too.
I made mine with NextJS (React SSR) & styled-components instead. I'd love to know your thoughts on my approach if you have time.
Check it out here: https://www.frontendmentor.io/solutions/near-perfect-pixels-11-w-nextjs-and-styledcomponents-FYfg1bDZY