Skip to content
This repository has been archived by the owner on Nov 18, 2017. It is now read-only.

Comment width #49

Closed
jferris opened this issue May 30, 2014 · 1 comment
Closed

Comment width #49

jferris opened this issue May 30, 2014 · 1 comment

Comments

@jferris
Copy link
Contributor

jferris commented May 30, 2014

At the current size, comments wrap after only 2-4 words, which makes them really hard to read. They occupy the same amount of space regardless of the width of my browser.

tiny_comment

cpytel added a commit that referenced this issue Jun 6, 2014
@cpytel
Copy link
Member

cpytel commented Jun 8, 2014

I've made a number of changes to the style of the comments to make them a little wider and to give us more room within the limited space we have. See a screenshot of the same comment to see how it looks now.

One of the changes @plapier suggested was the use a Sans Serif font which will be more readable at smaller font sizes. We have an interesting problem with this in that Carnival currently doesn't serve any fonts, but generally takes the approach of providing basic styles that will work well. As such we haven't really needed to change the blog styles to support Carnival at all. However, Carnival can't really provide an alternate, non browser supported font.

What I've done for now is just used font-family: sans-serif.

I think this is a reasonable default for Carnival to provide, and on OS X, that results in Helvetica. If further font customization is going to be done, I think it should be on the blog scss files that it happens.

These changes are in master and deployed to staging.

@cpytel cpytel closed this as completed Jun 8, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants