Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: add search #2589

Merged
merged 18 commits into from Dec 1, 2020
Merged

docs: add search #2589

merged 18 commits into from Dec 1, 2020

Conversation

segunadebayo
Copy link
Member

@segunadebayo segunadebayo commented Nov 23, 2020

Description
This PR adds a custom Omnibar search to the docs as a temporary replacement for Algolia search. I've got the code for algolia in there as well. Once Algolia re-indexes the website, we'll swap out the omnibar with Algolia to get access to search analytics.

Possible Enhancement
Add quick and dirty analytics code that sends the search term to a spreadsheet for the team. Might not be needed once Algolia responds.

Fixes: #2588 #2264

@vercel
Copy link

vercel bot commented Nov 23, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/chakra-ui/chakra-ui/2wmo1giba
✅ Preview: https://chakra-ui-git-docs-add-search.chakra-ui.vercel.app

@changeset-bot
Copy link

changeset-bot bot commented Nov 23, 2020

🦋 Changeset detected

Latest commit: a89061e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 9 packages
Name Type
@chakra-ui/layout Patch
@chakra-ui/react Patch
@chakra-ui/test-utils Patch
chakra-cra Patch
chakra-nextjs Patch
chakra-nextjs-ts Patch
@chakra-ui/props-docs Patch
@chakra-ui/docs Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@segunadebayo segunadebayo changed the title feat: add search docs: add search Nov 23, 2020
@ljosberinn
Copy link
Contributor

fixes #2588 #2264

@lgtm-com
Copy link

lgtm-com bot commented Nov 23, 2020

This pull request introduces 1 alert when merging 9fe87a6 into 9a3c806 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 29, 2020

This pull request introduces 1 alert when merging 0c46cd0 into cf70dc6 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@with-heart with-heart added the Topic: Website 🖥 Issues and PRs relating to the Chakra UI website label Nov 29, 2020
@with-heart with-heart self-requested a review November 29, 2020 18:32
@lgtm-com
Copy link

lgtm-com bot commented Nov 29, 2020

This pull request introduces 3 alerts when merging f170723 into 44ee1f9 - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 29, 2020

This pull request introduces 3 alerts when merging f7dbac2 into 44ee1f9 - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 30, 2020

This pull request introduces 2 alerts when merging 6b52741 into 88db4c7 - view on LGTM.com

new alerts:

  • 2 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 30, 2020

This pull request introduces 3 alerts when merging df056a3 into 88db4c7 - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 30, 2020

This pull request introduces 3 alerts when merging 628e553 into 88db4c7 - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 30, 2020

This pull request introduces 3 alerts when merging 480c371 into 0ddf6e2 - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Nov 30, 2020

This pull request introduces 3 alerts when merging dad078a into 0ddf6e2 - view on LGTM.com

new alerts:

  • 3 for Unused variable, import, function or class

@segunadebayo segunadebayo marked this pull request as ready for review November 30, 2020 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Topic: Website 🖥 Issues and PRs relating to the Chakra UI website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Search to docs for easier navigation & lookup
3 participants