Technical documentation page - What can i improve?

It’s my another FCC project, Technical documentation page. Please give me your feedback to help me to improve my skills. I really appreciate every comment and advice.



Thank you in advance.

1 Like

@karolroldi, looks great! :100: It passes all the tests and works well.

I see you’re using the <article> element. Do you need that? I don’t see it being used in your CSS. It seems like you can take it out and have the following structure instead:



Also, some say that selecting IDs in your CSS will lead to more complicated and slower code when working in larger projects.

Other than that, nice work! :sunny:

1 Like

Great work… All I can say… May be in future you can include the top navs in a humbug …

Looks clean. I noticed two things. First, personally I think the code box font-size is a little too small for me when viewing on a large screen. Maybe you could put a media tag for that. Second, when resizing to smaller screen (<400px), your image:

<img id="css-img" src="" alt="contest of the DOM">

is a little awkward. I think because it’s part of the ordered list element, so it’s indented, but it looks like it would be better off with auto left and right margins.

Looks very clean, though, otherwise.

I second this. Very good idea!

1 Like

I would say work on the menu make it better desktop view and for mobile view.

1 Like

You’re right i don’t need it. I thought that it is better to put all <section> elements to <article> which will contain entire content.

thanks for your reply :slight_smile:

I’m sure i’ll do this when i improve my skills.

Font size is bigger now. Does it look better?
I forget to put </ol> element so the image was a part of the ordered list. Hope it’s better now.

Thanks for your feedback.

Yes, it does.

I’m not sure why, though, but when I checked your code from my phone, it was not responsive. Only from the desktop when resizing the screen.