-
Notifications
You must be signed in to change notification settings - Fork 18
Modify "See all results" text in Algolia's search modal/dialog #640
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
Modify "See all results" text in Algolia's search modal/dialog #640
Conversation
laliconfigcat
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this can be configured without css modifications:
In this documentation https://docusaurus.io/docs/search#connecting-algolia
thre is a point to this documentation with algolia's full config.
https://docsearch.algolia.com/docs/api/#resultsfootercomponent
Could you please try it?
Deploying configcat-docs with
|
| Latest commit: |
5aeff60
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://4d1180a8.docs-xmy.pages.dev |
| Branch Preview URL: | https://modify-see-all-results-text.docs-xmy.pages.dev |
Thanks for the feedback. I made the changes. Let me know what you think. |
| import type { Config } from '@docusaurus/types'; | ||
| import { themes as prismThemes } from 'prism-react-renderer'; | ||
| import type * as Preset from '@docusaurus/preset-classic'; | ||
| import { relative } from 'path'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unused import.
laliconfigcat
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the change!
No prob. I am going to do the same for the blog. |
`resultsFooterComponent` itself. The wrapper div created a duplicate parent which makes the "See all results" text appear smaller.
|
I removed the wrapper div because |
| export default function SearchBar() { | ||
| return ( | ||
| <DocSearch | ||
| appId='0MLXBNIK0Q' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please add a comment to the docusaurus config next to the appid, secret there that tells us that if we change those ids, we have to do it here too?
Describe the purpose of your pull request
Related issues (only if applicable)
Before:
After:
How to test? (only if applicable)
Preview link: https://modify-see-all-results-text.docs-xmy.pages.dev/docs/search/
What part of the application were affected by the changes?
What should be tested?
Requirement checklist