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.

Sign up for GitHub

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

Jump to bottom

Update Playfair Display & Playfair Display SC #7

Closed
avramchuk opened this issue Apr 13, 2015 · 9 comments
Closed

Update Playfair Display & Playfair Display SC #7

avramchuk opened this issue Apr 13, 2015 · 9 comments
Assignees
@davelab6

Comments

@avramchuk
Copy link

A cyrillic character «Ы» is wrong in the Playfair Display and Playfair Display SC.

https://www.google.com/fonts/specimen/Playfair+Display
https://www.google.com/fonts/specimen/Playfair+Display+SC

Fixed version is here:
https://github.com/clauseggers/Playfair-Display
@avramchuk avramchuk changed the title Fix Playfair Display & Playfair Display SC Update Playfair Display & Playfair Display SC Apr 13, 2015
@davelab6 davelab6 self-assigned this Apr 13, 2015
@u3k
Copy link

u3k commented Jun 16, 2015

I do confirm that glyphs are still switched and instead of capital "Ы" it has something like "IЬ"

@ladytellur
Copy link

The problem exists.

It should be noted that https://github.com/clauseggers/Playfair-Display font as a solution will not be exactly_the_same google Playfair Display font with the letter "Ы".

clauseggers/Playfair-Display font thinner and lower with the same parameters, as used for google Playfair Display font variant.

@davelab6 davelab6 self-assigned this Jan 20, 2016
@davelab6 davelab6 modified the milestone: Update Families Jul 16, 2016
@omerziv
Copy link

omerziv commented Jul 27, 2016

Hey @davelab6 - Please update to the latest release to solve this issue.

@piepaga
Copy link

piepaga commented Aug 10, 2016

Hi,
is it possible that not all the font files have been fixed?
With Chrome (which loads the file https://fonts.gstatic.com/s/playfairdisplay/v10/2NBgzUtEeyB-Xtpr9bm1CRw5vVFbIG7DatP53f3SWfE.woff2) I see again the cyrillic character "bI" wrong: capital is still something like "IЬ".

@aditrif
Copy link

aditrif commented Sep 29, 2016

Hello, please update this font family to display correctly the Romanian lowercase characters "ș" and "ț", right now they are distorted. Apparently the correct glyphs are being used on the creator's page here: http://www.forthehearts.net/typeface-design/playfair-display/

@birdkiwi
Copy link

birdkiwi commented Jun 9, 2017

Problem still exists

@m4rc1e
Copy link
Collaborator

m4rc1e commented Jun 14, 2017

Problem still exists in Playfair Display:

screen shot 2017-06-14 at 11 58 01

http://jsbin.com/kureverako/edit?html,output

It has been solved in Playfair Display SC:

screen shot 2017-06-14 at 11 59 55

http://jsbin.com/rivoyepesa/edit?html,output

@m4rc1e m4rc1e mentioned this issue Jul 10, 2017
Closed
@davelab6
Copy link
Member

@m4rc1e this is urgent, please hotfix only

m4rc1e added a commit to m4rc1e/playfair-v1.003-hotfix that referenced this issue Aug 22, 2017
@m4rc1e
Fix uni042b glyph in Roman weights.
b878fd6 
This solves issue google/fonts#7.

There is a newer release of Playfair but the GF version and upstream version have different proportions. @davelab6 suggested a quick hotfix since this issue is urgent.;
m4rc1e referenced this issue in m4rc1e/fonts Aug 22, 2017
@m4rc1e
playfairdisplay: v1.003 added.
4a3d703 
Fixes issue #7.

Temporary hotfix repository created, https://github.com/m4rc1e/playfair-v1.003-hotfix
@m4rc1e m4rc1e mentioned this issue Aug 22, 2017
Merged
davelab6 pushed a commit that referenced this issue Aug 22, 2017
@m4rc1e @davelab6
playfairdisplay: v1.003 added. (#1160)
a6909e3 
Fixes issue #7.

Temporary hotfix repository created, https://github.com/m4rc1e/playfair-v1.003-hotfix
@davelab6 davelab6 mentioned this issue Aug 28, 2017
Closed
@davelab6
Copy link
Member

Fixed

screen shot 2017-08-29 at 18 56 21
screen shot 2017-08-29 at 18 56 16

@AndisGrossteins AndisGrossteins mentioned this issue Sep 3, 2017
Open
@eliheuer eliheuer mentioned this issue Oct 17, 2018
Merged
@eliheuer eliheuer mentioned this issue Oct 25, 2018
Closed
This was referenced Jun 17, 2021
Merged
Merged
Merged
Merged
Merged
nathan-williams pushed a commit that referenced this issue Apr 16, 2022
@felipesanches
add code test to avoid spaces in fallback names
997e558 
(issue #7)

Also updated the CHANGELOG.md file.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@davelab6 davelab6

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
9 participants
@u3k @aditrif @davelab6 @omerziv @birdkiwi @avramchuk @ladytellur @m4rc1e @piepaga