Skip to content

Fix sorting of advanced attribute list#10091

Merged
droidmonkey merged 1 commit intokeepassxreboot:developfrom
marco-langer:fix/advanced-attribute-list-sort
Feb 1, 2025
Merged

Fix sorting of advanced attribute list#10091
droidmonkey merged 1 commit intokeepassxreboot:developfrom
marco-langer:fix/advanced-attribute-list-sort

Conversation

@marco-langer
Copy link
Copy Markdown
Contributor

Sort advanced attribute list using locale aware sort.

Fixes #6175

Testing strategy

  1. Add an entry to a database
  2. Go to "Edit entry" -> "Advanced"
  3. Add some additional attributes via the "Add" button. E.g.: "Test1", "Test12", "Test2"
  4. The additional attributes should be sorted now as "Test1", "Test2", "Test12"

Type of change

  • ✅ Bug fix (non-breaking change that fixes an issue)

@marco-langer marco-langer marked this pull request as draft December 10, 2023 18:53
@droidmonkey droidmonkey added this to the v2.8.0 milestone Jan 6, 2024
@droidmonkey droidmonkey modified the milestones: v2.8.0, v2.7.10 Aug 19, 2024
@droidmonkey droidmonkey marked this pull request as ready for review January 12, 2025 17:14
Sort advanced attribute list using locale aware sort.

Fixes keepassxreboot#6175
@droidmonkey droidmonkey force-pushed the fix/advanced-attribute-list-sort branch from 052993a to c1dd402 Compare January 22, 2025 04:44
@droidmonkey droidmonkey merged commit 8ca90a0 into keepassxreboot:develop Feb 1, 2025
pull bot pushed a commit to Graysonbarton/keepassxc that referenced this pull request Feb 1, 2025
Sort advanced attribute list using locale aware sort.

Fixes keepassxreboot#6175
droidmonkey pushed a commit that referenced this pull request Feb 2, 2025
Sort advanced attribute list using locale aware sort.

Fixes #6175
@droidmonkey droidmonkey added the pr: backported Pull request backported to previous release label Feb 2, 2025
pull bot pushed a commit to blog2i2j/keepassxreboot.._..keepassxc that referenced this pull request Feb 22, 2025
Sort advanced attribute list using locale aware sort.

Fixes keepassxreboot#6175
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: backported Pull request backported to previous release user interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sort advanced attribute list using locale aware sort

2 participants