| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
This mostly looks good to me, but there are a couple of layout quirks it highlighted:
I also think we're going to have to make some changes to the way the side bar navigation works, as the single top level listing isn't particularly helpful, especially once you start digging down into the Additional Topics section. Looking at http://flask.pocoo.org/docs/0.12/foreword/ for inspiration, I'm wondering how hard it would be to add (below the top level navigation links) sidebar sections for:
|
Sorry, something went wrong.
I'm good with this.
I can change this title here, though personally I'd like the title to be shortened to "packaging projects".
Not hard, but I've been wanting to better organize the files here. If I do those together in a separate PR, it'll be significantly easier. |
Sorry, something went wrong.
|
I promise I'm coming back to this, I've just been sick. |
Sorry, something went wrong.
|
@ncoghlan I've updated this to remove the markers. I'm gonna hold off on changing the name until I do the next PR to organize the docset into subfolders. The staging site is updated: http://temp.theadora.io/pypug/distributing.html Is there anything else I need to do before merging this? |
Sorry, something went wrong.
|
We don't have great communication channels for pre-announcing upcoming visual changes like this, but it's probably worth giving at least a day or two's notice to distutils-sig and a pointer to the discussion in #304 before actually landing the change. |
Sorry, something went wrong.
|
Good call, will do.
…On Mon, May 22, 2017, 7:46 PM Nick Coghlan ***@***.***> wrote:
We don't have great communication channels for pre-announcing upcoming
visual changes like this, but it's probably worth giving at least a day or
two's notice to distutils-sig and a pointer to the discussion in #304
<#304> before
actually landing the change.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#305 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAPUcxxM4QG4a2z9xAZqjM7NCmDOwhrrks5r8khngaJpZM4NGmKB>
.
|
Sorry, something went wrong.
|
I'm not volunteering to do it, so it would be fine to skip this step, but I'm wondering if it may be worth starting to use the PSF blog to pre-announce some of the upcoming changes. See http://pyfound.blogspot.com.au/2017/01/time-to-upgrade-your-python-tls-v12.html for a previous example of that related to a PyPI change. Restructuring and a theme change for the packaging user guide doesn't have the same risk of breaking anything that that did, but it still has significant potential to surprise users when they're expecting the site to look a particular way, and it suddenly starts looking different. |
Sorry, something went wrong.
|
Good idea. Actually, I was thinking about it and I think it might be better
for us to move to the same theme as the Python 3 docs, which is similarly
high contrast and readable and has the benefit of being consistent with the
Python docs. I'll float that idea by distutils-sig/pypa-dev. (Soon)
…On Mon, May 22, 2017, 7:53 PM Nick Coghlan ***@***.***> wrote:
I'm not volunteering to do it, so it would be fine to skip this step, but
I'm wondering if it may be worth starting to use the PSF blog to
pre-announce some of the upcoming changes. See
http://pyfound.blogspot.com.au/2017/01/time-to-upgrade-your-python-tls-v12.html
for a previous example of that related to a PyPI change.
Restructuring and a theme change for the packaging user guide doesn't have
the same risk of *breaking* anything that that did, but it still has
significant potential to surprise users when they're expecting the site to
look a particular way, and it suddenly starts looking different.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#305 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAPUc_gXpZIsJX4nlyDrO574Ae-q3cXhks5r8kosgaJpZM4NGmKB>
.
|
Sorry, something went wrong.
|
@ncoghlan this has been updated to use pydoctheme. I personally like the way it turned out. New staging here: http://temp.theadora.io/pypug-pydoctheme/index.html I'll send off an email to distutils-sig tomorrow to get some discussion going. :) |
Sorry, something went wrong.
|
Alright, cool. This is finally ready for review, but please do not merge.
|
Sorry, something went wrong.
|
I added the [WIP] prefix to indicate the "Do not merge" status. General idea and approach still sounds good to me :) |
Sorry, something went wrong.
|
👍 thanks
…On Thu, May 25, 2017, 11:24 PM Nick Coghlan ***@***.***> wrote:
I added the [WIP] prefix to indicate the "Do not merge" status.
General idea and approach still sounds good to me :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#305 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAPUcydTlVSBJ3wtkdx9mTlI8V7ZJKFhks5r9nAjgaJpZM4NGmKB>
.
|
Sorry, something went wrong.
|
I wonder if you should check with python-dev. If I recall discussions about the new theme correctly, one of the goals was to have a distinctive theme for CPython. |
Sorry, something went wrong.
|
@merwok good point, I couldn't find any recent discussions. @ncoghlan is a core developer and was one of the (many) people asking to align with pydoctheme, so I'm mostly going off his guidance. Nick, do you know if we would run awry of the core team / PSF if we use the theme? (I know we already have branding permission) |
Sorry, something went wrong.
|
Note that the PyPA/CPython overlap is more than just me - it includes @dstufft and @pfmoore as well :) I think PEP 453's bundling of pip (and the policies around that: https://www.python.org/dev/peps/pep-0453/#policies-governance), together with the use of a python.org subdomain for the packaging user guide itself, and the organisational backing of the PSF's Packaging Working Group, mean it's reasonable to re-use the theme to better reflect that collaborative governance. It's probably worth asking on python-dev about preferred names for the theme package, though - it may make more sense to call it "psf-docs-theme" and suggest that folks avoid using it if their project governance structure doesn't run directly through the PSF. |
Sorry, something went wrong.
|
python-dev thread regarding pypa-theme vs psf-docs-theme vs both: https://mail.python.org/pipermail/python-dev/2017-May/148029.html |
Sorry, something went wrong.
|
Looking up something on the Python docs on my phone this morning alerted me to a somewhat serious step back compared to the RtD theme: it's much more difficult to navigate the docs on mobile, compared to the RtD theme's menu drawer. All of the sidebar, header, and footer text is awfully small—I had a hard time with it. Some screenshots, to compare: I'm not sure that it's a showstopper necessarily (I don't know what, if any, work is being done on making the theme more small-screen friendly), but it seemed like something I ought to call attention to. |
Sorry, something went wrong.
|
I don't think the current pydocs theme has had any work done on it to make it responsive at all, so if there are some CSS changes that could be taken from the Alabaster or RtD themes to improve mobile readability, that could potentially be useful for the CPython docs as well. |
Sorry, something went wrong.
Yep, this is true. I'm not sure how much weight we want to put on this - I'm not sure what kind of person reads Python packaging documentation on their phone :D That said, if we do go the route of python/pydoctheme -> pypa/pypa-theme, it's possible some one could add responsiveness upstream and we can benefit from that easily. :) |
Sorry, something went wrong.
|
I merged python/python-docs-theme#1, so the common theme should now be pip-installable via: git+https://github.com/python/python-docs-theme.git#egg=python-docs-theme |
Sorry, something went wrong.
|
@jonparrott No opposition from me, it just means it needs to be copied in as a file, rather than selected from the dropdown that GitHub provide during repo creation :) |
Sorry, something went wrong.
|
@ncoghlan Hahaha, laziness wins every time. I should (hopefully) have some time tomorrow afternoon to populate that repo and get this PR updated and ready to merge. :) |
Sorry, something went wrong.
|
Once pypa/pypa-docs-theme#1 is merged, I'll be able to update this to use our common theme and then it'll be ready to merge. :) |
Sorry, something went wrong.
|
Alright, this is ready for final review & merge. @ncoghlan @dstufft Staging is here: http://temp.theadora.io/pypug-pydoctheme/index.html |
Sorry, something went wrong.
There was a problem hiding this comment.
LGTM! I'll do a quick post to distutils-sig about it.
Sorry, something went wrong.
Thanks! Merging, but if @dstufft has any post-hoc comments I'll address them in a follow-up PR. :) |
Sorry, something went wrong.
|
Hi! Just wanted to point out some things I spotted. These are probably just nits. On http://temp.theadora.io/pypug-pydoctheme/tutorials/index.html, the part that shows that adds contrast to the "«" for collapsing the sidebar does not extend till that character, making it barely visible unless you look for it. Also, the same "«" isn't staying in the (vertical) center as I scroll down, it doesn't move. I think it does on the CPython docs currently. Where do I make issues for these - on the python org or the pypa org? |
Sorry, something went wrong.
|
@pradyunsg huh, good catch. I'll track that down now, I think there was just some wires crossed with the new collapsable sidebar. |
Sorry, something went wrong.
|
@jonparrott Thanks! ^.^ If there's anything else, I'll make issues on the PyPA theme? |
Sorry, something went wrong.
|
Yep, please do! Thank you. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Towards #304