DOC using pydata doc theme - #1774
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1774 +/- ##
==========================================
- Coverage 94.29% 94.21% -0.08%
==========================================
Files 46 46
Lines 7922 7922
==========================================
- Hits 7470 7464 -6
- Misses 452 458 +6 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
I had a very cursory look, but it looks cool!! Thanks |
|
Maybe it's just me and old habits die hard, but looking at the rendered doc, the fact that the left-hand side navigation has disappeared feels like something is kind of missing. Here is the left-hand side navigation I am talking about in the current doc.
It took me a while to find the header 😅, also you need one more click to see the different sections, first click on "User manual" then get a not so nice looking TOC, then you can chose the section you want to look at. Previously you could see the different sections directly ... Maybe it's just me though, and also I have no idea whether this comes with pydata-sphinx-theme and there is not much we can do to tweak it. On the plus side, probably people are more used to pydata-sphinx-theme that many projects use and having a search bar is definitely nice. |
|
I changed a bit the looking of the TOC you referenced @lesteve . The navigation is still there but not visible from the main page. There is a navigation sidebar for each section User Manual, Module reference, Development, instead of having a global navigation sidebar. It indeed comes from pydata-sphinx-theme, and I don't think we could tweak it. Main sections are in the header and subsections are in the sidebar. IMO it's fine, it makes the sidebar clearer, showing less information. |
|
Yes, I agree, will do it :) |
|
I clicked around a bit and it looks good. Two more things from me:
|
tomMoral
left a comment
There was a problem hiding this comment.
A first batch of comments
|
Regarding the doc, I would say:
I was a bit surprised by the front page, mayeb the logo is a bit big and the purpose of the lib not so clear? Not sure how to improve though. |
|
I like both, no real preference |
tomMoral
left a comment
There was a problem hiding this comment.
It starts to look super good I like it :)
Co-authored-by: Thomas Moreau <[email protected]>
5b62c20 to
0a767f6
Compare
tomMoral
left a comment
There was a problem hiding this comment.
I find this new doc nice. Just one small question before merging.
tomMoral
left a comment
There was a problem hiding this comment.
I think this PR is mostly done.
Just 2 extra comments. also add an entry in the changelog.
Co-authored-by: Thomas Moreau <[email protected]>
Co-authored-by: Thomas Moreau <[email protected]>
The changelog has been added in the last commit |
|
thanks a lot @Nanored4498 ! This looks pretty and it is a nice re-organization :) |




As discussed with @tomMoral, I tried to use the pydata theme for the sphinx doc. Let me know if it seems nice to you.