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

switch to sphinx-book-theme #5368

Closed
wants to merge 1 commit into from

Conversation

ltalirz
Copy link
Member

@ltalirz ltalirz commented Feb 19, 2022

fixes #4441

@ltalirz
Copy link
Member Author

ltalirz commented Feb 19, 2022

Besides the logo size that will need to be adjusted, I have to say that I find the default font choices of the pydata-sphinx-theme a bit nicer

Old
image

New
image

@chrisjsewell
Copy link
Member

Fine, but I already have a PR doing this

@ltalirz
Copy link
Member Author

ltalirz commented Feb 19, 2022

@chrisjsewell You mean a PR switching the theme of the AiiDA docs?

Could you link to #4441 please; that way I would have noticed it

@chrisjsewell
Copy link
Member

I was holding it off while I fixed some things: executablebooks/sphinx-book-theme#459

@ltalirz
Copy link
Member Author

ltalirz commented Feb 19, 2022

Also, is there a built-in way to display a twitter handle?

@ltalirz
Copy link
Member Author

ltalirz commented Feb 19, 2022

Really like the announcement banner support; I think this will come in very handy

@chrisjsewell
Copy link
Member

You mean a PR switching the theme of the AiiDA docs?

No, I have a personal branch doing this and a number of other things, like moving from sphinx-panels to sphinx-design, but had opened the PR yet while improve some things on my end

@chrisjsewell
Copy link
Member

There's lots of work going on in this space, with sphinx-book-theme and others (python/docs-community#1) so I was wary to push forward a change right now

@ltalirz
Copy link
Member Author

ltalirz commented Feb 19, 2022

Ok, then I'll point that out in the issue and leave this change to you

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.

docs: consider switching to sphinx-book-theme
2 participants