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

latin: Add U+2122 (™) #78

Open
criana opened this issue Jul 14, 2015 · 28 comments

Comments

@criana
Copy link

@criana criana commented Jul 14, 2015

I recently discovered the Open Sans library via Google Fonts is missing the trademark symbol. It's currently an issue with a website I'm working on, and was able to verify it with the font preview tool on Google Fonts.

Although Adobe's version of the library has the TM. Any chance of getting the Google hosted library fixed?

opensans_trademarkcharactermissing

@enathu

This comment has been minimized.

Copy link

@enathu enathu commented Jul 14, 2015

I have checked this issue with a CodePen, and the Trademark (™) sign is showing as intended. You can have a look at the CodePen here

However, I can also confirm that that the Google Font preview tool, doesn't show the ™ mark in any font, as I've tried with it. May be it's an issue with preview tool, not with the font. Hope this get fixed.

gf-tm

Cheers.

@criana

This comment has been minimized.

Copy link
Author

@criana criana commented Jul 14, 2015

@enathu thanks for sharing, that's very interesting. I wonder why on our site pulling the Open Sans font gives us the TM in a box representation.
opensans_trademarkincopy

@criana

This comment has been minimized.

Copy link
Author

@criana criana commented Jul 14, 2015

@enathu I just made a slight edit to the CodePen and was able to reproduce the issue. Remove "sans-serif" from after Open Sans and the TM becomes a [TM] in a box, broken character. I believe the library may still be missing this item and when "sans-serif" is supplied it defaults to the system sans-serif font which does contain that character.

Does that make sense?

@enathu

This comment has been minimized.

Copy link

@enathu enathu commented Jul 14, 2015

@criana Alright, that makes sense, now. So, I think it's an issue with GF API, since it's showing the same cross (missing glyphs) for all other fonts for trademark sign. Even though the trademark sign is inside the font file, as I just checked with the GF hosted Open Sans files.

Pinging @davelab6 for fix. :)

@davelab6

This comment has been minimized.

Copy link
Member

@davelab6 davelab6 commented Jul 17, 2015

The directory's preview isn't an accurate reflection of the characters available in the font. The sans-serif thing is interesting, I'm not sure what is happening there... my guess is that when you add sans-serif then the browser uses the ™ that is in the default sans-serif font (likely Arial on windows) but when sans-serif is not there, is uses a default font (not sure what) which for some reason has this character in a box like that.

I'm using a mac right now, and the ™ in the codepen does look slightly different with/without the sans-serif, but there is no box shape.

The unicode U+2122 is not in the latin subsets, though, and thus isn't being served... so it could be a candidate for inclusion in the latin-ext subset.

https://github.com/google/fonts/blob/master/tools/encodings/latin_unique-glyphs.nam

https://github.com/google/fonts/blob/master/tools/encodings/latin-ext_unique-glyphs.nam

@davelab6 davelab6 changed the title Open Sans Library Missing Trademark Add U+2122 (™) to latin-ext Jul 17, 2015
@davelab6 davelab6 self-assigned this Jul 17, 2015
@codeman38

This comment has been minimized.

Copy link

@codeman38 codeman38 commented Jul 17, 2015

U+2122 is part of the legacy Latin character sets for both Windows and Mac OS, so it feels like it should possibly even be included in latin.

>>> u'\u2122'.encode('cp1252')
'\x99'
>>> u'\u2122'.encode('macroman')
'\xaa'
@criana

This comment has been minimized.

Copy link
Author

@criana criana commented Jul 17, 2015

@davelab6,
Thanks for looking into this more. I've attached a screen shot of what I see in CodePen. I'm running Chrome on a Mac with OS 10.10.4. I also do no have any versions of OpenSans running locally on my machine. This is only when sans-serif is removed from the font-declaration.

screen shot 2015-07-17 at 6 34 55 pm

@davelab6

This comment has been minimized.

Copy link
Member

@davelab6 davelab6 commented Jul 17, 2015

Here's what I see in Chrome 46, also 10.10.4, also none installed locally...

screen shot 2015-07-17 at 18 48 19

@criana

This comment has been minimized.

Copy link
Author

@criana criana commented Jul 17, 2015

I was able to verify this too after a thorough cleaning of fonts and caches on my machine. Sorry for the mixup.

@davelab6

This comment has been minimized.

Copy link
Member

@davelab6 davelab6 commented Jul 20, 2015

No worries @criana - the suggestion to add ™ is a good one :)

@davelab6 davelab6 added the subset label Jul 20, 2015
@BenRacicot

This comment has been minimized.

Copy link

@BenRacicot BenRacicot commented Oct 9, 2015

Hey @davelab6 any updates on this? I'm having the exact same issue as @criana perhaps someone has a nicer fix than adding a fallback font?

@qJake

This comment has been minimized.

Copy link

@qJake qJake commented Apr 8, 2016

This is still an issue - Open Sans does not include the ™ character in any subset, and if a fallback font is not specified via CSS, it defaults (on most PCs) to a serif font, which looks ugly:

image

The ™ character should be included in the latin-ext subset. 😄 👍 ™️

@davelab6

This comment has been minimized.

Copy link
Member

@davelab6 davelab6 commented Apr 8, 2016

Re-acknowledging this issue :) I'll see what I can do!

@davelab6 davelab6 modified the milestone: Glyph Sets Jul 17, 2016
@davelab6 davelab6 removed the subset label Jul 17, 2016
@danimalweb

This comment has been minimized.

Copy link

@danimalweb danimalweb commented Sep 30, 2016

Any update on this? Thanks :)

@m4rc1e

This comment has been minimized.

Copy link
Collaborator

@m4rc1e m4rc1e commented Jun 8, 2017

Made up a quick Jsbin:

LGTM http://jsbin.com/perivejuqa/edit?html,output

@m4rc1e m4rc1e closed this Jun 19, 2017
@qJake

This comment has been minimized.

Copy link

@qJake qJake commented Jun 19, 2017

@m4rc1e In #785 you said:

#78: Works just fine for me, added comment, skipping

It still does not render for a lot of us. Your JSbin renders for me (incorrectly) like this:

image

Can you try testing on a fresh VM with Windows 10 + Chrome 59? On a VM here with that environment I get this (same behavior):

image

I'd like to vote to reopen unless there is concrete proof that this isn't an issue, or the decision has been made for the ™ character to not be included in any subsets of Open Sans on Google Fonts.

@m4rc1e m4rc1e reopened this Jun 19, 2017
@m4rc1e

This comment has been minimized.

Copy link
Collaborator

@m4rc1e m4rc1e commented Jun 19, 2017

Thanks @qJake ,

Here's my screenshot on Win 10, Chrome 59. You're correct

screen shot 2017-06-19 at 15 15 37

However, the glyph exists in the fonts:
screen shot 2017-06-19 at 15 14 29

It gets served properly on OS X. I will look into this more.

@davelab6

This comment has been minimized.

Copy link
Member

@davelab6 davelab6 commented Jul 21, 2017

@m4rc1e the reason is that 0x2122 is not included in the latin-ext.nam file used by the GF API to generate subsets of fonts, so even if it is in a font file (and it will be in more font files, since it is part of the GF Latin Pro set) currently it won't be served.

@roozbehp

This comment has been minimized.

Copy link

@roozbehp roozbehp commented Nov 22, 2017

Working on adding ™ to latin itself.

@stuartpb

This comment has been minimized.

Copy link

@stuartpb stuartpb commented Nov 22, 2017

Why latin over latin-ext? Is this following the reasoning from this comment that common legacy native Latin character sets include the character?

@davelab6 davelab6 changed the title latin-ext: Add U+2122 (™) latin: Add U+2122 (™) Nov 23, 2017
@roozbehp

This comment has been minimized.

Copy link

@roozbehp roozbehp commented Nov 23, 2017

Generally, normal text in English and Western European languages are expected to just need 'latin'. It's relatively unreasonable for such a common character to need to pull in the whole latin-ext subset, which could be pretty large.

@davelab6 davelab6 mentioned this issue Nov 23, 2017
2 of 3 tasks complete
@davelab6

This comment has been minimized.

Copy link
Member

@davelab6 davelab6 commented Nov 23, 2017

Thank you @roozbehp :D

@davelab6 davelab6 closed this Nov 23, 2017
@stuartpb

This comment has been minimized.

Copy link

@stuartpb stuartpb commented Nov 23, 2017

Shouldn't this issue stay open until the change is integrated?

@davelab6

This comment has been minimized.

Copy link
Member

@davelab6 davelab6 commented Nov 25, 2017

Ok, yes, reopening until your is plumbed all the way through to production :)

@davelab6

This comment has been minimized.

Copy link
Member

@davelab6 davelab6 commented Mar 22, 2018

Coming soon...

@VeeDoubleYuh

This comment has been minimized.

Copy link

@VeeDoubleYuh VeeDoubleYuh commented Jan 16, 2019

Curious if there are any updates on this. I see it's been quite a while since there has been a post.

I haven't read every post here, but if anyone needs to pull in a "pretty" TM for their site and they don't mind an extra request, you can do what I'm doing right now. I've placed this line at the end of my HTML's <body>.
<link href="https://fonts.googleapis.com/css?family=Cairo:300,400,600,700&text=™" rel="stylesheet">
The "text=™" will pull only that glyph rather than the whole font according to this.

Then, you just make the above font (Choose whatever font you wish, of course. I just liked the TM in this one.) the second in your font stack in CSS. Not a perfect fix, but it's a decent workaround until there is a native TM in this font.

You can also put it in the <head>, but I felt that it was a small enough enhancement that it wasn't something that should slow everything else down.

@davelab6

This comment has been minimized.

Copy link
Member

@davelab6 davelab6 commented Jan 16, 2019

@code23-isaac

This comment has been minimized.

Copy link

@code23-isaac code23-isaac commented Jan 21, 2019

Coming soon...

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.