# Feedback for the My Tribute Page

**URL:** <https://forum.freecodecamp.org/t/feedback-for-the-my-tribute-page/494460>\
**Category:** I Built This\
**Created:** [January 27, 2022, 4:55pm UTC](https://forum.freecodecamp.org/t/feedback-for-the-my-tribute-page/494460 "2022-01-27T16:55:30Z")\
**Posts on this page:** 7\
**Page:** 1

<div class="post-metadata">

**Author:** ![just\_p](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/just_p/32/247117_2.png) [@just\_p](https://forum.freecodecamp.org/u/just_p)\
**Post date:** [January 27, 2022, 4:55pm UTC](https://forum.freecodecamp.org/t/feedback-for-the-my-tribute-page/494460/1 "2022-01-27T16:55:30Z")

</div>

Hello everyone! I made my first project, so I need constructive criticism 😊

> **[Tribute Page](https://codepen.io/just_p/pen/ExbjYJN)**
>
> ...

---

<div class="post-metadata">

**Author:** ![LucLH](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/luclh/32/246602_2.png) [@LucLH](https://forum.freecodecamp.org/u/LucLH)\
**Post date:** [January 27, 2022, 5:43pm UTC](https://forum.freecodecamp.org/t/feedback-for-the-my-tribute-page/494460/2 "2022-01-27T17:43:40Z")

</div>

Hello @just_p,

First, for a beginning it is good! Now one point I think you should considerate is the structure of your HTML. All his content should be within the tags body.

```auto
<!DOCTYPe html>
<html>
  <!-- The head tags are important because they define the meta data of your page/website -->
  <head>
    <meta charset="utf-8" />
    <!-- In CodePen, you don't need the link tag below, but I find it is still. good habit to take -->
    <link rel="stylesheet" type="text/css" href="css/style.css" />
    <title>Your Page Title</title>
  </head>
  <body>
    <!-- Here should be your HTML code, the one you did -->
  </body>
<html>

```

I know, it works well without these updates, but they are necessary. It is like in CSS, if you got only one property into one CSS definition you can not put the semicolon “;” at the end of the property: value. But it is not because it works that people should do like this. It should always have the semicolon at the end of a CSS property.

After, for the CSS, everyone does as he wants. Just look what you can optimise and here you go. The more you will practice the more you will realize! 😉

---

<div class="post-metadata">

**Author:** ![medic1111](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/medic1111/32/246962_2.png) [@medic1111](https://forum.freecodecamp.org/u/medic1111)\
**Post date:** [January 27, 2022, 7:59pm UTC](https://forum.freecodecamp.org/t/feedback-for-the-my-tribute-page/494460/3 "2022-01-27T19:59:12Z")

</div>

Good job overall. I would probably style that figcap perhaps, but that’s just a visual preference thing.  
You go =)

---

<div class="post-metadata">

**Author:** ![Roma](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/roma/32/112899_2.png) [@Roma](https://forum.freecodecamp.org/u/Roma)\
**Post date:** [January 27, 2022, 9:09pm UTC](https://forum.freecodecamp.org/t/feedback-for-the-my-tribute-page/494460/4 "2022-01-27T21:09:40Z")

</div>

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

- Keep the test script when forking the pen (`<script src="https://cdn.freecodecamp.org/testable-projects-fcc/v1/bundle.js"></script>`)  
The test script, **with all tests passing** , should be included when you submit your projects.  
Your page passes **8/10** user stories. Click the red button to see which test(s) are failing and text to help you correct the issue.  
**Be sure and read more than just the first line of the failing message**. The ability to read and comprehend error messages is a skill you’ll need to acquire as a developer. Ask questions on what you don’t understand.
- Accessibility is about being accessible to all users. Review the [giving meaningful text to links](https://learn.freecodecamp.org/responsive-web-design/applied-accessibility/give-links-meaning-by-using-descriptive-link-text) lesson. For a more thorough explanation read [Web Accessibility in Mind](https://webaim.org/techniques/hypertext/link_text).  
“ **статью на wikipedia** ” is not accessible

On a side note, you’ve used codepen correctly.  
Codepen provides the boilerplate for you and only expects the code you’d put within the `body` element in the HTML editor.

In the future, when asking for feedback on a project it’s best to open the topic in the #project-feedback subforum.  
I’ve taken the liberty of moving this for you.

---

<div class="post-metadata">

**Author:** ![just\_p](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/just_p/32/247117_2.png) [@just\_p](https://forum.freecodecamp.org/u/just_p)\
**Post date:** [January 28, 2022, 8:39am UTC](https://forum.freecodecamp.org/t/feedback-for-the-my-tribute-page/494460/5 "2022-01-28T08:39:32Z")

</div>

Yes, the appearance should be improved. Thanks for the feedback

---

<div class="post-metadata">

**Author:** ![just\_p](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/just_p/32/247117_2.png) [@just\_p](https://forum.freecodecamp.org/u/just_p)\
**Post date:** [January 28, 2022, 8:51am UTC](https://forum.freecodecamp.org/t/feedback-for-the-my-tribute-page/494460/6 "2022-01-28T08:51:42Z")

</div>

Thank you so much for your feedback! I corrected everything, now the score is 10/10

---

<div class="post-metadata">

**Author:** ![system](https://global.discourse-cdn.com/freecodecamp/original/3X/e/1/e1c433102e34b46dfaa3e15d13ab9a514b127078.svg) [@system](https://forum.freecodecamp.org/u/system)\
**Post date:** [July 29, 2022, 8:52pm UTC](https://forum.freecodecamp.org/t/feedback-for-the-my-tribute-page/494460/7 "2022-07-29T20:52:02Z")

</div>

Closed on July 29, 2022, 8:52pm UTC

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