Skip to content
This repository was archived by the owner on Jun 25, 2022. It is now read-only.

Conversation

@changemewtf
Copy link
Contributor

This will finally provide an authoritative source for the most trivial rack use cases.

I also changed the site to a basic two-column layout, with the incredibly-long News section on the left and everything else on the right. This makes it much easier to scroll down and see what information is available, as opposed to getting lost in the News and Releases lists.

You can see what it looks like here.

The CSS has been tested in Chrome and Safari.

Syntax Highlighting is achieved with a CDNJS copy of the "syntaxhighlighter" library. I'm not a fan of the color scheme, but couldn't find a better theme with a non-white background (white backgrounds just didn't stand out enough).

This will finally provide an authoritative source for the most trivial
rack use cases.

I also changed the site to a basic two-column layout, with the
incredibly-long News section on the left and everything else on the
right. This makes it much easier to scroll down and see what information
is available, as opposed to getting lost in the News and Releases lists.

The CSS has been tested in Chrome and Safari.
@leahneukirchen
Copy link
Member

Max Cantor [email protected] writes:

You can see what it looks like here.

I like it. Perhaps we could make the side a little narrower, so that it
fits 960px.

Christian Neukirchen [email protected] http://chneukirchen.org

@changemewtf
Copy link
Contributor Author

Glad you like it!

Any particular reason 960px is the target? The site live on rack.github.io now has a 976px-wide content box that's 1026px-wide total counting padding & borders, whereas the one in my pull request is actually thinner at 980px content and 1010px counting padding & borders.

Wouldn't take much to change it to 960px counting padding & borders if that's what you're thinking; just curious as to the motivation.

@leahneukirchen
Copy link
Member

Max Cantor [email protected] writes:

Sure. Any particular reason 960px is the target? The site live on
rack.github.io now has a 976px-wide content box that's 1026px-wide
total counting padding & borders, whereas the one in my pull request
is actually thinner at 980px content and 1010px-wide counting padding
& borders.

Wouldn't take much to change it to 960px counting padding & borders if
that's what you're thinking; just curious as to the motivation.

Easy, my browser is 980px wide. ;)

But the page on rack.github.io displays without scrollbar here, even if
I make it narrower.

Christian Neukirchen [email protected] http://chneukirchen.org

@changemewtf
Copy link
Contributor Author

Ooh. I see what you mean. I've unintentionally removed the responsiveness of the page layout. I'll see if I can bring that back.

@changemewtf
Copy link
Contributor Author

OK. Should be better now. The only potential problem is that the code snippets won't word-wrap because they're in a <pre> tag, but I think it's fine to have a horizontal scrollbar on them if the browser window is that small (I had to go all the way down to 875px wide for the text to be occluded).

leahneukirchen added a commit that referenced this pull request May 1, 2014
Add a 5-second tutorial to the top of the site, and reorganize some of the main info.
@leahneukirchen leahneukirchen merged commit c3399ad into rack:master May 1, 2014
@leahneukirchen
Copy link
Member

Thanks!

@changemewtf
Copy link
Contributor Author

Awesome! Thanks for such a quick followup.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants