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

Sort even more alphabetically #173

Merged
merged 4 commits into from
Jan 4, 2024
Merged

Sort even more alphabetically #173

merged 4 commits into from
Jan 4, 2024

Conversation

lynn
Copy link
Contributor

@lynn lynn commented Jan 4, 2024

Don't sort all the vowels to the top, and don't do any fancy syllable splitting. This gives a plain, Latin-style sort.

Meanwhile, it doesn't look like Array.sort is stable, so we'll tack on the original string at the end of the sort key. For example is sorted as da,dâ. This basically has the effect of ignoring diacritics in a primary sort but then deferring to them as a tie-breaker.

@lynn lynn merged commit c6aa271 into master Jan 4, 2024
1 check passed
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