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

Focus on search box #21

Closed
wants to merge 1 commit into from
Closed

Conversation

burodepeper
Copy link

@burodepeper burodepeper commented Jun 13, 2016

The default installation for mkdocs had the auto focus on the search input field, so I copied the base.js to the cinder directory, and I'm kind of assuming here that this will work. Let me put it this way: as long as you haven't modified any of the html or css, I'm not really sure why this won't work... ; )

@anlau
Copy link
Contributor

anlau commented Jan 21, 2019

@chrissimpkins, could you elaborate on why this PR was closed?

This behaviour can probably be introduced using extra_javascript, but I think it should be part of the theme (like it is with MkDocs default theme).

I can open a PR with lines 25-27.

Thanks

@chrissimpkins
Copy link
Owner

There was a refactor of the theme here with the release of MkDocs v1.0 and I believe this was closed when that took place. I am not opposed to including the functionality but we need to make sure that the JS change works as expected.

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.

3 participants