Clipboard landing page using Html,Css and Scss

Community feedback

• 

  P

  Fluffy Kas•7,715

  @FluffyKasPosted almost 3 years ago

  Hiyo,

  Well done, it looks pretty good! Responsiveness is also fine, except a few smaller issues.

  1. Your font-size is very small below 770px, almost unreadable. You should definitely increase that.

  2. Heading levels are a bit all over the place. There's an important reason to have some sort of structure when using them. You could learn more from this video.

  3. In the footer, your social media icons should be wrapped in links (since it's their function to point at social media websites). Don't forget to include aria-labels ^^

  4. Alt texts: read this article, then perhaps you could make some adjustments :)

  Other than these, you did very good so well done! Happy coding ^^

  1

  ---

  

  Ozan Zorlu•280

  @zorluozanPosted almost 3 years ago

  @FluffyKas Hi, thank you for your feedback.

  0
• 

  Dev Rathore•2,600

  @GitHub-dev12345Posted almost 3 years ago

  Congratulation to complete your challenge: my small Suggestion for your project: you give the <br> ( Break tag ) in para line, check the project design paragraph, and then give the <br> tag according the deisgn:

  I hope you find this helpful👍

  0

  ---

  

  Ozan Zorlu•280

  @zorluozanPosted almost 3 years ago

  @GitHub-dev12345 Thank you for your feedback.

  0

  

  P

  Fluffy Kas•7,715

  @FluffyKasPosted almost 3 years ago

  @zorluozan Don't do that :D @GitHub-dev12345, please read what br element is for.

  0

  

  Dev Rathore•2,600

  @GitHub-dev12345Posted almost 3 years ago

  @FluffyKas br element for break the line properly , and i personal used br tag in my project got it 😎

  0

  

  Ozan Zorlu•280

  @zorluozanPosted almost 3 years ago

  @FluffyKas I didn't do that :D . I think there may be some css tricks to do that thing but I don't know.

  0

  

  P

  Fluffy Kas•7,715

  @FluffyKasPosted almost 3 years ago

  @zorluozan Yes, if you wanted your text to follow the design and break at certain point, you can add a max-width on it (and margin: auto to center it) to control its width ^^

  0

  

  P

  Fluffy Kas•7,715

  @FluffyKasPosted almost 3 years ago

  @GitHub-dev12345 As I said above, please read MDN. It specifies exactly when it should be used and when isn't. ^^ br element has a semantic meaning. If you'd like to style any text, you should do so by using css!

  0