Huddle Landing Page With a Single Introductory Section

Community feedback

• 

  Lucas 👾•104,340

  @correlucasPosted about 2 years ago

  Hello Yunus, congratulations for your new solution!

  I saw your solution live site and the html markup, there's one thing to improve regarding the markup, you can replace the block wrapping the logo as header or nav if you mKe the logo as link. And improve the general content aspect giving less padding between the logo on top ane the social icons bottom. And use max-width to limit how much the hero image and section can grow.

  👨‍💻Here's my solution for this challenge if you wants to see how I build it: https://www.frontendmentor.io/solutions/huddle-lp-section-vanilla-css-logo-animation-easter-egg-bem-meb3nl6zF8

  Marked as helpful

  1

  ---

  

  Yunus Emre Çınar•360

  @yunusemrecinarPosted about 2 years ago

  @correlucas Hi, Thanks for your advice I did changed what you said. I think I'm using <header> for logo. Is not it ? And one more thing. You're saying, give less padding between logo and top. I'm assuming this is for mobile version right ?

  0

  

  Lucas 👾•104,340

  @correlucasPosted about 2 years ago

  @Yunuscinar41 you've used a class as header         <div class="header"> not tag

  I open it and about 1000px there's a gap between the header and footer, its the same for you?

  1

  

  Yunus Emre Çınar•360

  @yunusemrecinarPosted about 2 years ago

  @correlucas Yes, it's same to me. I thought, that will be look nicer but I changed. I changed the tag too. Sorry about that I thought you mean the class one .

  1

  

  Lucas 👾•104,340

  @correlucasPosted about 2 years ago

  @Yunuscinar41 thats fine, then I'll open again with my computer, I've opened with a tablet with desktop mode and give you ahain a feedback, but anyway are only two details, the solution overall is fine Yunus 👏

  1
• 

  Eduardo•910

  @KTrick01Posted about 2 years ago

  Hi! Try to add two dots at the start of your url, like this: background: url("../images/bg-desktop.svg") no-repeat; If im not wrong that will solve the problem!

  1

  ---

  

  Yunus Emre Çınar•360

  @yunusemrecinarPosted about 2 years ago

  @KTrick01 Hello there! ./images/bg-desktop.svg") no-repeat; and .images/bg-desktop.svg") no-repeat; will solve. I'm actually writing this but you went fast :)

  0