TIY-LR-FEE-2015-June / assignments

0 stars 3 forks source link

2.2-timeline-Daniel Condit #57

Open DanielCondit opened 9 years ago

DanielCondit commented 9 years ago

Deliverables

Requirements

DanielCondit commented 9 years ago

https://github.com/DanielCondit/2.2-timeline/pull/1

Took me forever to get everything organized, I feel I'm overcomplicating and taking unnecessary steps.

rtablada commented 9 years ago

@DanielCondit This started to get some of the requirements but there are some things to see when working with positioning. Notice that since you positioned things outside of the parent, you then had to do some pretty hacky stuff and everything after it was affected and had to position EVERYTHING relative. There is a better and simpler position option that will simplify things a lot and will clear up some of that muddiness.

Review

See if you can get things a bit cleaner and less hacky.


I also noticed that your HTML is a bit more complicated that it needs to be. For example, having an h3 inside of .title div with nothing else inside of is redundant. Same with things like date and p. Clear this up.

Todo

DanielCondit commented 9 years ago

Fixed css and html for timeline. Ready for turn in.

rtablada commented 9 years ago

@DanielCondit I don't see any commits pushed...? Maybe still just on your local box?