googlefonts / roboto-classic

Development of a Roboto Variable font
SIL Open Font License 1.1
154 stars 15 forks source link

Change Weight axis range from 250-900 to 100-900 #24

Closed davelab6 closed 4 years ago

davelab6 commented 5 years ago

Currently the Weight axis values in fvar do not go down to 100, as expected for CSS, but have carried over the legacy OS/2 table values which went down to 250 as a workaround for Windows auto-bolding of values of 249 and below.

Today:

<Axis>
  <AxisTag>wght</AxisTag>
  <MinValue>250.0</MinValue>
  <DefaultValue>400.0</DefaultValue>
  <MaxValue>900.0</MaxValue>
</Axis>

What I expect:

<Axis>
  <AxisTag>wght</AxisTag>
  <MinValue>100.0</MinValue>
  <DefaultValue>400.0</DefaultValue>
  <MaxValue>900.0</MaxValue>
</Axis>
davelab6 commented 5 years ago

@paleydreier please schedule this :)

paleydreier commented 5 years ago

cc @sannorozco @pichotta

dberlow commented 5 years ago

Well, fvar are you wish, but... Having the wght min at 250’s not such a bad thing because it’s not that light.

On Oct 15, 2018, at 8:24 PM, Dave Crossland notifications@github.com wrote:

Currently the Weight axis values in fvar do not go down to 100, as expected for CSS, but have carried over the legacy OS/2 table values which went down to 250 as a workaround for Windows auto-bolding of values of 249 and below.

Today:

wght 250.0 400.0 900.0 What I expect: wght 100.0 400.0 900.0 — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.
sannorozco commented 5 years ago

Now both VFs have corrected fvar tables:

—Roboto-VF.ttf —Roboto-min-VF.ttf

davelab6 commented 5 years ago

I see in https://github.com/TypeNetwork/Roboto/commit/47598f229c77f87ecfc2e3767a6e2402e92f7c71 you did update only the Roboto-min-VF.ttf

Please check in Roboto-VF.ttf and the sources :)

sannorozco commented 5 years ago

Yes, only Roboto-min-VF.ttf needed to update far with these values, I checked again and far on both VFs seems good. Made a small update in min.designspace.

davelab6 commented 4 years ago

@sannorozco @dberlow can this issue be closed?