Skip to content

Conversation

kmcginnes
Copy link
Collaborator

@kmcginnes kmcginnes commented Mar 5, 2025

Description

Fixes two issues:

  • Ensure long text is wrapped no matter what on search result items to ensure the add to graph button is visible
  • Update replacePrefix() to ignore case for its matching
  • Also include generated prefixes that only match a single type

Validation

  • Tested in RDF dataset

With prefixes

CleanShot 2025-03-05 at 16 14 40@2x

Without prefixes

CleanShot 2025-03-05 at 16 14 58@2x

Related Issues

Check List

  • I confirm that my contribution is made under the terms of the Apache 2.0
    license.
  • I have run pnpm checks to ensure code compiles and meets standards.
  • I have run pnpm test to check if all tests are passing.
  • I have covered new added functionality with unit tests if necessary.
  • I have added an entry in the Changelog.md.

@kmcginnes kmcginnes marked this pull request as ready for review March 5, 2025 22:10
@kmcginnes kmcginnes force-pushed the fix-search-result-wrapping branch from 9161b73 to a6dc7bb Compare March 5, 2025 22:12
@andreachild
Copy link
Contributor

LGTM

@kmcginnes kmcginnes merged commit 77ab0d4 into aws:main Mar 5, 2025
2 checks passed
@kmcginnes kmcginnes deleted the fix-search-result-wrapping branch March 5, 2025 23:31
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.

Long RDF URIs push the add to graph button off the screen
2 participants