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

Mobile First design for Meet Landing Page

P
Shaun Pour 630

@ShaunPour

Desktop design screenshot for the Meet landing page coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


What are you most proud of, and what would you do differently next time?

The shifts from mobile to desktop and using different images per screen size was an interesting challenge since at the time, this was a new concept to me (I originally did this a good while back but had to go back and tweak parts of it more recently).

What challenges did you encounter, and how did you overcome them?

The blueish color of the image was a little tricky and I couldn't figure out a way to do that directly with my image so resorted to changing the background color of the text element that went on top of it and adding some extra padding to give it that color.

What specific areas of your project would you like help with?

Anything you like.

Community feedback

Mirjax2000 680

@Mirjax2000

Posted

Hi i checked your code.

you need to make 1440 px container. Some peeple have very wide monitor my is 5140px long. so when i full screen this two photos are one meter on the sides form the center fo the screen.

max-width: 1440px;

dont use IDs, Save them for forms and javascript. There are other techniques how to target elements. Check another selectors, for example

nth(1)child:
last and first child
first of type
last of type
:not()
> *;
and many more.

too much DIVs, use MAIN, Section,article,aside, footer,nav.header etc... It will help you with SEO evaluation and also webbrowser will know beeter how to print the page on the screen, and also some readers for deaf peeple will better understand what section they are in.

try BEM method to better describe classes in HTML.

dont set font to fixed values in pixels use REM or EM units. Some user has different default font size in their browser, if you hradcode it it will displeay at 18px, even they sets to 24 for example.

that footer is mess. sorry for that.

 <div class="bottom">
          <img src="images/mobile/image-footer.jpg" alt="" class="footer-img-mobile">
          <img src="images/tablet/image-footer.jpg" alt="" class="footer-img-tablet">
          <img src="images/desktop/image-footer.jpg" alt="" class="footer-img">
      </div>

this is bad aproach. First of all what borwser do is loading HTML structure. CSS and Javascript after all HTML. those images will load with HTML and they will be displayed antil browser load CSS codes. It will also take lot of bandwith, every picture has some size and it will slow downloading all site. Instead use picture elements and resolution switching technique. It is hard one, but as you see designer of that challenges do this whit pourpuse to use that.

Also work on Fonts, there is bad fonts. Try to host them from your side. @font-face is the way.

that thing with backround picture on footer is big one. You can use position that picture element to your parrent. or give footer element some width and hight and use background picture as filler.

i hope i help you a bit.

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