Skip to content

Conversation

jguegel
Copy link
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 to fix every search i could find

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
Copy link
Contributor Author

jguegel commented Sep 28, 2025

@naveensingh need an idea to improve line length...

@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 jguegel requested a review from naveensingh September 28, 2025 06:30
removed unused import
hopefully fix for MaxLineLength
@jguegel jguegel requested a review from naveensingh September 28, 2025 08:42
Copy link
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.

  • It still doesn't work in QueueActivity.
  • More formatting errors below.

https://github.com/FossifyOrg/General-Discussion#contributing-code

Jan Guegel added 2 commits September 28, 2025 12:17
remove normalizing in MainActivity since we are normalizing in fragments
fix incorrect (not needed) lambda parameter in several fragment
@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
Copy link
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.

Thanks.

@naveensingh naveensingh merged commit 5522883 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.

Ignore diacritics in search
2 participants