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

Font issues with non [a-zA-Z] characters #39

Closed
zertrin opened this issue Feb 20, 2013 · 11 comments
Closed

Font issues with non [a-zA-Z] characters #39

zertrin opened this issue Feb 20, 2013 · 11 comments

Comments

@zertrin
Copy link

zertrin commented Feb 20, 2013

As can be seen on the screenshot below, selfoss has some problems rendering diacritics or "special" characters in sources' titles.

For example : çéèêüß cause the problem... (I haven't tested all of the possibilities, but this seems rather generic about non [a-zA-Z] characters.
The font of the word containing such a character falls back to the default one of the browser, beginning with the faulty character and gets back to selfoss normal font with the beginning of the next word, until another problematic character appear.

Moreover the apostrophe sign is rendered with its escaped HTML equivalent: ’

selfoss

Viele Grüße aus München

@zertrin
Copy link
Author

zertrin commented Feb 22, 2013

I'd like to add that this misbehavior is also observed in the content of the articles too:

Capture

@SSilence
Copy link
Member

Which browser do you use? In chrome I don't have the issue with the font.

Does the problem with the ’ only occur in tumblr? It seems they send it with escaped & (’).

Regards
Tobi

@zertrin
Copy link
Author

zertrin commented Feb 24, 2013

I'm using Opera 12.14 (latest version).

You're right, the ' problem occurs only with tumblr feeds. However I just noticed that it doesn't happen for each and every '

@xavez
Copy link

xavez commented Mar 21, 2013

Is it a good idea to have a custom font for the content anyway? Wouldn’t it be a better idea to use either Georgia or Helvetica for a feed reader? They contain all the necessary characters, look great on any screen and don’t interfere with the reading experience.

In most cases I would be inclined to say that using a custom font is a great idea, but in a web RSS reader I would really try to keep the typography as “native” as possible.

I did some tweaks to the CSS on my installation and I much prefer this reading experience:

Schermafbeelding 2013-03-21 om 10 52 16

@zertrin
Copy link
Author

zertrin commented Mar 21, 2013

I agree with @xavez, especially when the "new" Web font techniques seem not yet perfectly integrated in every common browser.

At least, I'd like to see what you did tweak to get to your reading experience.

@xavez
Copy link

xavez commented Mar 21, 2013

@zertrin not a whole lot, but it does the trick for me:

  • Removed the 3-columns layout, because reading in columns on a screen is annoying: scrolling up and down to read columns of text is definitely no typography best practice (and the web is not a print magazine :))
  • Removed the striped background pattern (noisy)
  • Changed the font-face to Georgia (personal preference, Helvetica would do great as well)
  • Changed the text color to a light shade of black for easy-on-the-eyes contrast (#404040)
  • Changed the font-size to be big enough (125%)
  • Added a max-width to the .entry-content so it is a pleasurable reding length (38em)

I think the interface can use some love in general but hey, let’s start with the basics :)

@SSilence
Copy link
Member

Does the problem still exists with the current master version? The fonts was updated in one of the last commits.

@zertrin
Copy link
Author

zertrin commented Mar 29, 2013

I just checked out the latest github version :

commit 4c2ee6728d55c286bea3786c6786c3b9d72de03d
Author: Tobias Zeising <tobi....de>
Date:   Fri Mar 29 13:32:05 2013 +0100

The problem changed somewhat but is still not really solved... see screenshot below :

11fezqgregre

EDIT: Still using latest version of "stable" Opera (12.14). Note that this does not occur in Google Chrome...

@SSilence SSilence reopened this Apr 1, 2013
@SSilence
Copy link
Member

SSilence commented Apr 1, 2013

I will add a new option for using the system sans font.

@armandabric
Copy link

I agree with the @xavez suggestions. This enchantment can increase a lot the UI and the readability of the content.

@SSilence
Copy link
Member

SSilence commented Apr 2, 2013

I have added a new parameter use_system_font. Setting this to 1 will remove the fonts.css and your default font (Arial) will be used.

@SSilence SSilence closed this as completed Apr 2, 2013
jtojnar added a commit that referenced this issue Feb 16, 2023
It would replace apostrophes with `&#39;` breaking the content equality assertion in the next commit.
jtojnar added a commit that referenced this issue Feb 16, 2023
It would replace apostrophes with `&#39;` breaking the content equality assertion in the next commit.
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

No branches or pull requests

4 participants