googlefonts / googlefonts/fontheight

CLI: sort reports by word list & location

Open
#82 0 comments 0 reactions 0 assignees View on GitHub
cli enhancement
Dominant language
Rust
Stars
9
Forks
1
PR merge metrics
No merged PRs in 30d

Description

Ideally I'd like to sort by word list, and then by location

Questions/Thoughts:
- Should `Report` impl `Ord`? I don't think so
- Should `Location` impl `Ord`? It could, but how do you manage axis order?
- Currently they're unsorted due to use of a `HashSet`
- Sorting alphabetically is an easy option but probably not what people want (personally I'd want more common axes sorted first, like wght and wdth)
- We could use insertion order, but we'd have to pull in a new dependency for stable maps
- Or we could use axis order for the font, but this means you must have a reference to a font to be able to sort. I think this is my current preferred option
- Exemplars are already sorted, so no more details to sort on

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.