Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

feat(enquiries): suggest existing questions to the user #199

Merged
merged 6 commits into from
Sep 2, 2021

Conversation

LoneRifle
Copy link
Contributor

@LoneRifle LoneRifle commented Sep 1, 2021

Problem

Closes #173

Solution

  • refine default handleSubmit flag to just push history; regardless of
    specified hook, GA event will be sent
  • add handleAbandon hook, capturing when a search is abandoned
  • allow search icon to be disabled by setting showSearchIcon to false
  • retain search text value even on abandoning
  • make SearchBox a component, not a form
  • introduce also the means to open search items in new window to retain user context as much as possible
  • use SearchBox to suggest questions

Screenshot

image

- refine default handleSubmit flag to just push history; regardless of
  specified hook, GA event will be sent
- add handleAbandon hook, capturing when a search is abandoned
- allow search icon to be disabled by setting `showSearchIcon` to false
@LoneRifle LoneRifle requested a review from nauynix September 1, 2021 10:37
@LoneRifle LoneRifle force-pushed the feat/enquiries/suggest branch from bc5da92 to 816505e Compare September 1, 2021 11:55
@nauynix
Copy link
Contributor

nauynix commented Sep 2, 2021

pressing enter on question title in enquiry modal now leads to search page

@LoneRifle
Copy link
Contributor Author

pressing enter on question title in enquiry modal now leads to search page

good catch! added a flag to control this behaviour.

@nauynix
Copy link
Contributor

nauynix commented Sep 2, 2021

image

You can't submit now because it thinks `questionTitle` is empty

@LoneRifle LoneRifle force-pushed the feat/enquiries/suggest branch from baf3ea9 to dd9a001 Compare September 2, 2021 08:11
@LoneRifle
Copy link
Contributor Author

spotted and corrected the missing inputRef

Copy link
Contributor

@nauynix nauynix left a comment

Choose a reason for hiding this comment

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

lgtm

@LoneRifle LoneRifle force-pushed the feat/enquiries/suggest branch from b92f157 to 0ece128 Compare September 2, 2021 08:37
@LoneRifle LoneRifle force-pushed the feat/enquiries/suggest branch from 0ece128 to 30411b1 Compare September 2, 2021 08:55
@LoneRifle LoneRifle merged commit 0df1e64 into develop Sep 2, 2021
@LoneRifle LoneRifle deleted the feat/enquiries/suggest branch September 2, 2021 09:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

when submitting an enquiry form, auto complete as user is typing Question Title
2 participants