Skip to content

Fix Search now ignores accents diacritics 95#274

Merged
naveensingh merged 18 commits intoFossifyOrg:mainfrom
jguegel:ignore-accents-diacritics-95
Sep 30, 2025
Merged

Fix Search now ignores accents diacritics 95#274
naveensingh merged 18 commits intoFossifyOrg:mainfrom
jguegel:ignore-accents-diacritics-95

Conversation

@jguegel
Copy link
Copy Markdown
Contributor

@jguegel jguegel commented Sep 27, 2025

Type of change(s)

  • Bug fix
  • Feature / enhancement
  • Infrastructure / tooling (CI, build, deps, tests)
  • Documentation

What changed and why

  • Normalized searchresult list and text to compare
  • ignore accents and diacritics

Tests performed

  • Tried with Android 16 and the following files, which worked: Search: Test ResultList: Tést, Tëst

Closes the following issue(s)

Checklist

  • I read the contribution guidelines.
  • I manually tested my changes on device/emulator (if applicable).
  • I updated the "Unreleased" section in CHANGELOG.md (if applicable).
  • I have self-reviewed my pull request (no typos, formatting errors, etc.).
  • All checks are passing.

jguegel and others added 5 commits September 27, 2025 18:03
@jguegel jguegel changed the title Ignore accents diacritics 95 Fix Search now ignores accents diacritics 95 Sep 27, 2025
Comment thread app/src/main/kotlin/org/fossify/filemanager/fragments/MyViewPagerFragment.kt Outdated
Comment thread CHANGELOG.md Outdated
Comment thread app/src/main/kotlin/org/fossify/filemanager/fragments/ItemsFragment.kt Outdated
Comment thread app/src/main/kotlin/org/fossify/filemanager/fragments/ItemsFragment.kt Outdated
Comment thread app/src/main/kotlin/org/fossify/filemanager/fragments/ItemsFragment.kt Outdated
Comment thread app/src/main/kotlin/org/fossify/filemanager/fragments/MyViewPagerFragment.kt Outdated
Comment thread app/src/main/kotlin/org/fossify/filemanager/fragments/MyViewPagerFragment.kt Outdated
@naveensingh naveensingh added the waiting for author If the author does not respond, the issue will be closed. Otherwise, the label will be removed. label Sep 28, 2025
jguegel and others added 3 commits September 28, 2025 08:11
Co-authored-by: Naveen Singh <36371707+naveensingh@users.noreply.github.com>
Signed-off-by: Jan Guegel <jan@guegel.eu>
Signed-off-by: Jan Guegel <jan@guegel.eu>
@jguegel jguegel requested a review from naveensingh September 28, 2025 06:15
Copy link
Copy Markdown
Member

@naveensingh naveensingh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Searches are now case-sensitive due to the switch to normalizeString().

Comment thread CHANGELOG.md
Comment thread app/src/main/kotlin/org/fossify/filemanager/fragments/ItemsFragment.kt Outdated
Comment thread app/src/main/kotlin/org/fossify/filemanager/fragments/ItemsFragment.kt Outdated
Comment thread app/src/main/kotlin/org/fossify/filemanager/fragments/RecentsFragment.kt Outdated
Co-authored-by: Naveen Singh <36371707+naveensingh@users.noreply.github.com>
@jguegel jguegel requested a review from naveensingh September 28, 2025 11:05
Copy link
Copy Markdown
Member

@naveensingh naveensingh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You missed the MIME type browser launched from the storage tab.

Signed-off-by: Jan Guegel <jan@guegel.eu>
@jguegel
Copy link
Copy Markdown
Contributor Author

jguegel commented Sep 28, 2025

but now every location should be fixed.. maybe time for a proper nap

@jguegel jguegel requested a review from naveensingh September 28, 2025 15:34
@naveensingh
Copy link
Copy Markdown
Member

It is still broken. I'm really starting to feel like your tester here.

Storage tab ➜ Expand Internal or SD Card ➜ Tap on Images/Videos/Other-types ➜ Tap the search button ➜ Search

@jguegel
Copy link
Copy Markdown
Contributor Author

jguegel commented Sep 28, 2025

i am sorry.. that search i overlooked indeed.. looking into it now

fix missing expection toast

Signed-off-by: Jan Guegel <jan@guegel.eu>
@naveensingh naveensingh removed the waiting for author If the author does not respond, the issue will be closed. Otherwise, the label will be removed. label Sep 29, 2025
Comment thread app/src/main/kotlin/org/fossify/filemanager/activities/MimeTypesActivity.kt Outdated
Co-authored-by: Naveen Singh <36371707+naveensingh@users.noreply.github.com>
@jguegel jguegel requested a review from naveensingh September 30, 2025 09:14
@naveensingh naveensingh merged commit fec28af into FossifyOrg:main Sep 30, 2025
5 checks 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.

Search - Ignore accents and diacritics

2 participants