This is my tribute page. Much appreciated for your feedback :-)

Project Feedback
#1

Hey everyone! I’m working on Responsive web design projects… I’m a total beginner in web development. Please help me to find my flaws .

This one is a tribute page to Kazi Nazrul Islam:

https://codepen.io/MRB-Faysal/pen/dyvZVME

Thanks for your help!

1 Like
#2

Welcome to the forums @MRBFaysal. Your page looks good. Some things to revisit;

  • Codepen provides the boilerplate for you. It only expects the code you’d put within the body element in the HTML editor. (No need to include the body tags). For anything you want to add to the <head> element click on the ‘Settings’ button, then HTML and add it into the ‘Stuff for <head>’ box.
  • Run your HTML code through the W3C validator.
    • There are HTML syntax/coding errors you should be aware of and address.
    • Since copy/paste from codepen you can ignore the first warning and first two errors.
  • Codepen provides validators for HTML, CSS and JS. Click on the chevron in the upper right of each section and then click on the respective ‘Analyze’ link.
    • The one for CSS is good. Use it and address the issue(s).
    • (The one for HTML misses things which is why I recommend W3C)
  • Accessibility is about being accessible to all users. Review the giving meaningful text to links lesson. For a more thorough explanation read Web Accessibility in Mind.
    • “wikipedia entry” is not accessible
  • The image does not show.
    1. It’s known that imgur sometimes blocks codepen. You’ll should find another free image hosting site to use.
    2. You’ve linked to the imgur URL that holds the image. You need to link to the image on that page…not the entire page.
1 Like
#3

Thanks a lot Sir.
The image and wikipedia entry is fixed here: https://codepen.io/MRB-Faysal/pen/dyvZVME .

#4

The image at the top is over passing the width of the page, making it took ugly. Try changing the width of the image to a max-width of 100%, like so: max-width:100%.

This is to keep it in the bounds of the website but do not set the height. Tell me if that works

#5

After changing max-width:100%
it appears like https://codepen.io/MRB-Faysal/full/dyvZVME
Is it okay Sir?

1 Like
#6

Yes, this is much better. If my solution has helped you, you can leave a like or something :grinning_face_with_smiling_eyes:

1 Like
#7

It looks better @MRBFaysal. Seeing the image displaying properly now. There are still some issues I originally mentioned that still need to be revisited;

  • Run your code through the W3C validator. The link was provided above.
    • There is an error that you need to be aware of and address.
  • Once again, this is codepen so you do not need the html elements and you do not need the link to the stylesheet. Codepen does this for you.
  • Something I just noticed. User’s are not seeing your page the way you want. You have font-family: Segoe UI;.
    • Not everyone uses Microsoft and has that font downloaded.
    • You provide no default font (sans-serif or serif)
    • As a suggestion, link (head element in HTML) or @import (CSS) a font from Google fonts.
#8

This topic was automatically closed 182 days after the last reply. New replies are no longer allowed.