peps icon indicating copy to clipboard operation
peps copied to clipboard

Infra: Move numerical index to own page

Open hugovk opened this issue 1 year ago • 3 comments

Fixes https://github.com/python/peps/issues/2044.

Re: https://github.com/python/peps/issues/2044#issuecomment-1074914852

The frontpage index (PEP 0) is very long, and starts the PEPs grouped by category, followed by a big table sorted by number.

This duplicates the PEP list (there's 655 right now), and can make searching in the page a bit confusing.

This PR creates a new numerical.html page with the numerical index.

I've kept the "Numerical Index" header on the frontpage and added a link to the new page. I moved the header up, before the remaining groups, but don't have a strong opinion on where this should be.


📚 Documentation preview 📚: https://pep-previews--3992.org.readthedocs.build/numerical/

hugovk avatar Sep 24 '24 15:09 hugovk

Thanks both for the reviews!

We now have an empty Contents sidebar:

image

We could remove the header, but some of the titles and author lists are pretty long, so shall we instead let this table be wider? Something like:

image

hugovk avatar Sep 25 '24 14:09 hugovk

so shall we instead let this table be wider

I'd perhaps left-align to where the sidebar is (was), but I would hesistate to make the table much wider than 60-64em (~900-950px), as it becomes difficult to read. For reference, GitHub's central area has a width of ~830px, and Wikipedia's new 'Standard' design has a width of ~950px.

A

AA-Turner avatar Sep 25 '24 15:09 AA-Turner

In local tests I used:

nav#pep-sidebar {
    display: none;
}
@media (min-width: 40em) {
    section#pep-page-section > article {
        width: 85%;
        float: unset;
    }
}
@media (min-width: 60em) {
    section#pep-page-section > article {
        max-width: 62em;
    }
}

AA-Turner avatar Sep 25 '24 20:09 AA-Turner

I had a go with that but the table of contents showed at the bottom of the page for PEP 0 and actual PEPs.

How does https://github.com/python/peps/pull/3992/commits/e439e06027267f8ed6e5559130f2a259229cd705 look?

hugovk avatar Sep 26 '24 19:09 hugovk