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

Base Apparel Coming Soon Page | HTML5, CSS Flexbox, JS

@syzwnftr

Desktop design screenshot for the Base Apparel coming soon page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Any feedback for improvement are greatly appreciated.

Community feedback

@pikapikamart

Posted

Hey, awesome work on this one. Layout in desktop looks great, it is responsive and the mobile layout is great as well. One thing though, inspect the layout in dev tools at the bottom, the img starts to shrink.

Some suggestions would be:

  • The alt value for the website-logo should only be alt="base apparel" lose the word "logo", since img is already an image, no need to describe it as one.
  • The input as well as the button should be inside a form element, since this will, on real site, will send data.
  • The input should have aria-invalid="true" being set to it, if the user submitted a wrong input:
if ( input is wrong){
  input.setAttribute("aria-invalid", "true");
} else {
  input.removeAttribute("aria-invalid");
}

This way, users will know that there is an error. Also have a search for aria-describedBy you will use this to link up the id of the error-message to the input element.

  • The button should have type="submit" and have aria-label="submit form", this way user will know what this button will do.
  • Lastly, the img that you are using, do not use max-height: 100vh, this creates the bug that inspecting layout in bottom dev tools, the img shrinks. You don't have to add that really.

Aside from those, really great job on this one.

Marked as helpful

0

@syzwnftr

Posted

@pikamart Hey, I really appreciates the thorough and detailed feedback you provided. I will look into these and make a change as you suggested. If you don't mind, can you review my previous challenge? Because I'm new to this and I really need this kind of feedback in order for me to improve. But of course, if you don't mind. Anyway, thanks again for this feedback.

0

@pikapikamart

Posted

@syzwnftr Sure, but maybe after 2-3 hours if you read this reply, just going to code some in the morning and after it, I will review some of your previous work^^

0

@erickamae-mateo

Posted

It's great! You just need to add spacing between your text. Whitespace helps your design to be more readable and words won't look crowded.

0

@syzwnftr

Posted

@erickamae-mateo Yeah you're right. I probably need to add more spacing. Thanks for your observation.

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