Skip to content

Implement content search within transcriptions#474

Merged
kdid merged 2 commits into
deploy/stagingfrom
preview/full-text-search
May 5, 2026
Merged

Implement content search within transcriptions#474
kdid merged 2 commits into
deploy/stagingfrom
preview/full-text-search

Conversation

@kdid
Copy link
Copy Markdown
Contributor

@kdid kdid commented Apr 30, 2026

Summary

When a keyword search is performed, the app now concurrently queries a fileset search endpoint for phrase matches within transcription annotations. If content matches are found, a view toggle appears in the results header letting users switch between standard item results and a "within content" view.

The content view displays matches grouped by work - each group shows the work thumbnail alongside a list of matching file sets, each with a thumbnail, a link to the specific canvas, and a snippet of the matching transcription text with the search term highlighted. Pagination is handled independently from the main results. Filters are suppressed in the content view with an explanatory note; the visibility facet is applied to the fileset query itself.

New files

  • lib/queries/fileset-search.ts - builds the fileset search request body
  • lib/utils/fileset-helpers.ts - groups fileset results by work
  • components/Search/TranscriptionResults.tsx - renders grouped content matches
  • components/Search/TranscriptionResults.styled.ts - styles for the above
  • components/Search/TranscriptionPagination.tsx - pagination for content results

@kdid kdid force-pushed the preview/full-text-search branch 5 times, most recently from 7f7597f to 6a80b16 Compare May 5, 2026 18:25
@kdid kdid self-assigned this May 5, 2026
@kdid kdid marked this pull request as ready for review May 5, 2026 18:33
@kdid kdid requested review from bmquinn, mathewjordan and mbklein May 5, 2026 18:38
@kdid kdid changed the title Implement content search of transcriptions Implement content search within transcriptions May 5, 2026
@kdid kdid force-pushed the preview/full-text-search branch from 6a80b16 to daf6444 Compare May 5, 2026 19:24
Copy link
Copy Markdown
Member

@mathewjordan mathewjordan left a comment

Choose a reason for hiding this comment

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

Looks great and love how much it adds.

@kdid kdid merged commit 4ccb753 into deploy/staging May 5, 2026
3 of 4 checks passed
@kdid kdid deleted the preview/full-text-search branch May 5, 2026 20:35
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.

3 participants