Skip to content

fix: Accented notes aren't well sorted in tree view - MEED-10328 - Meeds-io/meeds#4136#1651

Merged
akhanfir merged 1 commit intostable/7.1.x-exofrom
backport-10328
Mar 27, 2026
Merged

fix: Accented notes aren't well sorted in tree view - MEED-10328 - Meeds-io/meeds#4136#1651
akhanfir merged 1 commit intostable/7.1.x-exofrom
backport-10328

Conversation

@akhanfir
Copy link
Copy Markdown
Contributor

Before this change, when create some notes starting with different letters then create a note that starts with an accent char sample and check notes order in tree view, accented notes are listed at the end of the tree view. To fix this issue, locale-aware sorting using Collator has been applied in TreeUtils.tranformToJson() with PRIMARY strength, ensuring accents and case are ignored during comparison. After this change, notes are correctly ordered alphabetically regardless of accented characters.

@akhanfir akhanfir requested review from Jihed525 and mkrout March 27, 2026 08:59
@akhanfir akhanfir enabled auto-merge (squash) March 27, 2026 08:59
Jihed525
Jihed525 previously approved these changes Mar 27, 2026
@akhanfir akhanfir disabled auto-merge March 27, 2026 09:08
…eds-io/meeds#4136 (#1648)

Before this change, when create some notes starting with different letters then create a note that starts with an accent char sample and check notes order in tree view, accented notes are listed at the end of the tree view. To fix this issue, locale-aware sorting using Collator has been applied in TreeUtils.tranformToJson() with PRIMARY strength, ensuring accents and case are ignored during comparison. After this change, notes are correctly ordered alphabetically regardless of accented characters.
@sonarqubecloud
Copy link
Copy Markdown

@akhanfir akhanfir merged commit a035bee into stable/7.1.x-exo Mar 27, 2026
9 checks passed
@akhanfir akhanfir deleted the backport-10328 branch March 27, 2026 09:29
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