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

Buenard: Version 2.000 added #7573

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

simoncozens
Copy link
Collaborator

@simoncozens simoncozens added III VF Replacement Replace an existing family of static fonts with variable fonts I Font Upgrade labels Apr 18, 2024
@simoncozens simoncozens linked an issue Apr 18, 2024 that may be closed by this pull request
Copy link

FontBakery report

fontbakery version: 0.12.2

Experimental checks

These won't break the CI job for now, but will become effective after some time if nobody raises any concern.

[2] Buenard[wght].ttf
⚠️ WARN Validate location, size and resolution of article images.
  • ⚠️ WARN

    Family metadata at ofl/buenard does not have an article.


    [code: lacks-article]
ℹ️ INFO METADATA.pb: Validate family.minisite_url field.
  • ℹ️ INFO

    Please consider adding a family.minisite_url entry.


    [code: lacks-minisite-url]

All other checks

[16] Buenard[wght].ttf
⚠️ WARN Does GPOS table have kerning information? This check skips monospaced fonts as defined by post.isFixedPitch value
  • ⚠️ WARN

    GPOS table lacks kerning information.


    [code: lacks-kern-info]
⚠️ WARN Check math signs have the same width.
  • ⚠️ WARN

    The most common width is 460 among a set of 6 math glyphs.
    The following math glyphs have a different width, though:

Width = 469: greater, less

Width = 567: logicalnot

[code: width-outliers]
⚠️ WARN Check font contains no unreachable glyphs
  • ⚠️ WARN

    The following glyphs could not be reached by codepoint or substitution rules:

- CR

[code: unreachable-glyphs]

⚠️ WARN Check for codepoints not covered by METADATA subsets.
  • ⚠️ 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, yi, tifinagh
  • U+02D8 BREVE: try adding one of: canadian-aboriginal, yi
  • U+02D9 DOT ABOVE: try adding one of: canadian-aboriginal, yi
  • U+02DD DOUBLE ACUTE ACCENT: not included in any glyphset definition
  • U+03BC GREEK SMALL LETTER MU: try adding one of: greek, math

Or you can add the above codepoints to one of the subsets supported by the font: latin, latin-ext, menu

[code: unreachable-subsetting]
⚠️ WARN Ensure fonts have ScriptLangTags declared on the 'meta' table.
  • ⚠️ WARN

    This font file does not have a 'meta' table.


    [code: lacks-meta-table]
⚠️ WARN Checking OS/2 achVendID.
  • ⚠️ WARN

    OS/2 VendorID value 'NONE' is not yet recognized. If you registered it recently, then it's safe to ignore this warning message. Otherwise, you should set it to your own unique 4 character code, and register it with Microsoft at https://www.microsoft.com/typography/links/vendorlist.aspx


    [code: unknown]
ℹ️ INFO List all superfamily filepaths
  • ℹ️ INFO

    ofl/buenard


    [code: family-path]
ℹ️ INFO Font contains all required tables?
  • ℹ️ INFO

    This font contains the following optional tables:

- loca

- prep

- gasp

[code: optional-tables]

ℹ️ INFO EPAR table present in font?
ℹ️ INFO Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering?
  • ℹ️ 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 Show hinting filesize impact.
  • ℹ️ INFO

    Hinting filesize impact:

ofl/buenard/Buenard[wght].ttf
Dehinted Size 42.5kb
Hinted Size 42.5kb
Increase 24 bytes
Change 0.1 %
[code: size-impact]
ℹ️ INFO Font has old ttfautohint applied?
  • ℹ️ 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 2.000']


    [code: version-not-detected]
🔥 FAIL Ensure the font supports case swapping for all its glyphs.
  • 🔥 FAIL

    The following glyphs lack their case-swapping counterparts:

Glyph present in the font Missing case-swapping counterpart
U+0129: LATIN SMALL LETTER I WITH TILDE U+0128: LATIN CAPITAL LETTER I WITH TILDE
U+0133: LATIN SMALL LIGATURE IJ U+0132: LATIN CAPITAL LIGATURE IJ
U+0135: LATIN SMALL LETTER J WITH CIRCUMFLEX U+0134: LATIN CAPITAL LETTER J WITH CIRCUMFLEX
U+0177: LATIN SMALL LETTER Y WITH CIRCUMFLEX U+0176: LATIN CAPITAL LETTER Y WITH CIRCUMFLEX
U+1EF3: LATIN SMALL LETTER Y WITH GRAVE U+1EF2: LATIN CAPITAL LETTER Y WITH GRAVE
[code: missing-case-counterparts]
🔥 FAIL METADATA.pb: Check URL on copyright string is the same as in repository_url field.

But: font copyright string has 'https://github.com/googlefonts/buenard' and OFL text has 'https://github.com/googlefonts/buenard'

[code: mismatch]
🔥 FAIL Does DESCRIPTION file contain a upstream Git repo URL?
  • 🔥 FAIL

    Please host your font project on a public Git repo (such as GitHub or GitLab) and place a link in the DESCRIPTION.en_us.html file.


    [code: lacks-git-url]
🔥 FAIL Check Google Fonts glyph coverage.
  • 🔥 FAIL

    Missing required codepoints:

- 0x00AF (MACRON)


- 0x0100 (LATIN CAPITAL LETTER A WITH MACRON)


- 0x0101 (LATIN SMALL LETTER A WITH MACRON)


- 0x0102 (LATIN CAPITAL LETTER A WITH BREVE)


- 0x0103 (LATIN SMALL LETTER A WITH BREVE)


- 0x0104 (LATIN CAPITAL LETTER A WITH OGONEK)


- 0x0105 (LATIN SMALL LETTER A WITH OGONEK)


- 0x0106 (LATIN CAPITAL LETTER C WITH ACUTE)


- 0x0107 (LATIN SMALL LETTER C WITH ACUTE)


- 0x010A (LATIN CAPITAL LETTER C WITH DOT ABOVE)


- 103 more.

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

[code: missing-codepoints]
[1] Family checks
ℹ️ INFO Check axis ordering on the STAT table.
  • ℹ️ INFO

    None of the fonts lack a STAT table.

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

[code: summary]

Summary

💥 ERROR ☠ FATAL 🔥 FAIL ⚠️ WARN ⏩ SKIP ℹ️ INFO ✅ PASS 🔎 DEBUG
0 0 4 7 44 8 186 0
0% 0% 2% 3% 18% 3% 75% 0%

Note: The following loglevels were omitted in this report:

  • SKIP
  • PASS
  • DEBUG

@m4rc1e
Copy link
Collaborator

m4rc1e commented Apr 24, 2024

Many glyphs seem to have been removed. I don't think we can release this.

Regular-diffbrowsers_text-Windows-10-10 0 20348-SP0_firefox_124 0 2

@m4rc1e m4rc1e added the -- Glyphset issue Missing glyphs to match a GF subset, wrong glyph mapping, unicode issue, or else label Apr 25, 2024
@simoncozens simoncozens removed the -- Glyphset issue Missing glyphs to match a GF subset, wrong glyph mapping, unicode issue, or else label Apr 29, 2024
@simoncozens
Copy link
Collaborator Author

I pulled the glyphs from the existing fonts (which were not in the sources I got from the designer, so someone must have added them at some point...) and rebuilt them out of components.

This comment was marked as outdated.

This comment was marked as outdated.

Copy link

FontBakery report

fontbakery version: 0.12.6

Experimental checks

These won't break the CI job for now, but will become effective after some time if nobody raises any concern.

[2] Buenard[wght].ttf
⚠️ WARN Validate location, size and resolution of article images.
  • ⚠️ WARN

    Family metadata at ofl/buenard does not have an article.


    [code: lacks-article]
ℹ️ INFO METADATA.pb: Validate family.minisite_url field.
  • ℹ️ INFO

    Please consider adding a family.minisite_url entry.


    [code: lacks-minisite-url]

All other checks

[18] Buenard[wght].ttf
🔥 FAIL Shapes languages in all GF glyphsets.
  • 🔥 FAIL

    GF_Latin_Core glyphset:

Language FAIL messages
de_Latn (German) Some base glyphs were missing: ẞ
^ Shaper produced a .notdef
[code: failed-language-shaping]
  • ⚠️ WARN

    GF_Latin_Core glyphset:

Language WARN messages
de_Latn (German) Some auxiliary glyphs were missing: ẞ
[code: warning-language-shaping]
🔥 FAIL Check Google Fonts glyph coverage.
  • 🔥 FAIL

    Missing required codepoints:

- 0x1E9E (LATIN CAPITAL LETTER SHARP S)

[code: missing-codepoints]

⚠️ WARN Does GPOS table have kerning information? This check skips monospaced fonts as defined by post.isFixedPitch value
  • ⚠️ WARN

    GPOS table lacks kerning information.


    [code: lacks-kern-info]
⚠️ WARN Check math signs have the same width.
  • ⚠️ WARN

    The most common width is 460 among a set of 6 math glyphs.
    The following math glyphs have a different width, though:

Width = 469: greater, less

Width = 567: logicalnot

[code: width-outliers]
⚠️ WARN Check font contains no unreachable glyphs
  • ⚠️ WARN

    The following glyphs could not be reached by codepoint or substitution rules:

- CR

[code: unreachable-glyphs]

⚠️ WARN Check the direction of the outermost contour in each glyph
  • ⚠️ WARN

    The following glyphs have a counter-clockwise outer contour:

* Agrave (U+00C0) has a counter-clockwise outer contour

* Aring (U+00C5) has a counter-clockwise outer contour

* Aringacute (U+01FA) has a counter-clockwise outer contour

* D (U+0044) has a counter-clockwise outer contour

* Dcaron (U+010E) has a counter-clockwise outer contour

* Egrave (U+00C8) has a counter-clockwise outer contour

* Eth (U+00D0) has a counter-clockwise outer contour

* Igrave (U+00CC) has a counter-clockwise outer contour

* Ograve (U+00D2) has a counter-clockwise outer contour

* R (U+0052) has a counter-clockwise outer contour

* 69 more.

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

[code: ccw-outer-contour]
⚠️ WARN Ensure soft_dotted characters lose their dot when combined with marks that replace the dot.
  • ⚠️ WARN

    The dot of soft dotted characters should disappear in other cases, for example: i̅ ị̅ i̥̅ i̦̅ i̧̅ j̅ j̣̅ j̥̅ j̦̅ j̧̅ j̨̅ į̅ į̣̅ į̥̅ į̦̅ į̧̅ į̨̅

Your font fully covers the following languages that require the soft-dotted feature: Lithuanian (Latn, 2,357,094 speakers), Dutch (Latn, 31,709,104 speakers).

Your font does not cover the following languages that require the soft-dotted feature: Dan (Latn, 1,099,244 speakers), Bete-Bendi (Latn, 100,000 speakers), Dii (Latn, 71,000 speakers), Ngbaka (Latn, 1,020,000 speakers), Kom (Latn, 360,685 speakers), Mango (Latn, 77,000 speakers), Ma’di (Latn, 584,000 speakers), Sar (Latn, 500,000 speakers), Ejagham (Latn, 120,000 speakers), Yala (Latn, 200,000 speakers), Mundani (Latn, 34,000 speakers), Vute (Latn, 21,000 speakers), Zapotec (Latn, 490,000 speakers), Southern Kisi (Latn, 360,000 speakers), Fur (Latn, 1,230,163 speakers), Nateni (Latn, 100,000 speakers), Lugbara (Latn, 2,200,000 speakers), Kpelle, Guinea (Latn, 622,000 speakers), Basaa (Latn, 332,940 speakers), Aghem (Latn, 38,843 speakers), Gulay (Latn, 250,478 speakers), Bafut (Latn, 158,146 speakers), Koonzime (Latn, 40,000 speakers), Nzakara (Latn, 50,000 speakers), Makaa (Latn, 221,000 speakers), Cicipu (Latn, 44,000 speakers), Ebira (Latn, 2,200,000 speakers), Navajo (Latn, 166,319 speakers), Igbo (Latn, 27,823,640 speakers), Ijo, Southeast (Latn, 2,471,000 speakers), Mfumte (Latn, 79,000 speakers), Avokaya (Latn, 100,000 speakers), Ekpeye (Latn, 226,000 speakers), Belarusian (Cyrl, 10,064,517 speakers), South Central Banda (Latn, 244,000 speakers), Ukrainian (Cyrl, 29,273,587 speakers).

[code: soft-dotted]
⚠️ WARN Check for codepoints not covered by METADATA subsets.
  • ⚠️ 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: tifinagh, canadian-aboriginal, yi
  • 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+0302 COMBINING CIRCUMFLEX ACCENT: try adding one of: math, coptic, tifinagh, cherokee
  • U+0305 COMBINING OVERLINE: try adding one of: gothic, glagolitic, coptic, elbasan, math
  • U+0306 COMBINING BREVE: try adding one of: old-permic, tifinagh
  • U+0307 COMBINING DOT ABOVE: try adding one of: tai-le, malayalam, coptic, old-permic, math, syriac, tifinagh, canadian-aboriginal
  • U+030A COMBINING RING ABOVE: try adding syriac 16 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, menu

[code: unreachable-subsetting]
⚠️ WARN Ensure fonts have ScriptLangTags declared on the 'meta' table.
  • ⚠️ WARN

    This font file does not have a 'meta' table.


    [code: lacks-meta-table]
⚠️ WARN Checking OS/2 achVendID.
  • ⚠️ WARN

    OS/2 VendorID value 'NONE' is not yet recognized. If you registered it recently, then it's safe to ignore this warning message. Otherwise, you should set it to your own unique 4 character code, and register it with Microsoft at https://www.microsoft.com/typography/links/vendorlist.aspx


    [code: unknown]
ℹ️ INFO List all superfamily filepaths
  • ℹ️ INFO

    ofl/buenard


    [code: family-path]
ℹ️ INFO Font contains all required tables?
  • ℹ️ INFO

    This font contains the following optional tables:

- loca

- prep

- GPOS

- GSUB

- gasp

[code: optional-tables]

ℹ️ INFO Does DESCRIPTION file contain a upstream Git repo URL?
ℹ️ INFO Check for presence of an ARTICLE.en_us.html file
  • ℹ️ INFO

    This font doesn't have an ARTICLE.en_us.html file.


    [code: missing-article]
ℹ️ INFO EPAR table present in font?
ℹ️ INFO Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering?
  • ℹ️ 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 Show hinting filesize impact.
  • ℹ️ INFO

    Hinting filesize impact:

ofl/buenard/Buenard[wght].ttf
Dehinted Size 70.5kb
Hinted Size 70.5kb
Increase 24 bytes
Change 0.0 %
[code: size-impact]
ℹ️ INFO Font has old ttfautohint applied?
  • ℹ️ 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 2.000']


    [code: version-not-detected]
[1] Family checks
ℹ️ INFO Check axis ordering on the STAT table.
  • ℹ️ INFO

    None of the fonts lack a STAT table.

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

[code: summary]

Summary

💥 ERROR ☠ FATAL 🔥 FAIL ⚠️ WARN ⏩ SKIP ℹ️ INFO ✅ PASS 🔎 DEBUG
0 0 2 9 42 10 188 0
0% 0% 1% 4% 17% 4% 75% 0%

Note: The following loglevels were omitted in this report:

  • SKIP
  • PASS
  • DEBUG

@emmamarichal
Copy link
Collaborator

@simoncozens

Fontbakery report:

  • for the Germandbls cap, I usually add it and put 2 S into it, to avoid any tofu or issues.

Outlines:

  • There is no weight variation for the quotesingle and the yen
Screen.Recording.2024-05-16.at.15.02.16.mov
Screen.Recording.2024-05-16.at.15.00.53.mov
  • In regular, we have tofu for phẩm -> ẩ I don't have that issue in the Bold
Screen.Recording.2024-05-16.at.15.05.50.mov

<p>
Buenard is a high-quality serif typeface for art books.
Its based on the Transitional Roman classical structure, with less contrasted strokes and heavier serifs.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its -> It's

@@ -1,9 +1,8 @@
Copyright (c) 2011, FontFuror ([email protected]),
with Reserved Font Names "Buenard" and "Buenard Pro".
Copyright 2024 The Buenard Project Authors (https://github.com/googlefonts/buenard)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we don't keep 2011?

@vv-monsalve vv-monsalve added the -- Needs Upstream Resolution Upstream fix required before moving forward label May 23, 2024
@emmamarichal
Copy link
Collaborator

@simoncozens, what is the status for that PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-- Needs Upstream Resolution Upstream fix required before moving forward I Font Upgrade III VF Replacement Replace an existing family of static fonts with variable fonts
Projects
Status: PR GF
Development

Successfully merging this pull request may close these issues.

Update Buenard
4 participants