Skip to content

Multiple small UX enhancements related to investigation sharing/user suggestion component#3868

Merged
tillprochaska merged 5 commits intorelease/4.0.0from
fix/investigation-share-email-normalization
Sep 24, 2024
Merged

Multiple small UX enhancements related to investigation sharing/user suggestion component#3868
tillprochaska merged 5 commits intorelease/4.0.0from
fix/investigation-share-email-normalization

Conversation

@tillprochaska
Copy link
Contributor

This PR fixes a few small UX bugs related to investigation sharing and user suggestions:

  • Previously, when clearing the input field for user suggestions, the old suggestions for the previous value were still displayed.
  • Email addresses weren’t properly normalized, so [email protected] (leading whitespace) or [email protected] wouldn’t match [email protected].
  • Due to a bug in the underlying Blueprint component library, we were sending duplicate calls to the suggestions API endpoint on every keystroke.

We’ve recently updated this in the backend, but hadn’t yet updated the frontend.
Previously, after you cleared the input field, the component would still display suggestions for the previous input value until the user typed at least 5 chars (the min prefix length).
…tization

This automatically sanitizes the entered value by removing leading and trailing whitespace. When copy-pasting email addresses, it commonly happens that the copy-pasted text includes leading or trailing whitespace.
@tillprochaska tillprochaska force-pushed the fix/investigation-share-email-normalization branch from 1a8ac4c to 5b6b0ad Compare September 23, 2024 16:13
@tillprochaska tillprochaska marked this pull request as ready for review September 23, 2024 16:21
Copy link
Contributor

@stchris stchris left a comment

Choose a reason for hiding this comment

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

Looks good to me! Thanks for the fixes

@tillprochaska tillprochaska merged commit f265c88 into release/4.0.0 Sep 24, 2024
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.

2 participants