brownplt / ZZZ-captain-teach

Set sail for Learning Island! Deprecated and no longer maintained
5 stars 1 forks source link

Some small style tweaks. #5

Open sbirch opened 11 years ago

sbirch commented 11 years ago

This is a small visual tweak to the side-text.

screen shot 2013-09-07 at 5 46 53 pm screen shot 2013-09-07 at 5 16 44 pm

jpolitz commented 11 years ago

Opinions @dbp , @shriram ?

The lighter border on the inline-examples seems like the nicest thing. I don't know if I care about the Georgia; and this commit doesn't make the Georgia change globally (e.g. on the course listing page), but the inline examples do look much nicer.

The white background I could go either way on.

dbp commented 11 years ago

I like the border and the background color. Don't care about the font.

sbirch commented 11 years ago

Thanks, Joe.

re: the border, it's not so much the color as distinction between the text and the rest of the page -- making the column more obvious.

Sounds like Georgia is out :).

Another thing which I played with but couldn't figure out was to change the code font from "monospace" to one in particular. Courier and Courier New -- the default monospaces for most readers -- don't do a very good job at distinguishing things like L and 1 (which actually lead to some confusion for me on this assignment.) Others, such as the Adobe source family or Consolas explicitly account for this.