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.

Sign up for GitHub

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

Jump to bottom

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

d0ugal
d0ugal reviewed Oct 22, 2014
View reviewed changes
.gitignore
@@ -6,3 +6,6 @@ htmlcov/
mkdocs.egg-info/
*.pyc
.coverage


.idea
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This kinda thing belongs in your global .gitignore - https://help.github.com/articles/ignoring-files/#create-a-global-gitignore

d0ugal added a commit that referenced this pull request Oct 22, 2014
@d0ugal
Merge pull request #168 from ebertti/readthedocs
8747506 
Fix on Readthedocs theme
@d0ugal d0ugal merged commit 8747506 into mkdocs:master Oct 22, 2014
@d0ugal
Copy link
Member

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
@d0ugal
Add notes to the release docs for #171, #168 and #104
131928f
@ericholscher
Copy link
Contributor

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 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
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ebertti @d0ugal @ericholscher