Install page: Show toc and add older versions link

Review Request #834 — Created July 25, 2021 and submitted

Information

pidgin/nest
default
28616283f35e

Reviewers

Modified the install page so that it now shows:
- The table of contents at the beginning of the page. Had to modify the
  markdown a bit to make this appear (it only shows it if there's no first level
  heading in the md file).
- Added a new section to tell users where they can find the installers/binaries
  for older versions of Pidgin/Finch.

Ran dev-server.sh and verified content looked as intended

sorvival
sorvival
sorvival
  1. 
      
  2. This is how the changes look when rendered:

    Image

  3. 
      
jrchamp
  1. While the goal was to get a Table of Contents, this is a big accessibility win as far as document structure. The general consensus is that a page should only have one H1 and that it should be the page's title. This change addresses that issue for the install page.

    The only other file with a potentially problematic H1 is development/building/2.x.y/_index.md

    1. Nice find, we can fix that one as well but to keep things simple let's do it in a separate review.

  2. hugo/content/install/_index.md (Diff revision 3)
     
     

    The original wording is good, though I would probably add a comma at each of the two pauses. In case it helps, here's an alternative wording.

    If you need binaries for older versions of Pidgin, visit our [package repository main page](https://sourceforge.net/projects/pidgin/files/Pidgin). That is also where you can find the checksums for each file and their corresponding gpg signatures.
    
    1. Thanks for the suggestion, I applied it in the latest rev! :)

  3. 
      
sorvival
grim
  1. Ship It!
  2. Nice work! Thanks!!

  3. 
      
grim
Review request changed

Status: Closed (submitted)

Loading...