Skip to content

Comments

Make app list sort case-insensitive#661

Open
orhtej2 wants to merge 3 commits intoYunoHost:devfrom
orhtej2:patch-2
Open

Make app list sort case-insensitive#661
orhtej2 wants to merge 3 commits intoYunoHost:devfrom
orhtej2:patch-2

Conversation

@orhtej2
Copy link
Contributor

@orhtej2 orhtej2 commented Nov 16, 2025

[code-server, AppFlowy, Element] would previously sort to [AppFlowy, Element, code-server], now they sort to [AppFlowy, code-server, Element] which is a bit more expected. App ĄĘ will still come last, perhaps we should remove accents as well?

Actually I stand corrected

console.log(["Bénévalibre", "ab", "Ąaaa", "Baba", "Bfbf", "ä", "b", "B"].sort(new Intl.Collator("en").compare));

>Array ["ä", "Ąaaa", "ab", "b", "B", "Baba", "Bénévalibre", "Bfbf"]

@orhtej2 orhtej2 marked this pull request as ready for review November 16, 2025 20:10
@orhtej2 orhtej2 marked this pull request as draft November 16, 2025 20:18
@orhtej2 orhtej2 marked this pull request as ready for review November 16, 2025 20:56
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.

1 participant