github / training.github.com

Legacy (no longer published) GitHub Training site
79 stars 46 forks source link

optional topics, seriously #119

Closed brntbeer closed 11 years ago

brntbeer commented 11 years ago

cc @cobyism. wasnt sure weather or not to use a "whole" div there or not.

Additionally, we could move the "Objectives" section below "topics and optional topics"

I'm torn!

matthewmccullough commented 11 years ago

Suggestion. Feel free to ignore.

cobyism commented 11 years ago

For layout, yeah .whole is fine there, as long as the list on the right doesn’t get too much longer than the left one—if that were case you’d be better putting that bottom section at the bottom of the left column, which I think is what you were suggesting, no? I’d say just leave it where it is unless it becomes an issue.

brntbeer commented 11 years ago

@matthewmccullough will do

@cobyism thanks!

jordanmccullough commented 11 years ago

Thanks for the updates @brntbeer. Looks good to me, including the removal of SSH. :+1:

brntbeer commented 11 years ago

One thing no one caught is that i had the "making git work with svn" in there twice :)