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

montserrat: v7.200 added #1294

Merged
merged 1 commit into from
Nov 3, 2017
Merged

montserrat: v7.200 added #1294

merged 1 commit into from
Nov 3, 2017

Conversation

m4rc1e
Copy link
Collaborator

@m4rc1e m4rc1e commented Oct 31, 2017

Changes since previous release

  • Glyph refinements
  • Cyrillic and Cyrillic-ext charsets added

screen shot 2017-10-31 at 16 08 38

Taken from the upstream repo, https://github.com/JulietaUla/Montserrat

Fb Report

There is a total of 120 tests on <Section: Default>.
Start ... running 1718 individual test executions.
>> com.google.fonts/test/006
   DESCRIPTION.en_us.html must have less than 1000 bytes.
 * FAIL: DESCRIPTION.en_us.html must have size smaller than 1000 bytes.

   Result: FAIL

>> com.google.fonts/test/011
   Fonts have equal numbers of glyphs?
 * FAIL: Fonts have different numbers of glyphs:

| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-ExtraBold.ttf | 1943 |
| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-SemiBoldItalic.ttf | 1970 |
| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-Bold.ttf | 1943 |
| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-Italic.ttf | 1970 |
| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-ThinItalic.ttf | 1970 |
| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-Light.ttf | 1943 |
| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-LightItalic.ttf | 1970 |
| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-MediumItalic.ttf | 1970 |
| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-BlackItalic.ttf | 1970 |
| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-Regular.ttf | 1943 |
| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-ExtraBoldItalic.ttf | 1970 |
| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-ExtraLight.ttf | 1943 |
| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-SemiBold.ttf | 1943 |
| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-BoldItalic.ttf | 1970 |
| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-Thin.ttf | 1943 |
| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-ExtraLightItalic.ttf | 1970 |
| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-Medium.ttf | 1943 |
| /Users/marc/Documents/googlefonts/fonts/ofl/montserrat/Montserrat-Black.ttf | 1943 |


   Result: FAIL

>> com.google.fonts/test/012
   Fonts have equal glyph names?
 * FAIL: Fonts have different glyph names.

   Result: FAIL

======== <Section: Default> ========
1718 tests in section


Section results:

    ERROR: 0
    FAIL: 3
    WARN: 122
    SKIP: 36
    INFO: 74
    PASS: 1483

======== END <Section: Default> ========

Total:

    ERROR: 0
    FAIL: 3
    WARN: 122
    SKIP: 36
    INFO: 74
    PASS: 1483


DONE!


Summary:
Description: Same as before, not changing.

Glyphsets: We need to improve these tests. The Italic contains more glyphs because it contains localized Bulgarian forms. @felipesanches we should include these edge cases in the checks.

IE 11 Rendering
montserrat-waterfall

LGTM. @davelab6 no x-heights jumps :-)

Design Changes
The existing glyphs have been improved significantly since the previous release. Here's a small text sample.

montserrat-design

Changes since previous release
- Glyph refinements
- Cyrillic and Cyrillic-ext charsets added

Taken from the upstream repo, https://github.com/JulietaUla/Montserrat
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

Successfully merging this pull request may close these issues.

2 participants