Skip to content
Submitted over 3 years ago

Responsive landing page using flexbox

Mohamed•325
@MohamedBehhar
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Huddle landing page with a single introductory section
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

I just finished the last challenge at the newbie level, I'm very happy, I would like to invite you guys to view my solution and give me your feedback.

Community feedback

  • Anna•545
    @NitaLewskaPosted over 3 years ago

    Also your icons don't have a round border=)

    0
    Mohamed•325
    @MohamedBehharPosted over 3 years ago

    @NitaLewska I used fontawesome library and I didn't found identical icons, what library did you use.

    0
    Anna•545
    @NitaLewskaPosted over 3 years ago

    Fontawesome, and added borders manually

    0
  • Anna•545
    @NitaLewskaPosted over 3 years ago

    Hi! it seems the background image isn't showing properly

    You should delete the first slash here: background-image: url(/images/bg-desktop.svg); -> background-image: url(images/bg-desktop.svg);

    0
    Mohamed•325
    @MohamedBehharPosted over 3 years ago

    @NitaLewska Thanks a lot for your feedback, yes I didn't pay attention before uploading the project. Actually, I added two point before the background URL like this "../images/bg-desktop.svg" since the bg is located in the folder one level up from the current folder, and its work.

    1

Please log in to post a comment

Log in with GitHub

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