Skip to content

Add opensearch support to the mathlib docs#256

Merged
PatrickMassot merged 4 commits into
leanprover-community:newsitefrom
eric-wieser:eric-wieser/chrome-omnibox-support
May 1, 2022
Merged

Add opensearch support to the mathlib docs#256
PatrickMassot merged 4 commits into
leanprover-community:newsitefrom
eric-wieser:eric-wieser/chrome-omnibox-support

Conversation

@eric-wieser

@eric-wieser eric-wieser commented Mar 3, 2022

Copy link
Copy Markdown
Member

This should make supported browsers offer a "search mathlib docs" feature in the title bar.

It's not clear to me whether this will work in the mathlib doc repo, there are rumors online that it has to be at the site root.

I'll make a PR there anyway to test (leanprover-community/doc-gen#161)

@eric-wieser eric-wieser marked this pull request as draft March 3, 2022 18:15
@eric-wieser eric-wieser marked this pull request as ready for review March 16, 2022 14:13
@eric-wieser

Copy link
Copy Markdown
Member Author

I can't really come up with a way of testing this other than deploying it and seeing what happens.

@PatrickMassot PatrickMassot merged commit 760b800 into leanprover-community:newsite May 1, 2022
@eric-wieser

Copy link
Copy Markdown
Member Author

Thanks for merging this. Now that I can test it, it seems that this doesn't actually work in chrome :(

Maybe it works in firefox like @gebner said leanprover-community/doc-gen#161 did? Either way, if it doesn't work in chrome we may as well revert this and put the change in doc-gen instead.

@gebner

gebner commented May 1, 2022

Copy link
Copy Markdown
Member

I don't think this is deployed yet. At least I can't find any mention of opensearch.xml in the page source code.

@bryangingechen

Copy link
Copy Markdown
Collaborator

The jobs I re-ran just finished, so hopefully it's deployed now?

@gebner

gebner commented May 1, 2022

Copy link
Copy Markdown
Member

Can confirm that it works on firefox.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants