GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||||||||||||||||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||||||||||||||||||||||||||||||||||
GeezerVison is showing github.com — Switch the theme to 'furo' by pablogsal · Pull Request #679 · python/devguide Conversation
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Why not in this one 😉 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
I added a logo, should render better. Could you double check that it looks good? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
The "copy to clipboard" icons aren't loading for me in the preview (Chrome/macOS): https://cpython-devguide--679.org.readthedocs.build/ Image address is https://cpython-devguide--679.org.readthedocs.build/#_static/copy-button.svg |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
It only happens in the preview, if you build locally they work: |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Aha, okay! And they work on other pages: https://cpython-devguide--679.org.readthedocs.build/documenting/ https://cpython-devguide--679.org.readthedocs.build/_static/copy-button.svg So maybe just not working in the preview homepage. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
CC: @pradyunsg |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Yeah, I am quite sure is some artifact of the preview. I am confident they will work on the deployed version. If not, we can remove the plugin while we investigate. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
And otherwise looks good with a quick scroll through on Chrome/Android too. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
And the logo looks great, but it's a bit big vertically, pushing the menu down to the corner and making it a bit small and squashed to use: |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
One option for handling this is making the entire sidebar scrollable. Example for how to do this is at https://pradyunsg.me/furo/customisation/sidebar/#using-html-sidebars. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
I suggest unsetting pygments_style = "sphinx" although that's obviously a stylistic choice for someone else to decide on. :)
Sorry, something went wrong.
There was an error while loading. Please reload this page.
I think that's likely due to caching with RTD's preview website? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
I stand corrected! It's using the wrong link: That's a bug to file with the extension! |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Ok, I will try that. We also need to remove the ethical adds because we cannot add those without further discussion. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
It looks really nice, and thanks for working on this. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I'm sorry, I re-read what I wrote, and it sounded like I'm saying that the new theme is not trustworthy or not credible, which is not true, and I apologize for that. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
That's would be nice to do in the future, but for now, changing the devguide is an improvement on almost all fronts and can be used in the future as a stepping stone for changing in all other places. With this we get:
I understand the wish to keeps those aligned but I humbly think these benefits outweigh any possible separation: in practical terms is better for the end consumer. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I am sure @willingc can expand on what I said :) |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Hurray! That was the goal! ^>^
Hehe, no need to apologize. I understood what you meant. And as the primary author of Furo, I'm probably the only person who could take offense to that. :)
FWIW, there are similar concerns with moving packaging.python.org as well. All these sites derive a degree of credibility from the fact that they "look like the official Python docs". FWIW, if you're thinking of adopting Furo with some large documentation set... I'd say that sort of thing that would not work, since Furo is designed with the idea of "you can get everywhere from the left sidebar" -- which doesn't work at all for large docs. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
If there's genuine interest in the modernization of the Python documentation and As an honest aside (rant?), I actually initially want to put effort into modernizing python-docs-theme -- which would've dribbled down to pypa-docs-theme and hence pip's documentation (which I'm a maintainer of, and whose documentation was something I wanted to be improve). However, each time in the past that I had proposed making any changes to the Python docs theme, those were shut down pretty quick with a fairly heavy hand. In a similar vein, my experience contributing to the PEPs repo was, straight-up, a bad experience. Based on how long a PR making fair straightforward improvements for the mobile experience has just sat there, I ended up deciding that it's a better investment of effort to just write a separate Sphinx theme -- which is Furo. All this is to say, if there's something around Python documentation that I can help with, I'd be very happy to! ^.^ |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
I think that is a bit of an overstatement, the last comment from a core dev was: python/python-docs-theme#46 (comment) which is 14 days ago. Clarification: I mean this in the sense that the PR is actively being reviewed and is not abandoned or not acknowledged. Is jut that the iteration takes quite a lot. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
IDK, I think it looks quite good and the navigation is not bad: |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Yea, sorry, I didn't mean to imply that it was sitting unattended to -- just that it has been taking a long time. :) |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
|
@willingc @Mariatta @JulienPalard Unfortunately, I don't have the bandwidth to work on this if we are not sure that we are going to move forward, so I would be more comfortable if we can have confirmation that this is the way to go. Please, see the previous discussion regarding why I think we should proceed. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
A few years ago, there was a discussion that we didn't want the dev guide to be the same as the docs.python.org site. I really think a modern theme would be beneficial for many reasons especially making it responsive which is helpful for folks in countries where their access is limited and includes cell phones. Let's do this. Let's use the Furo theme for the Docs WG and make it a high priority to move the existing theme to something responsive and work on CSS to give it a closer look and feel for the footer. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
I agree. I also think I am a bit frustrated that we keep prioritizing a suboptimal status quo again and again in many places based on reasons that in my opinion don't impact the user experience that much or, in the other hand, we strive to produce a perfect result that makes everything great and everyone happy and that stops any small effort that combined could have made us reach that point. This has lead to many existing efforts taking too long or not happening at all. I'm talking about moving the pep repo to sphinx, having dark themes for the python theme, a responsive design for BPO, better highlighting for code blocks... And a large list of improvements that would have objectively made the experience of our users better but at the end have never materialised because of this or they are taking too long (some times even several years). I think we must stop this trend (not specifically with this PR for sure, but the trend in general). And the solution cannot be to promise or plan, it has to be to act and push forward, little by little, transforming imperfect improvements into a really good end goal. And this has to be done with decisiveness in places that are visible so we can leverage the momentum everywhere. Otherwise we will burn the little fuel that we have and the energy of contributors (internal or external) that have energy to propose plans and try to make them a reality. And unfortunately, I personally think this is what's currently happening.
Should I close this PR then? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@pablogsal Please leave it open. I would really like this to be merged, but I want an opportunity to build consensus with Mariatta, Ned, and Julian. Thanks! |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I also completely agree that we need to be helpful for new contributors and committers. Documentation is the gateway to the future. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Thank you for the additional context. I was missing the background of what motivated this change, so it's really helpful to know. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
I'm also frustrated by the slow evolution of the current theme. But I'm also particularly bad at understanding CSS (let alone writing some), this is completly foreign to me, so it has always been hard (and unapealing) for me to make those PR move forward. I'm impressed by your furo theme :) I try to test the PRs though, and try to make them easier to test, but I often find issues (dark theme dispalying a mix of dark and light theme, responsive mode hiding the language switcher, ...) so yes, it's hard and long to reach a mergeable point ☹. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Gentle ping, as 5 weeks has already passed. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I'm merging this, I think this is a good idea, let's move forward! python-docs-theme is evolving, but it still lags behind and with my inexistant HTML/CSS skill I won't be of any help there. For the "element of trust", the domain is enough for me. Having a common style between python.org, docs.python.org, devguide.python.org, wiki.python.org, ... is something I'd like too, but we're far far away from having a common style, so this enhancement will not hurt much. Thanks @pablogsal for the PR and thanks @pradyunsg for the theme, it looks really nice! |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Thanks everyone for working on this! It looks much better on mobile now. Side question: is there some way to manually toggle between dark and light themes? I noticed it automatically detects the browser's theme and chooses light/dark correspondingly, but I've gotten questions from friends with astigmatism who find the dark theme hard to read and want to toggle to the light theme without changing the browser's theme. Sorry if this isn't the right place to bring it up, should I raise this at the furo repo instead? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Woops, thanks for pointing that out :). |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Thanks everyone! Well done 😄 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.