Skip to content

Intuitive search ordering of filtered data #6192

Open
rapterjet2004 wants to merge 1 commit into
masterfrom
issue-6170-intuitive-search-results
Open

Intuitive search ordering of filtered data #6192
rapterjet2004 wants to merge 1 commit into
masterfrom
issue-6170-intuitive-search-results

Conversation

@rapterjet2004
Copy link
Copy Markdown
Contributor

@rapterjet2004 rapterjet2004 commented May 13, 2026

Screenshot 2026-05-18 at 10 38 03 AM Screenshot 2026-05-18 at 10 36 58 AM

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔖 Capability is checked or not needed
  • 🔙 Backport requests are created or not needed: /backport to stable-xx.x
  • 📅 Milestone is set
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

@rapterjet2004 rapterjet2004 self-assigned this May 13, 2026
@rapterjet2004 rapterjet2004 added the 3. to review Waiting for reviews label May 13, 2026
@github-actions
Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/25810436517/artifacts/6975286920
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

@github-actions
Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/25878536561/artifacts/7002121146
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

@mahibi mahibi requested a review from Copilot May 15, 2026 12:21
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Improves search ordering in the conversations list so that filtered results show exact matches first, then word-boundary matches, then the remainder, across local conversations, open conversations, contacts, and message results.

Changes:

  • Introduces a generic processOrderAndAdd helper to reorder a list by up to two predicates before invoking an addAction.
  • Applies this ordering to the four result groups (local conversations, open conversations, contacts, messages) using regex-based matching.
  • Adds a small Iterable<T>.split extension to partition lists by a predicate.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/25928985499/artifacts/7022662582
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

Signed-off-by: rapterjet2004 <juliuslinus1@gmail.com>
@rapterjet2004 rapterjet2004 force-pushed the issue-6170-intuitive-search-results branch from f7976a0 to 72f0b00 Compare May 18, 2026 16:04
@github-actions
Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/26045148322/artifacts/7063465933
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

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

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unintuitive search results

3 participants