Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

React Website (Responsive, BEM, Custom Hooks, Global useContext)

@jhunxein

Desktop design screenshot for the Huddle landing page with curved sections coding challenge

This is a solution for...

  • HTML
  • CSS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hello everyone. This is the edited and maybe the final design of this web. I fixed some issues that sir @pikamart mention, big thanks to you sir.🤙 Although there are still some minor issue when the window resolution is higher (around 2000px wide). But yeah, guess this is ok.

Did my best to this one. 💪✌

Happy Coding Everyone!

Community feedback

@pikapikamart

Posted

Hey, great work on this one. Layout is good both in desktop and mobile and it resizes well.

A couple of suggestions

  1. Maybe you could adjust, lower the breakpoint for the newsletter section? Because many users won't be able to see the desktop layout for it since the width you specified in min-width is much higher than the resolution of the browser like mine. Lowering it down will be awesome

  2. Buttons. There are components which is supposed to be a link, but button tag was used instead of a tag which is appropriate in this one. Changing them to a tag will make the markup more semantic.

Regarding you query. Well.

  1. I think making it as a background-image is the right way? Since they are just for decorations right so background-image is the best choice. But, as you said they will be cut out. Why? Because the container that will contain them don't have the needed height right. But still it is achievable with a little bit of tweaking using paddings or height. Like adding a padding-top to an article section and that allotted padding-top space is for the background-image of the curvy thing. That way, you can placed it fine and avoid usage of img tag. Because the padding-top will push the content towards bottom right, so there will be a space in the top. That is where you place the background-image. The position is just background-position: center top something like that.

I hope I gave some sense to what I have said. Overall, you did really good in here. While you're at it, it will be great to add some text if user enters wrong email. Well that is just a suggestion ^^

1

@jhunxein

Posted

Hello @pikamart, I'll definitely do what you said. Thank you for your words 😍

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord