Freeseer / freeseer

Designed for capturing presentations at conferences. Pre-fill a list of talks to record, record them, and upload them to YouTube with our YouTube Uploader.
http://freeseer.readthedocs.org
GNU General Public License v3.0
215 stars 110 forks source link

Fix #571: Footer broken fix #621

Closed umcsdon closed 10 years ago

umcsdon commented 10 years ago

Fix #571

Introduction:

This problem has been asked so many times in readthedoc's issue section, still no easy fix.

What's worse, the footer is no longer inside

umcsdon commented 10 years ago

Here is my fix. It requires pull the source code from sphinx_rtd_theme repo, and customize it.

It works.

@zxiiro @dideler

dideler commented 10 years ago

Nice work!

Now how do we proceed. Fork the Sphinx theme and subtree / submodule it? Or paste the files directly into our repo assuming we're happy with the theme and don't want updates? /cc @Freeseer/owners

Edit: I'm leaning towards subtree.

umcsdon commented 10 years ago

@dideler I think it also solves the logo problem. Just need to modify the {% block sidebartitle %} section inside layout.html(source) accordingly.

{% block sidebartitle %}
          <a href="{{ pathto(master_doc) }}" class="fa fa-home"> {{ project }}</a>
{% endblock %}

Edit: I will try it tonight.

zxiiro commented 10 years ago

+1 for subtreeing it

umcsdon commented 10 years ago

@dideler @zxiiro Should I close this one & subtreeing it in the next commit?

dideler commented 10 years ago

@umcsdon I forked the theme to Freeseer/sphinx_rtd_theme and added it as a subtree in the 621-rtd-theme-subtree branch. Does that work for you as a starting point? If so, I'll merge it in.

FYI, here is how I added the subtree, from the toplevel of working tree:

git remote add sphinx-rtd-theme git@github.com:Freeseer/sphinx_rtd_theme.git
git subtree add --prefix=docs/source/_themes/ sphinx-rtd-theme master --squash
umcsdon commented 10 years ago

@dideler That's awesome. Please merge it in.

dideler commented 10 years ago

@umcsdon I made a PR at #624 and want to give others a moment to leave their comments before merging it in. If no one opposes, I'll merge it in later today.

umcsdon commented 10 years ago

@dideler No problem.

dideler commented 10 years ago

@umcsdon you're good to go. You can use this existing PR if you want.