Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add .read class to improve readability #4

Closed
wants to merge 1 commit into from

Conversation

gizak
Copy link

@gizak gizak commented Mar 25, 2014

Enhance readability;

  1. overwrite the post background colour to white
  2. the title font: museo-slab
  3. the content font: museo-sans
  4. adjust font size

@aigarsdz
Copy link
Owner

Hello! I appreciate your effort, but the problem with Museo font is it's narrow language support. I took a screenshot to illustrate, what I'm talking about (open it in a new tab to see more clearly). The second paragraph is written in a different language, and letters with diacritical signs are rendered using the fallback font.

I want brume to support other languages in addition to English.

I'm not an expert in readability, but can we improve it using the current font and background colour?

@gizak
Copy link
Author

gizak commented Mar 30, 2014

Hi aigarsdz!

Good tips! I didn't consider about multi languages.

I am not a UI expert either, but intuitively think it may be better to make the font size and line height a litter higher :)

@aigarsdz
Copy link
Owner

You're probably right. I'll try to make improvements and will see, how that looks.

aigarsdz pushed a commit that referenced this pull request Apr 6, 2014
@aigarsdz aigarsdz closed this Jun 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants