search

rodivansito / capstoneproject

0 stars 0 forks source link

Layouts #2

Open tommygebru opened 5 years ago

tommygebru commented 5 years ago

You're layouts are looking pretty good Rodrigo, the only thing I would suggest is doing more to use the negative or whitespace in your project. The typography is pretty small, so thats something you can tweak to take up more space. Also another thing is you could play with a 960px layout like in the project SecretAgent!

tommygebru commented 5 years ago

Your Courses or Learning section carry different classnames. Consider reducing your codebase so that this section of the project uses one classname and one group of CSS styles For example 

.coursesAll {
height: 18rem;
width: 15rem;
background: white;
margin: 2rem;
}
tommygebru commented 5 years ago

Similarly you can apply this kind of standardization and using fixed widths with repeating html containers within each section. For example, the within the div.details there is a repeating html elements that can be rewritten to use more of the same code.

Also this applies to the html content within the Thesis section, specifically within the aside element aside.aside