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

💄 Change Sphinx theme to furo #791

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

webknjaz
Copy link
Member

@webknjaz webknjaz commented Nov 16, 2020

@webknjaz
Copy link
Member Author

@pradyunsg
Copy link
Member

TBH, Furo doesn't really fit well with the structure of this documentation -- open any guide and that should provide context on why this sidebar approach can't work for this documentation.

@webknjaz
Copy link
Member Author

webknjaz commented Nov 17, 2020

@pradyunsg could you expand on that? I don't see anything concerning except for TOC.

@pradyunsg
Copy link
Member

@pfmoore
Copy link
Member

pfmoore commented Nov 18, 2020

@pradyunsg I just checked that link (on Chrome, on a big desktop screen) and I don't see what the issue is that you're flagging up.

@webknjaz
Copy link
Member Author

@pradyunsg I don't see any problems there. Not sure what you mean to point out...

@pradyunsg
Copy link
Member

Screenshot 2020-11-18 at 3 23 44 PM
Screenshot 2020-11-18 at 3 23 52 PM
Screenshot 2020-11-18 at 3 24 01 PM

I feel like the sidebar becomes overwhelming to navigate (which is expected, since that's what the sidebar was designed to do when there's a bit much in the docs, due to Furo's bias toward smaller documentation trees).

@pfmoore
Copy link
Member

pfmoore commented Nov 19, 2020

Hmm. If you mean that having lots of long titles for the guides means that the left hand sidebar is clumsy looking, then yes it is. But I dismissed that criticism as I see it as all part of the general "feel" of Furo, having bigger, more spaced out text. I don't like that style much, because in general I think it makes for harder navigation in text-heavy documentation, but "I don't like the look of Furo" isn't really the point here.

If you want an example in another document, I find pip's "Reference Guide" sidebar awkward (too many items, the wide spacing makes it difficult to locate items at a glance) and also the "Development -> Architecture of pip's internals" sidebar (long titles, same as here).

As I say, I don't like the style much in general, but if we're settling on it as a "PyPA look", then I don't think it's significantly worse here than in other cases. Disclaimer: I'm not a web designer, I'm just going off what I personally like and dislike, which shouldn't be given undue weight.

@pradyunsg
Copy link
Member

I hear you and generally agree.

I feel like we actually need another layer of navigational capabilities for our (generally) larger docs, because this content is actually not structured for how Sphinx RTD theme and Furo handle navigation trees (one big tree that we can present all of). This is different from Furo's own docs, as well as various other projects that have adopted it (attrs, urllib3 being the most prominent examples).

I have some ideas for dealing with the more complex navigation trees, but I'll only be able to get to making something for those ideas after pip 20.3 (and a bit of downtime after that).

(the idea is basically mkdocs-material's tabs approach for splitting the various larger sections)

Base automatically changed from master to main January 21, 2021 18:44
@webknjaz webknjaz force-pushed the maintenance/furo-theme-sphinx branch from c7ebc09 to ed7f3e0 Compare February 3, 2021 18:05
@webknjaz webknjaz added type: enhancement A self-contained enhancement or new feature state: needs clarification labels Feb 21, 2021
@webknjaz webknjaz force-pushed the maintenance/furo-theme-sphinx branch from ed7f3e0 to e260502 Compare February 21, 2021 19:11
@webknjaz
Copy link
Member Author

@pradyunsg anything actionable here?

@pradyunsg
Copy link
Member

I'd hold off on this for a bit. There's a "sibling theme" for Furo that I'm working on that'll likely be a better fit. :)

@webknjaz webknjaz force-pushed the maintenance/furo-theme-sphinx branch 2 times, most recently from 25d5882 to 1538bc9 Compare September 2, 2021 17:56
@webknjaz
Copy link
Member Author

webknjaz commented Sep 2, 2021

@pradyunsg any progress on that other theme? I rebased this PR just in case.

@astrojuanlu
Copy link
Contributor

xref python/docs-community#1 (comment) :)

@webknjaz webknjaz force-pushed the maintenance/furo-theme-sphinx branch 2 times, most recently from 8b5c1e4 to 486e415 Compare December 5, 2021 23:33
@webknjaz webknjaz marked this pull request as draft December 5, 2021 23:33
@webknjaz webknjaz mentioned this pull request Nov 6, 2023
@webknjaz webknjaz force-pushed the maintenance/furo-theme-sphinx branch from 486e415 to 6cd0dc5 Compare November 6, 2023 13:54
@webknjaz webknjaz marked this pull request as ready for review November 6, 2023 13:56
@webknjaz
Copy link
Member Author

webknjaz commented Nov 6, 2023

cc @willingc

@webknjaz webknjaz force-pushed the maintenance/furo-theme-sphinx branch from 6cd0dc5 to f52d232 Compare November 6, 2023 14:00
Copy link
Contributor

@willingc willingc left a comment

Choose a reason for hiding this comment

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

+1 to improved theme

@pradyunsg pradyunsg added this pull request to the merge queue Nov 6, 2023
@sinoroc
Copy link
Contributor

sinoroc commented Nov 6, 2023

Merged via the queue into pypa:main with commit e1adbbd Nov 6, 2023
4 checks passed
@pradyunsg
Copy link
Member

pradyunsg commented Nov 6, 2023

@sinoroc Oooh, I missed that! Let's fix those in follow ups!

@sinoroc
Copy link
Contributor

sinoroc commented Nov 6, 2023

@pradyunsg So, do I understand correctly that local table of contents within the document itself (the .. contents:: directive) are not supported and cause these issues?

@jeanas
Copy link
Contributor

jeanas commented Nov 6, 2023

Yes, that's it. See #1358.

@sinoroc
Copy link
Contributor

sinoroc commented Nov 6, 2023

@pradyunsg It seems to me like we lost the favicon in the transition. Could it be? Or is my browser playing tricks on me?

@pradyunsg
Copy link
Member

#1360 filed for that!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
state: needs clarification type: enhancement A self-contained enhancement or new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants