Use of Georgia for numerals is not good - there are many issues here, easily fixed by not using Georgia for the pager - it looks very odd.
bartik-pager-whacked.png

CommentFileSizeAuthor
#4 bartik-pager-font.patch634 bytesJeff Burnz
#4 bartik-pager-font.png36.53 KBJeff Burnz
bartik-pager-whacked.png14.16 KBJeff Burnz

Comments

Bojhan’s picture

Bojhan commented

Can't we align there buts?

Jeff Burnz’s picture

Jeff Burnz commented

Technically they already are, Georgia is a non-lining, old style font which use text figures: http://en.wikipedia.org/wiki/Text_figures, which is real good when you want numerals to stand out, say within a paragraph of text or as a headline, but in something stand alone like the pager it looks rather odd.

aspilicious’s picture

aspilicious commented

we should choose another font for the pager, this doesn't look good.

Jeff Burnz’s picture

Jeff Burnz commented
Status: Active » Needs review
StatusFileSize
new bartik-pager-font.png36.53 KB
new bartik-pager-font.patch634 bytes

Set to font-family: "Helvetica Neue", Helvetica, Arial, sans-serif; and 90%.

aspilicious’s picture

aspilicious commented

Let Jensimmons deside or Bojhan or ... someone else if they don't respond.
For me this is good to go.

bleen’s picture

bleen commented

I'm fine with this too ...

screenshot from #4:

Jeff Burnz’s picture

Jeff Burnz commented

#4: bartik-pager-font.patch queued for re-testing.

Bojhan’s picture

Bojhan commented
Status: Needs review » Reviewed & tested by the community

I am not sure I like it in terms of typography, but indeed the current is hard to scan = unusable.

yoroy’s picture

yoroy commented

This uses the same typefaces as defined for the main menu tabs, so it's consistent with that. RTBC it is.
(I did scan Bartiks style.css and there are a lot of slightly different font-size statements in there, 86%, 90%, 92%… which may be fine but the overall size of the text seems too small to me. other issue, I know :-)

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.