Portfolio Project Feedback, first time using bootstrap

Code Feedback
1

Hey guys,

Just finished my portfolio pen http://codepen.io/ali-ahmad/pen/yMMmQE

Feedback would be great. I’m trying to center the green FCC button, any advice on that? Thank you!

2

Hi! You green FCC button is inside the div with id “about”. You can add the “text-center” class to this div. If you don’t want all the contents inside your div to be centered, then enclose the button inside a

or

and add the “text-center” class only to that

or

. Hope this helps.
1 Like
3

I don’t think your bootstrap row divs are actually doing anything. You’re closing it as soon as you’re opening it. Instead you should be putting contents inside it. You don’t necessarily need to define row/class either. Remember it’s only useful when putting things side by side. If you want things to just be on top of each other you don’t really need it. If anything the col classes are actively stopping the button from being centered with CSS.

1 Like
4

Hi,

Following what jx2bandito wrote:

you should have your row div that include your column divs for each sections.
This is how you build the grid system.
For more information on that you can take a look at the html/css 101 at udacity, it really helped me understand.
Here is the link to the course:
https://classroom.udacity.com/courses/ud304/
One more tips:
To see where goes what you can add the following into your css :

*{
 border: 1px solid red !important;
 }

This way you see exactly what is aligned, where and how. Once you are finished with your main grid layout you can then remove it.

It helped me a lot once I figured that out.

1 Like
5

Thanks for the advice guys, it’s really helpful. :slight_smile: