Skip to content

Use the ReadTheDocs theme as the default Sphinx theme #9972

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

Merged
merged 1 commit into from
Jun 28, 2018

Conversation

javiereguiluz
Copy link
Member

I can't make our Sphinx theme work and I can't fix any of the issues reported about it. So let's stop wasting time on this and let's use the default Sphinx theme provided by ReadTheDocs (https://github.com/rtfd/sphinx_rtd_theme).

It doesn't look like symfony.com, but it's beautiful, responsive and it works. In the future we can use it as the base to create our own theme.

This is how it looks after running cd _build && make html:

index

page

@javiereguiluz javiereguiluz added this to the 2.8 milestone Jun 27, 2018
@javiereguiluz javiereguiluz merged commit a5a2171 into symfony:2.8 Jun 28, 2018
javiereguiluz added a commit that referenced this pull request Jun 28, 2018
…viereguiluz)

This PR was merged into the 2.8 branch.

Discussion
----------

Use the ReadTheDocs theme as the default Sphinx theme

I can't make our Sphinx theme work and I can't fix any of the issues reported about it. So let's stop wasting time on this and let's use the default Sphinx theme provided by ReadTheDocs (https://github.com/rtfd/sphinx_rtd_theme).

It doesn't look like symfony.com, but it's beautiful, responsive and it works. In the future we can use it as the base to create our own theme.

This is how it looks after running `cd _build && make html`:

![index](https://user-images.githubusercontent.com/73419/41984189-5b998672-7a30-11e8-83f5-f6fdeb437a44.png)

![page](https://user-images.githubusercontent.com/73419/41984192-5d43f606-7a30-11e8-9db4-366ac6fdafec.png)

Commits
-------

a5a2171 Use the ReadTheDocs theme as the default Sphinx theme
@javiereguiluz javiereguiluz deleted the fix_9246 branch June 28, 2018 06:49
Copy link
Contributor

@HeahDude HeahDude left a comment

Choose a reason for hiding this comment

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

👍

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

Successfully merging this pull request may close these issues.

3 participants