Box glyphs don't close vertically

[gavineadie]

The box glyphs don't form continuous vertical lines .. horizontals are OK. Note the gaps in the vertical lines below (from Xcode). Menlo, SourceCodePro and other "coding" fonts get it right:

[gavineadie]

.. on the other hand, subscripts do look nice:

[chrissimpkins]

The box glyphs don't form continuous lines

Mind giving this same block of text a try with this test version that uses tighter vertical spacing? It is renamed 'Hack Tight' so that it can be installed along with the release version.

https://github.com/chrissimpkins/Hack/blob/vertical-spacing/build/test_builds/HackTight-Regular-h.ttf

I'll take a look at the position of the glyphs to confirm that this isn't an alignment issue with the glyphs themselves rather than a line spacing issue.

[gavineadie]

It's a little tighter, but doesn't close the gap. I do prefer the slightly tighter spacing though:

[chrissimpkins]

Close but no cigar. OK, probably in the glyphs themselves. Going to shelve this until I get the vertical spacing issues sorted out or else we would redo it anyways. Will get back on it once I have a stable set of vertical metrics to work with. Will be releasing at least one tighter vert spacing version along with the current spacing down the road. Will let you know when ready.

[msharov]

Other double line joints also do not quite line up, with ╩ ═ ╧ being a pixel lower than ╔ ╦. I'm using 16 point Hack-Regular.

[chrissimpkins]

Vertical metrics were updated across the entire font with significantly tighter vertical spacing. Mind giving these fonts a try and let me know how close we are on these glyphs? Thanks.

Available in #111

[gavineadie]

Looks good

Note, I'm not using the 'junction' glyphs (mentioned by @msharov), just the 'straights' and 'corners' .. Thanks for your work on this.

[chrissimpkins]

Fantastic. My original vertical metrics changes led to these problems. We have requests for looser spacing variants so we will need to address this down the line. Thank you very much for looking into this issue again.

[chrissimpkins]

I will close this issue when we release these changes unless you have any other associated alignment concerns.

Await feedback from @msharov about the other glyph set and will open a separate issue if this is not addressed.

[chrissimpkins]

These vertical spacing changes are now included in v2.015. Thank you very much for the issue report and the testing that you performed on this issue.