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

Fix on Readthedocs theme #168

Merged
merged 4 commits into from Oct 22, 2014
Merged

Fix on Readthedocs theme #168

merged 4 commits into from Oct 22, 2014

Conversation

@ebertti
Copy link
Contributor

@ebertti ebertti commented Oct 22, 2014

  • better responsive for small screen
  • back with javascript for small screen, ref to jquery, new ref for modernizr and fix repository icon for bitbucket
@@ -6,3 +6,6 @@ htmlcov/
mkdocs.egg-info/
*.pyc
.coverage


.idea

This comment has been minimized.

d0ugal added a commit that referenced this pull request Oct 22, 2014
@d0ugal d0ugal merged commit 8747506 into mkdocs:master Oct 22, 2014
1 check passed
1 check passed
@tomchristie
continuous-integration/travis-ci The Travis CI build passed
Details
@d0ugal
Copy link
Member

@d0ugal d0ugal commented Oct 22, 2014

Manually tested and merged! Thanks!

Quite a subtle change, but makes it look much better :)

d0ugal added a commit that referenced this pull request Oct 22, 2014
@ericholscher
Copy link
Contributor

@ericholscher ericholscher commented Oct 22, 2014

If this would be applicable to the base Sphinx theme, it would be awesome
to have this upstream as well.

On Wed, Oct 22, 2014 at 11:57 AM, Dougal Matthews notifications@github.com
wrote:

Manually tested and merged! Thanks!

Quite a subtle change, but makes it look much better :)


Reply to this email directly or view it on GitHub
#168 (comment).

Eric Holscher
Maker of the internet residing in Portland, Or
http://ericholscher.com

@d0ugal
Copy link
Member

@d0ugal d0ugal commented Oct 22, 2014

@ericholscher I think these changes just add things in that were lost in the process of copying over.

@d0ugal d0ugal mentioned this pull request Jan 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants