Skip to content
This repository was archived by the owner on Feb 23, 2023. It is now read-only.

Adding Algolia Autocomplete search bar to /docs #658

Merged
merged 3 commits into from
May 31, 2022

Conversation

alessandrak
Copy link
Contributor

Adding Algolia Autocomplete search bar to /docs with customized content preview.

The AutocompleSearch component uses Algolia css and custom css, which is in the static/autocomplete-stellar.css file.
All keys are in config.js file and are public and production keys.

Screen size >= 680px
Top
Screen Shot 2022-03-28 at 09 50 09
Bottom
Screen Shot 2022-03-28 at 10 13 01

Screen size < 680px
Screen Shot 2022-03-28 at 10 10 48


.aa-Panel button {
background-color: rgba(62, 27, 219, 0.04);
padding-top: 8px;
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's use rem for things like padding, margin, and font size

Copy link
Contributor

@quietbits quietbits left a comment

Choose a reason for hiding this comment

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

New search looks good! 🎉 Could you please remove everything from the old search (DocSearch)? That way it won't be confusing which one is used and avoid potential conflicts with the new one. 🙏

Comment on lines 1 to 2
export const DOCSEARCH_DOCS_KEY = "a90bee98943eff1ca1adfdaac1e434d8";
export const DOCSEARCH_DOCS_INDEX_NAME = "stellar";
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we can remove these now.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the review! The new search is for /docs pages only, so I think we'll still need DocSearch for the /api pages for now.

@quietbits quietbits merged commit 9f0612e into stellar:master May 31, 2022
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.

3 participants