Feedback for my first 3 projects

Hello, friends.

Not so long time ago, about 4 days, i started to study in FCC.
I don’t know yet can I actually become a programmer someday, because nearly always, while i study, it seems to me that I’m completely dumb in programming.)
But here are my first 3 pens:
Tribute page: https://codepen.io/coldindoor/pen/YzNBdzm
Survey form: https://codepen.io/coldindoor/pen/LYxaVgL
Product landing page: https://codepen.io/coldindoor/full/xxgeGJZ

Please, feel free to give me feedback. I will be glad.

2 Likes

Hi @bashirovin !

In the future, you will get more responses if you make seperate posts for each project.

You just started. It is to early to start making predications about whether or not you will become a programmer.

Just focus on the learning.

Whenever you start something new, you will not be that great at it.
It takes time and practice :grinning:

Tribute page feedback
Keep the test suite in your page
<script src="https://cdn.freecodecamp.org/testable-projects-fcc/v1/bundle.js"></script>

You are passing 8/10.
Read the full error messages to find and fix those issues.
I would review the lesson on giving links meaningful text.

Wikipedia entry is not that accessible.

You have a few errors in your html and css.
Run your code through the html validator and css codepen analyzer.

Survey form
Keep the test suite in your page
<script src="https://cdn.freecodecamp.org/testable-projects-fcc/v1/bundle.js"></script>
You are passing 15/17.
Read the full error message to find and fix those issues.
You have a couple of errors in your html.
Run your code through the html validator
You should add the cursor pointer to the submit button.
You need to work on responsiveness for smaller devices.

Product page
Keep the test suite in your page
<script src="https://cdn.freecodecamp.org/testable-projects-fcc/v1/bundle.js"></script>
You are passing 7/16.
Read the full error messages to find and fix those issues.
You have a few errors in your html and css.
Run your code through the html validator and css codepen analyzer.

You might consider adding a smooth scroll.
I would add a cursor pointer to the email submit button and let’s get to work buttons.

You need to work on responsiveness for smaller devices.

Hope that helps!

1 Like

Thanks for answer.
How to use this script?
script src=“https://cdn.freecodecamp.org/testable-projects-fcc/v1/bundle.js”></script
Where should I input it?

I think your projects have a lot of potential, it’s hard to think you’ve done this in just a handful of days. Keep up the good work!

The code for the script goes at the top of your HTML section.

Something that jwilkins.oboe mentioned was making your projects responsive for smaller screens. When I first started I went the same route as you and wrote my first project to be for a desktop and then tried to make it work smaller. But, it’s much easier to write your pages starting out small then adjusting them for larger screens. I currently write mine to fit a screen of 320 pixels to start out with (the smallest screen size I could find), then see where my page looks weird when it gets bigger and make those adjustments.

2 Likes

Yes, it works with script) Thanks)
How are you changing screen on 320px to look?:slight_smile:

When viewing your project in CodePen on a large screen there is a button that says “Change View”, at the bottom of that menu is a layout that looks like a large box on the left and 3 small boxes on the right. Choose that one, then you can adjust your page’s width.

I’m away from my desktop, just let me know and I’ll post a screen shot next time I’m on.

I found this layout, thank you. Is this 320px screen already? Or I need to change something?

God, my survey is looking funny in this layout.:slight_smile:

You’ll need to click the right side of the of the preview window and drag it, a little box will pop up saying how many pixels it is.

I’ve had to completely start over some of my projects since it was easier than trying to fix what I had done, haha. I think they came out better for it though.

Ok, i did it. So I need to work in 320px to make more responsive pens?

That’s what I do, I’ve also seen similar advice in the forums from other users.

Keep the test suite in your page
Why?

Hi @darrenwoodford !

Welcome to the forum!

One main reason you should include the test suite is because when you ask for feedback on the forum, people will check if you are passing all of the tests.

Another good reason, is that if you need help with a project and don’t understand the error message, people can open up the test suite on your project and assist you.

Ah OK, thanks. I always thought it was a good idea to separate your JS and if you link to it then you also get the benefit of auto updates.

I think there is some confusion on what the test suite is.

This is the test suite.

<script src="https://cdn.freecodecamp.org/testable-projects-fcc/v1/bundle.js"></script>

It is ok to include this in your html while you are building out the FCC projects.

All other javascript that you add should be in a separate section.
Hope that makes sense