Add params to offlineSearch examples

[svrnm]

While trying out the offline search feature I struggled to get it working until I figured out that the offlineSearch should be within params.

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[deining]

Thanks for your contribution! Your suggestion is valid IMHO, I consider it an improvement to the user guide.
Looking at your changes in search.md, I realized , that offlineSearch: true appears two more times in the subchapters Changing the summary length of the local search results and Changing the maximum result count of the local search. Shouldn't we mention params at these occurrences as well? @svrnm: do you want to go for this as well?

[deining]

Additional note: I just realized that in the chapter documenting how to enable GCSE search engine, the line gcs_engine_id = "xxxx" should be within params as well. We should address this in a separate ticket/PR IMHO.

[svrnm]

Thanks for your contribution! Your suggestion is valid IMHO, I consider it an improvement to the user guide. Looking at your changes in search.md, I realized , that offlineSearch: true appears two more times in the subchapters Changing the summary length of the local search results and Changing the maximum result count of the local search. Shouldn't we mention params at these occurrences as well? @svrnm: do you want to go for this as well?

Yes, I can take a look

[Marzal]

Additional note: I just realized that in the chapter documenting how to enable GCSE search engine, the line gcs_engine_id = "xxxx" should be within params as well. We should address this in a separate ticket/PR IMHO.

Maybe there should be a full review, because there are more references without the params indications :

• https://www.docsy.dev/docs/adding-content/lookandfeel/#code-highlighting-with-prism
• https://www.docsy.dev/docs/adding-content/repository-links/

[svrnm]

@deining done, I fixed the other 2 places as well

[chalin]

@deining - is this PR good to go?
@Marzal or @deining - consider creating an issue for the other sections, thanks.

[chalin]

@deining - is this ready?

[deining]

LGTM

[svrnm]

thanks for merging :-)