PPS about draft #6990

vv-monsalve · 2023-11-20T23:59:16Z

@RosaWagner This PR adds a drafted Article for Playpen Sans to test how the contents would look.

github-actions · 2023-11-21T00:02:06Z

FontBakery report

fontbakery version: 0.10.4

[1] Experimental checks

🔥 FAIL: Shapes languages in all GF glyphsets. (com.google.fonts/check/glyphsets/shape_languages)

🔥 FAIL GF_Latin_Core glyphset:

Language	FAIL messages
fo_Latn (Faroese)	Shaper didn't attach uni0335.case to E
is_Latn (Icelandic)	Shaper didn't attach uni0335.case to E
teo_Latn (Teso)	Some base glyphs were missing: Ɔ, Ɛ, Ɨ, Ʉ, ɔ, ɛ, ɨ, ʉ, ᵃ, ᵉ, ᵋ, ᵒ, ᵓ, ᵘ, ᶤ, ᶶ, ⁱ
^	Shaper produced a .notdef

[code: failed-language-shaping]

⚠ WARN GF_Latin_Core glyphset:

Language	FAIL messages
lg_Latn (Ganda)	No variant glyphs were found for Eng
dyo_Latn (Jola-Fonyi)	No variant glyphs were found for Eng
ny_Latn (Nyanja)	No variant glyphs were found for Eng
wo_Latn (Wolof)	No variant glyphs were found for Eng

[code: warning-language-shaping]

[1] Family checks

ℹ INFO: Check axis ordering on the STAT table. (com.google.fonts/check/STAT/axis_order)

ℹ INFO From a total of 1 font files, 0 of them (0.00%) lack a STAT table.

And these are the most common STAT axis orderings:
('wght', 1) [code: summary]

[12] PlaypenSans[wght].ttf

⚠ WARN: Check for codepoints not covered by METADATA subsets. (com.google.fonts/check/metadata/unreachable_subsetting)

⚠ WARN The following codepoints supported by the font are not covered by
any subsets defined in the font's metadata file, and will never
be served. You can solve this by either manually adding additional
subset declarations to METADATA.pb, or by editing the glyphset
definitions.
U+02C7 CARON: try adding one of: canadian-aboriginal, tifinagh, yi
U+02C9 MODIFIER LETTER MACRON: not included in any glyphset definition
U+02D8 BREVE: try adding one of: canadian-aboriginal, yi
U+02D9 DOT ABOVE: try adding one of: canadian-aboriginal, yi
U+02DB OGONEK: try adding one of: canadian-aboriginal, yi
U+02DD DOUBLE ACUTE ACCENT: not included in any glyphset definition
U+0306 COMBINING BREVE: try adding one of: tifinagh, old-permic
U+030A COMBINING RING ABOVE: try adding syriac
U+030B COMBINING DOUBLE ACUTE ACCENT: try adding one of: osage, cherokee
U+030C COMBINING CARON: try adding one of: tai-le, cherokee
15 more.

Use -F or --full-lists to disable shortening of long lists.

Or you can add the above codepoints to one of the subsets supported by the font: latin, latin-ext, math, menu, symbols, vietnamese [code: unreachable-subsetting]

⚠ WARN: Check if category on METADATA.pb matches what can be inferred from the family name. (com.google.fonts/check/metadata/category_hints)

⚠ WARN Familyname seems to hint at "SANS_SERIF" but METADATA.pb declares it as "['HANDWRITING']". [code: inferred-category]

⚠ WARN: Does the font contain a soft hyphen? (com.google.fonts/check/soft_hyphen)

⚠ WARN This font has a 'Soft Hyphen' character. [code: softhyphen]

⚠ WARN: Ensure soft_dotted characters lose their dot when combined with marks that replace the dot. (com.google.fonts/check/soft_dotted)

⚠ WARN The dot of soft dotted characters should disappear in other cases, for example: į̛̀ į̛́ į̛̂ į̛̃ į̛̄ į̛̆ į̛̇ į̛̈ į̛̉ į̛̊ į̛̋ į̛̌ į̛̒ į̣̀ į̣́ į̣̂ į̣̃ į̣̄ į̣̆ į̣̇

Your font fully covers the following languages that require the soft-dotted feature: Ebira (Latn, 2,200,000 speakers), Igbo (Latn, 27,823,640 speakers), Lithuanian (Latn, 2,357,094 speakers), Navajo (Latn, 166,319 speakers), Dutch (Latn, 31,709,104 speakers), Ma’di (Latn, 584,000 speakers).

Your font does not cover the following languages that require the soft-dotted feature: Basaa (Latn, 332,940 speakers), Ukrainian (Cyrl, 29,273,587 speakers), Lugbara (Latn, 2,200,000 speakers), Ejagham (Latn, 120,000 speakers), Dan (Latn, 1,099,244 speakers), Aghem (Latn, 38,843 speakers), Koonzime (Latn, 40,000 speakers), Nateni (Latn, 100,000 speakers), Kom (Latn, 360,685 speakers), Belarusian (Cyrl, 10,064,517 speakers), Avokaya (Latn, 100,000 speakers). [code: soft-dotted]

ℹ INFO: Show hinting filesize impact. (com.google.fonts/check/hinting_impact)

ℹ INFO Hinting filesize impact:

	ofl/playpensans/PlaypenSans[wght].ttf
Dehinted Size	508.2kb
Hinted Size	508.3kb
Increase	24 bytes
Change	0.0 %
[code: size-impact]

ℹ INFO: Font has old ttfautohint applied? (com.google.fonts/check/old_ttfautohint)

ℹ INFO Could not detect which version of ttfautohint was used in this font. It is typically specified as a comment in the font version entries of the 'name' table. Such font version strings are currently: ['Version 1.001'] [code: version-not-detected]

ℹ INFO: EPAR table present in font? (com.google.fonts/check/epar)

ℹ INFO EPAR table not present in font. To learn more see When missing EPAR table, just emit INFO alert fonttools/fontbakery#818 [code: lacks-EPAR]

ℹ INFO: Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? (com.google.fonts/check/gasp)

ℹ INFO These are the ppm ranges declared on the gasp table:

PPM <= 65535:
flag = 0x0F
- Use grid-fitting
- Use grayscale rendering
- Use gridfitting with ClearType symmetric smoothing
- Use smoothing along multiple axes with ClearType®
[code: ranges]

ℹ INFO: Check for font-v versioning. (com.google.fonts/check/fontv)

ℹ INFO Version string is: "Version 1.001"
The version string must ideally include a git commit hash and either a "dev" or a "release" suffix such as in the example below:
"Version 1.3; git-0d08353-release" [code: bad-format]

ℹ INFO: Ensure fonts have ScriptLangTags declared on the 'meta' table. (com.google.fonts/check/meta/script_lang_tags)

ℹ INFO Latn [code: dlng-tag]
ℹ INFO Latn [code: slng-tag]

ℹ INFO: Font contains all required tables? (com.google.fonts/check/required_tables)

ℹ INFO This font contains the following optional tables:
- loca
- prep
- GPOS
- GSUB
- gasp [code: optional-tables]

ℹ INFO: List all superfamily filepaths (com.google.fonts/check/superfamily/list)

ℹ INFO ofl/playpensans [code: family-path]

Summary

💔 ERROR	🔥 FAIL	⚠ WARN	💤 SKIP	ℹ INFO	🍞 PASS	🔎 DEBUG
0	1	4	52	9	189	0
0%	0%	2%	20%	4%	74%	0%

Note: The following loglevels were omitted in this report:

SKIP
PASS
DEBUG

vv-monsalve · 2023-11-21T04:28:58Z

The changes are still not visible. I'm testing the inclusion of a table with multiple small images (the emojis) that shows to users which are the emojis included in the font, and to do it, I've tried to add a "head" with some styles. I'm wondering if this is even possible or if that is what is causing it not to work.

@nathan-williams could you please confirm about the above or if the "style" property should be added to each tag?

Additionally, to avoid the multiple images of the emojis in the font, would it be a way to link the Playpen Sans font itself so that we can use text? This would be ideal :)

RosaWagner · 2023-11-21T11:36:06Z

ofl/playpensans/article/ARTICLE.en_us.html

+}
+</style>
+</head>
+<body>


I think everything above that line should be removed, may come in conflict with the website.

Yeah, was hesitant, but was worth trying it.

RosaWagner · 2023-11-21T11:36:54Z

ofl/playpensans/article/ARTICLE.en_us.html

+Playpen Sans is the extension of technological and aesthetic potential, showing the best of both worlds with its digital capabilities and casual, handmade look. Is it spontaneous? Is it authentic? Thankfully, yes, and yes.
+</p>
+<p>
+To contribute, see <a href="https://github.com/TypeTogether/Playpen-Sans.git" target="_blank">github.com/TypeTogether/Playpen-Sans</a>


also target_blank should be removed

RosaWagner · 2023-11-21T11:38:49Z

ofl/playpensans/article/ARTICLE.en_us.html

+        <td style="font-size: 30px;">👆</td>
+        <td><img src="e-index-pointing.png"></td>
+    </tr>
+</table>


the table, tr and td elements are not listed in the allowed elements for the article

AFAIR Chris confirmed with Nate that tables were also supported.

vv-monsalve added 4 commits November 20, 2023 18:49

Article added to PPS

dac588d

PPS blank description

7ac73b5

imagesize-2 added to test

ccd3958

emoji size 2 test added

9d2d3c4

vv-monsalve added the I Tools / workflow / repo label Nov 21, 2023

vv-monsalve merged commit e3842c9 into main Nov 21, 2023
9 checks passed

vv-monsalve deleted the PPS-about-draft branch November 21, 2023 00:03

RosaWagner reviewed Nov 21, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

PPS about draft #6990

PPS about draft #6990

vv-monsalve commented Nov 20, 2023 •

edited

Loading

github-actions bot commented Nov 21, 2023

vv-monsalve commented Nov 21, 2023 •

edited

Loading

RosaWagner Nov 21, 2023

vv-monsalve Nov 21, 2023

RosaWagner Nov 21, 2023

RosaWagner Nov 21, 2023

vv-monsalve Nov 21, 2023

PPS about draft #6990

PPS about draft #6990

Conversation

vv-monsalve commented Nov 20, 2023 • edited Loading

github-actions bot commented Nov 21, 2023

FontBakery report

Summary

vv-monsalve commented Nov 21, 2023 • edited Loading

RosaWagner Nov 21, 2023

Choose a reason for hiding this comment

vv-monsalve Nov 21, 2023

Choose a reason for hiding this comment

RosaWagner Nov 21, 2023

Choose a reason for hiding this comment

RosaWagner Nov 21, 2023

Choose a reason for hiding this comment

vv-monsalve Nov 21, 2023

Choose a reason for hiding this comment

vv-monsalve commented Nov 20, 2023 •

edited

Loading

vv-monsalve commented Nov 21, 2023 •

edited

Loading