recursive-code-config icon indicating copy to clipboard operation
recursive-code-config copied to clipboard

Misleading documentation (OpenType Features)

Open theNestruo opened this issue 2 years ago • 1 comments

According to https://github.com/arrowtype/recursive-code-config/issues/5, "ss08 now controls no-serif L & Z, while the simplified @ is now under ss12". This is properly documented in the text body of the README.md, but the image included just below (font-data/img/recursive-ot_features.png) still shows ss08 as "Simplified 'at'", and there is no ss12.

theNestruo avatar Oct 18 '22 18:10 theNestruo

I have found the up-to-date version of the image at the arrowtype/recursive repository: https://github.com/arrowtype/recursive/blob/main/specimens/repo-artwork/recursive-v1.064-opentype_features.png.

But I have also found that the font-data/features/ss0x.fea file from this repository seems to be outdated in comparison with the src/features/features/ss0x.fea file from the arrowtype/recursive repository (!)

I have updated those files in my fork from the arrowtype/recursive repository to try to create a properly updated version of this repository... But I'm not confident enough to do a pull request; I'm blindly copying files, not really sure what I'm doing (sorry!).

theNestruo avatar Oct 20 '22 20:10 theNestruo

Hey @theNestruo, thank you so much for taking time to file this issue! Sorry it has taken me so long to actually get to these fixes.

I’ve updated the readme image to match the main Recursive repo (as you suggested).

I’ve removed all the feature code that wasn’t actually needed here... reading my code again, it looks like I’m only actually using font-data/features/calt-generated--code_fonts_only.fea, and nothing else. I’ve tested it, and it seems to be working well.

I’ll close this issue, as I think I have solved it, but please reopen it if you think I’ve missed anything!

Thank you again for your time on this!

arrowtype avatar Apr 11 '23 14:04 arrowtype